Tarantool discussions archive
 help / color / mirror / Atom feed
From: "Ilya Kosarev" <i.kosarev@tarantool.org>
To: "Konstantin Osipov" <kostja.osipov@gmail.com>, dev@tarantool.org
Cc: tarantool-discussions <tarantool-discussions@dev.tarantool.org>
Subject: Re: [Tarantool-discussions] [dev]  [rfc] iproto connections processing improvements
Date: Thu, 02 Jul 2020 17:22:49 +0300	[thread overview]
Message-ID: <1593699769.684917525@f473.i.mail.ru> (raw)
In-Reply-To: <20200630114713.GB272113@atlas>

[-- Attachment #1: Type: text/plain, Size: 1836 bytes --]


As a result of private discussion, here are the steps to be
implemented:
1. Greeting should be done by iproto solely. This means session
creation has to be moved to a later point (after iproto_msg_decode).
Thus iproto has to be able to reach iproto_msg_decode without tx
assistance. Iproto also should be able to finish connection itself in
case it is possible (connection being rejected by iproto).
2. Introduce state machine managed from tx. tx should be able to enable
different iproto states depending on tx work phase, for example, to
reject all connections on secondary index build.
3. To be more specific, we need to be able to classify different types
of connections, for example, replica connection vs client connection.
This means we need to add specific flag for replica authentication and
prioritize it if needed depending on the iproto state.
4. New approach to connections handling means we need to reconsider
clients behavior: specific error for this rejection type, reconnection
on timeout.
 
--
Ilya Kosarev 
>Вторник, 30 июня 2020, 14:47 +03:00 от Konstantin Osipov <kostja.osipov@gmail.com>:
> 
>* Ilya Kosarev < i.kosarev@tarantool.org > [20/06/30 14:19]:
>>
>> Ссылки были заинлайнены под слова.
>> Не учёл, что это работает не во всех клиентах.
>>  https://github.com/tarantool/tarantool/issues/3776
>>  https://github.com/tarantool/tarantool/issues/4646
>>  https://github.com/tarantool/tarantool/issues/4910
>>  
>> Про vtab/состояния конечно вопрос отдельный, я написал, как это вижу.
>>  
>> В iproto_msg_decode как раз есть возможность отделить одно от другого.
>Как именно?
>
>
>--
>Konstantin Osipov, Moscow, Russia
 

[-- Attachment #2: Type: text/html, Size: 2652 bytes --]

      reply	other threads:[~2020-07-02 14:22 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-05-29 13:47 [Tarantool-discussions] [server-dev] " Ilya Kosarev
     [not found] ` <20200529145512.GA189726@atlas>
2020-06-30 11:18   ` Ilya Kosarev
2020-06-30 11:47     ` Konstantin Osipov
2020-07-02 14:22       ` Ilya Kosarev [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1593699769.684917525@f473.i.mail.ru \
    --to=i.kosarev@tarantool.org \
    --cc=dev@tarantool.org \
    --cc=kostja.osipov@gmail.com \
    --cc=tarantool-discussions@dev.tarantool.org \
    --subject='Re: [Tarantool-discussions] [dev]  [rfc] iproto connections processing improvements' \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox