... and this is the main PR: https://github.com/apache/zookeeper/pull/1832

However, it seems that ZK 3.3.x may be broken and the jute protocol cannot
cover the case. You can review the PR and I already commented details.

Best,
tison.


tison <wander4...@gmail.com> 于2022年2月28日周一 09:33写道:

> Hi,
>
> Thanks for eolivelli's review and approval, this PR[1] is waiting for
> another reviewer to proceed. I'd like to bump this thread to see if any
> committer could help on reviewing :)
>
> Best,
> tison.
>
> [1] https://github.com/apache/zookeeper/pull/1826
>
>
> Enrico Olivelli <eolive...@gmail.com> 于2022年2月24日周四 23:20写道:
>
>> Tison
>>
>> Il Gio 24 Feb 2022, 15:29 tison <wander4...@gmail.com> ha scritto:
>>
>> > Here is an initial PR[1] you can give a review. I think the script is
>> > enough for the original purpose merging readOnly field.
>> >
>>
>> Very good
>> I left one comment
>>
>>
>>
>> Enrico
>>
>>
>>
>> > Best,
>> > tison.
>> >
>> > [1] https://github.com/apache/zookeeper/pull/1826
>> >
>> >
>> > Enrico Olivelli <eolive...@gmail.com> 于2022年2月23日周三 20:46写道:
>> >
>> > > Il Mer 23 Feb 2022, 10:46 tison <wander4...@gmail.com> ha scritto:
>> > >
>> > > > Hi Enrico,
>> > > >
>> > > > Thanks for your reply! Do we have end to end tests for the same
>> version
>> > > of
>> > > > client and server now?
>> > > >
>> > >
>> > > We haven't.
>> > > We only use the local code to run both the client and the server
>> > >
>> > > Enrico
>> > >
>> > >
>> > > If we already have such tests, then wrapping them among different
>> > versions
>> > > > is possible. Otherwise,
>> > > > we may add such end to end tests first XD
>> > > >
>> > > > Best,
>> > > > tison.
>> > > >
>> > > >
>> > > > Enrico Olivelli <eolive...@gmail.com> 于2022年2月22日周二 02:14写道:
>> > > >
>> > > > > I missed this thread.
>> > > > > If you manage to keep full compatibility with old clients then I
>> am
>> > +1
>> > > > >
>> > > > > We are missing compatibility tests, it may be a good time to start
>> > such
>> > > > > suite.
>> > > > > We can start by running the client (bash cli) in a docker
>> container
>> > > > > probably.
>> > > > >
>> > > > >
>> > > > > Enrico
>> > > > >
>> > > > > Il Lun 21 Feb 2022, 17:47 tison <wander4...@gmail.com> ha
>> scritto:
>> > > > >
>> > > > > > Bump the thread for one last try to see if any zookeeper is
>> > > interested
>> > > > in
>> > > > > > this topic.
>> > > > > >
>> > > > > > Best,
>> > > > > > tison.
>> > > > > >
>> > > > >
>> > > >
>> > >
>> >
>>
>

Reply via email to