From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp48.i.mail.ru (smtp48.i.mail.ru [94.100.177.108]) (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 BA2DA469710 for ; Tue, 19 May 2020 13:00:01 +0300 (MSK) Date: Tue, 19 May 2020 13:00:00 +0300 From: Kirill Yukhin Message-ID: <20200519100000.q7sqn3riiopxtuvv@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 1/1] session: remove box.session.push() 'sync' 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 04 май 23:42, Vladislav Shpilevoy wrote: > Closes #4689 > > @TarantoolBot document > Title: box.session.push() 'sync' is deprecated > > box.session.push() had two parameters - data to push and 'sync'. > The sync was a request ID with which the out of bound data should > be pushed into a socket. > > This was introduced as a workaround for #3450, and is useless > since its resolution. > > A user anyway can't push to different sessions, where that > parameter could be useful. And pushing into requests of the same > session, on the contrary, is something not really needed anywhere, > not portable to non-binary session types (console, background), > and is just dangerous since it is easy to add a bug here. > > The patch removes the parameter. Now there will be thrown a > 'Usage' error at attempt to use 'sync' parameter. In version 2.4 > it is deprecated, prints warnings into logs, but still works. In > 2.5 it is removed completely. I've checked your patch into master. -- Regards, Kirill Yukhin