Re: Fix slot synchronization with two_phase decoding enabled

On Tue, Jun 3, 2025 at 11:05 AM Nisha Moond <nisha.moond412@gmail.com> wrote:
>
>
> Attached v17 patches. Added a top-up patch 0002 implementing the idea
> suggested by Amit above.

I have started reviewing this, although I haven't done a complete
review yet, but I have a question on the fix we are trying to do, IIUC
we are disallowing to use 'two phase' and 'failover' options together
at the create slot time and now users has to create slot with one of
the option and later enable other option right (if user want to use
both options)? But don't you think it will affect usability? because
if a user wants to use both the options together then after creating
the slot they need to track when is the right time to enable the other
option?  Not sure if anyone else has this concern or it's just me?

--
Regards,
Dilip Kumar
Google