From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp49.i.mail.ru (smtp49.i.mail.ru [94.100.177.109]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dev.tarantool.org (Postfix) with ESMTPS id 6028346970F for ; Thu, 21 Nov 2019 20:38:07 +0300 (MSK) Date: Thu, 21 Nov 2019 20:38:06 +0300 From: Kirill Yukhin Message-ID: <20191121173806.yykujoewgiuliks6@tarantool.org> References: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: Subject: Re: [Tarantool-patches] [PATCH 0/3] Replication status errno List-Id: Tarantool development patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Vladislav Shpilevoy Cc: tarantool-patches@dev.tarantool.org Hello, On 05 ноя 17:59, Vladislav Shpilevoy wrote: > The patchset makes box.info.replication*.upstream/downstream show > errno description for system errors. > > I made that it shows strerror(errno) of the latest system error, > but probably it would be better to show errno value, and let a > user to translate it manually using Lua errno module. On the other > hand my way is simpler for a user when he manually calls box.info > in a console to check a replication status. That is discussable. > > Branch: http://github.com/tarantool/tarantool/tree/gerold103/gh-4402-replication-system-error > Issue: https://github.com/tarantool/tarantool/issues/4402 I've checked your patch set into master. -- Regards, Kirill Yukhin