From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pg1-f194.google.com (mail-pg1-f194.google.com [209.85.215.194]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by dev.tarantool.org (Postfix) with ESMTPS id A4D1E452566 for ; Tue, 5 Nov 2019 21:13:41 +0300 (MSK) Received: by mail-pg1-f194.google.com with SMTP id k13so3510492pgh.3 for ; Tue, 05 Nov 2019 10:13:41 -0800 (PST) Date: Tue, 5 Nov 2019 21:13:37 +0300 From: Konstantin Osipov Message-ID: <20191105181337.GJ29784@atlas> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline 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 * Vladislav Shpilevoy [19/11/05 17:57]: > 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. Your approach is fine. -- Konstantin Osipov, Moscow, Russia