You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@shenyu.apache.org by Han G <dh...@hotmail.com> on 2022/05/02 04:28:39 UTC
Re: [Question] The data saved in zookeeper looks strange, it has strange number because of serialization method.
Hi community,
Based on the issue mentioned in earlier mail, I’d like to fix it using plane byte without magic header, which will make it easy for other clients to register to ShenYu. But please pay attention that after this change, customers who are using zookeeper as register center or data sync center need to re-register to zookeeper.
I also plan to take this change to replace current zk client (com.101tec.zkclient) with Curator 5.2.1. But Curator 5.2.1 is not compatible with zookeeper 3.4.x version and before. Please let me know if any concern. Thanks a lot!
Related issue:
https://github.com/apache/incubator-shenyu/issues/3362
Thanks,
Han
From: Han G <dh...@hotmail.com>
Date: Thursday, April 28, 2022 at 17:45
To: dev <de...@shenyu.apache.org>
Subject: [Question] The data saved in zookeeper looks strange, it has strange number because of serialization method.
Hi folks,
I also created an ISSUE for this question, see https://github.com/apache/incubator-shenyu/issues/3360.
I'm working on the zookeeper client registration for dotnet client, and I found the serialization for zkClient looks strange, see below.
[Pasted image 20220428170506]<https://user-images.githubusercontent.com/11908658/165719204-df04cfc5-80a6-4bc5-b602-6a62d0f3dfab.png>
And admin also use the same serializer which is default one SerializableSerializer in package org.I0Itec.zkclient, which makes the format below cannot be deserialized correctly.
[image]<https://user-images.githubusercontent.com/11908658/165719634-c5b8d3ea-7ffd-4e03-a09a-26e9abffe6bf.png>
And will cause below exception.
[Pasted image 20220428170748]<https://user-images.githubusercontent.com/11908658/165719822-27e7992c-3132-44c5-843d-438812301432.png>
org.I0Itec.zkclient also has a byte serializer, and I've tested that it works well and has no strange number when writing in zookeeper.
I'm wondering if it's by design or a bug. Because I can't find the similar serializer in zookeeper official package or sdks in other language.
If it's a bug, I can have a try to fix it and using the byte serializer instead. It will change all the places where ZkClient was used. The changes mainly looks like below, and also including some other small changes.
-this.zkClient = new ZkClient(config.getServerLists(), sessionTimeout, connectionTimeout);
+this.zkClient = new ZkClient(config.getServerLists(), sessionTimeout, connectionTimeout, new BytesPushThroughSerializer());
If it's by design, I will try to investigate if any mitigation to solve this strange number issue.
Thanks,
Han
Re: [Question] The data saved in zookeeper looks strange, it has strange number because of serialization method.
Posted by XiaoYu <xi...@apache.org>.
good， it be merged
Han G <dh...@hotmail.com> 于2022年5月10日周二 16:39写道：
>
> Appreciate Yu for the quick response! I also created a PR to update the related documents. Please have a look, thank you!
> https://github.com/apache/incubator-shenyu-website/pull/564
>
> Thanks,
> Han
>
>
> From: XiaoYu <xi...@apache.org>
> Date: Tuesday, May 10, 2022 at 11:41
> To: dev@shenyu.apache.org <de...@shenyu.apache.org>
> Subject: Re: [Question] The data saved in zookeeper looks strange, it has strange number because of serialization method.
> Hi
>
> looks good for me，it merged into master
>
> thanks for your contributor
>
> Han G <dh...@hotmail.com> 于2022年5月10日周二 10:23写道：
> >
> > Hi folks, I also implemented the data sync part in this PR. Please help to review. Thank you for your time!
> >
> > Thanks,
> > Han
> >
> >
> > From: Han G <dh...@hotmail.com>
> > Date: Friday, May 6, 2022 at 11:33
> > To: dev@shenyu.apache.org <de...@shenyu.apache.org>
> > Subject: Re: [Question] The data saved in zookeeper looks strange, it has strange number because of serialization method.
> > Hi Yu, sure! After these changes, I can create PR to mention it in documentation.
> >
> > Since there’re too many changes, including registration in server and client, registration instance, data sync, I will use 2 PRs to finish the change.
> >
> > The first PR includes registration in server and client, registration instance and the UTs. I’ve already finished it. https://github.com/apache/incubator-shenyu/pull/3388
> >
> > The second PR will include data sync between admin and bootstrap.
> >
> > Thanks,
> > Han
> >
> >
> > From: XiaoYu <xi...@apache.org>
> > Date: Thursday, May 5, 2022 at 18:02
> > To: dev@shenyu.apache.org <de...@shenyu.apache.org>
> > Subject: Re: [Question] The data saved in zookeeper looks strange, it has strange number because of serialization method.
> > Hi
> >
> > we should used Curator 5.X
> >
> > if user want to this fuction，We should tell users in the documentation that
> > they should use a higher version of zk（3.6.x）
> >
> > Han G <dh...@hotmail.com> 于2022年5月2日周一 12:29写道：
> >
> > > Hi community,
> > >
> > >
> > >
> > > Based on the issue mentioned in earlier mail, I’d like to fix it using
> > > plane byte without magic header, which will make it easy for other clients
> > > to register to ShenYu. But please pay attention that after this change,
> > > customers who are using zookeeper as register center or data sync center
> > > need to re-register to zookeeper.
> > >
> > >
> > >
> > > I also plan to take this change to replace current zk client
> > > (com.101tec.zkclient) with Curator 5.2.1. But Curator 5.2.1 is not
> > > compatible with zookeeper 3.4.x version and before. Please let me know if
> > > any concern. Thanks a lot!
> > >
> > >
> > >
> > > Related issue:
> > >
> > > https://github.com/apache/incubator-shenyu/issues/3362
> > >
> > >
> > >
> > > Thanks,
> > >
> > > Han
> > >
> > >
> > >
> > >
> > >
> > > *From: *Han G <dh...@hotmail.com>
> > > *Date: *Thursday, April 28, 2022 at 17:45
> > > *To: *dev <de...@shenyu.apache.org>
> > > *Subject: *[Question] The data saved in zookeeper looks strange, it has
> > > strange number because of serialization method.
> > >
> > > Hi folks,
> > >
> > >
> > >
> > > I also created an ISSUE for this question, see
> > > https://github.com/apache/incubator-shenyu/issues/3360.
> > >
> > >
> > >
> > > I'm working on the zookeeper client registration for dotnet client, and I
> > > found the serialization for zkClient looks strange, see below.
> > > [image: Pasted image 20220428170506]
> > > <https://user-images.githubusercontent.com/11908658/165719204-df04cfc5-80a6-4bc5-b602-6a62d0f3dfab.png>
> > >
> > >
> > >
> > > And admin also use the same serializer which is default one
> > > SerializableSerializer in package org.I0Itec.zkclient, which makes the
> > > format below cannot be deserialized correctly.
> > > [image: image]
> > > <https://user-images.githubusercontent.com/11908658/165719634-c5b8d3ea-7ffd-4e03-a09a-26e9abffe6bf.png>
> > >
> > >
> > >
> > > And will cause below exception.
> > > [image: Pasted image 20220428170748]
> > > <https://user-images.githubusercontent.com/11908658/165719822-27e7992c-3132-44c5-843d-438812301432.png>
> > >
> > > org.I0Itec.zkclient also has a byte serializer, and I've tested that it
> > > works well and has no strange number when writing in zookeeper.
> > >
> > >
> > >
> > > I'm wondering if it's by design or a bug. Because I can't find the
> > > similar serializer in zookeeper official package or sdks in other language.
> > >
> > > If it's a bug, I can have a try to fix it and using the byte serializer
> > > instead. It will change all the places where ZkClient was used. The
> > > changes mainly looks like below, and also including some other small
> > > changes.
> > >
> > >
> > >
> > > -this.zkClient = new ZkClient(config.getServerLists(), sessionTimeout,
> > > connectionTimeout);
> > >
> > > +this.zkClient = new ZkClient(config.getServerLists(), sessionTimeout,
> > > connectionTimeout, new BytesPushThroughSerializer());
> > >
> > >
> > >
> > > If it's by design, I will try to investigate if any mitigation to solve
> > > this strange number issue.
> > >
> > >
> > >
> > > Thanks,
> > >
> > > Han
> > >
> > >
> > >
Re: [Question] The data saved in zookeeper looks strange, it has strange number because of serialization method.
Posted by Han G <dh...@hotmail.com>.
Appreciate Yu for the quick response! I also created a PR to update the related documents. Please have a look, thank you!
https://github.com/apache/incubator-shenyu-website/pull/564
Thanks,
Han
From: XiaoYu <xi...@apache.org>
Date: Tuesday, May 10, 2022 at 11:41
To: dev@shenyu.apache.org <de...@shenyu.apache.org>
Subject: Re: [Question] The data saved in zookeeper looks strange, it has strange number because of serialization method.
Hi
looks good for me，it merged into master
thanks for your contributor
Han G <dh...@hotmail.com> 于2022年5月10日周二 10:23写道：
>
> Hi folks, I also implemented the data sync part in this PR. Please help to review. Thank you for your time!
>
> Thanks,
> Han
>
>
> From: Han G <dh...@hotmail.com>
> Date: Friday, May 6, 2022 at 11:33
> To: dev@shenyu.apache.org <de...@shenyu.apache.org>
> Subject: Re: [Question] The data saved in zookeeper looks strange, it has strange number because of serialization method.
> Hi Yu, sure! After these changes, I can create PR to mention it in documentation.
>
> Since there’re too many changes, including registration in server and client, registration instance, data sync, I will use 2 PRs to finish the change.
>
> The first PR includes registration in server and client, registration instance and the UTs. I’ve already finished it. https://github.com/apache/incubator-shenyu/pull/3388
>
> The second PR will include data sync between admin and bootstrap.
>
> Thanks,
> Han
>
>
> From: XiaoYu <xi...@apache.org>
> Date: Thursday, May 5, 2022 at 18:02
> To: dev@shenyu.apache.org <de...@shenyu.apache.org>
> Subject: Re: [Question] The data saved in zookeeper looks strange, it has strange number because of serialization method.
> Hi
>
> we should used Curator 5.X
>
> if user want to this fuction，We should tell users in the documentation that
> they should use a higher version of zk（3.6.x）
>
> Han G <dh...@hotmail.com> 于2022年5月2日周一 12:29写道：
>
> > Hi community,
> >
> >
> >
> > Based on the issue mentioned in earlier mail, I’d like to fix it using
> > plane byte without magic header, which will make it easy for other clients
> > to register to ShenYu. But please pay attention that after this change,
> > customers who are using zookeeper as register center or data sync center
> > need to re-register to zookeeper.
> >
> >
> >
> > I also plan to take this change to replace current zk client
> > (com.101tec.zkclient) with Curator 5.2.1. But Curator 5.2.1 is not
> > compatible with zookeeper 3.4.x version and before. Please let me know if
> > any concern. Thanks a lot!
> >
> >
> >
> > Related issue:
> >
> > https://github.com/apache/incubator-shenyu/issues/3362
> >
> >
> >
> > Thanks,
> >
> > Han
> >
> >
> >
> >
> >
> > *From: *Han G <dh...@hotmail.com>
> > *Date: *Thursday, April 28, 2022 at 17:45
> > *To: *dev <de...@shenyu.apache.org>
> > *Subject: *[Question] The data saved in zookeeper looks strange, it has
> > strange number because of serialization method.
> >
> > Hi folks,
> >
> >
> >
> > I also created an ISSUE for this question, see
> > https://github.com/apache/incubator-shenyu/issues/3360.
> >
> >
> >
> > I'm working on the zookeeper client registration for dotnet client, and I
> > found the serialization for zkClient looks strange, see below.
> > [image: Pasted image 20220428170506]
> > <https://user-images.githubusercontent.com/11908658/165719204-df04cfc5-80a6-4bc5-b602-6a62d0f3dfab.png>
> >
> >
> >
> > And admin also use the same serializer which is default one
> > SerializableSerializer in package org.I0Itec.zkclient, which makes the
> > format below cannot be deserialized correctly.
> > [image: image]
> > <https://user-images.githubusercontent.com/11908658/165719634-c5b8d3ea-7ffd-4e03-a09a-26e9abffe6bf.png>
> >
> >
> >
> > And will cause below exception.
> > [image: Pasted image 20220428170748]
> > <https://user-images.githubusercontent.com/11908658/165719822-27e7992c-3132-44c5-843d-438812301432.png>
> >
> > org.I0Itec.zkclient also has a byte serializer, and I've tested that it
> > works well and has no strange number when writing in zookeeper.
> >
> >
> >
> > I'm wondering if it's by design or a bug. Because I can't find the
> > similar serializer in zookeeper official package or sdks in other language.
> >
> > If it's a bug, I can have a try to fix it and using the byte serializer
> > instead. It will change all the places where ZkClient was used. The
> > changes mainly looks like below, and also including some other small
> > changes.
> >
> >
> >
> > -this.zkClient = new ZkClient(config.getServerLists(), sessionTimeout,
> > connectionTimeout);
> >
> > +this.zkClient = new ZkClient(config.getServerLists(), sessionTimeout,
> > connectionTimeout, new BytesPushThroughSerializer());
> >
> >
> >
> > If it's by design, I will try to investigate if any mitigation to solve
> > this strange number issue.
> >
> >
> >
> > Thanks,
> >
> > Han
> >
> >
> >
Re: [Question] The data saved in zookeeper looks strange, it has strange number because of serialization method.
Posted by XiaoYu <xi...@apache.org>.
Hi
looks good for me，it merged into master
thanks for your contributor
Han G <dh...@hotmail.com> 于2022年5月10日周二 10:23写道：
>
> Hi folks, I also implemented the data sync part in this PR. Please help to review. Thank you for your time!
>
> Thanks,
> Han
>
>
> From: Han G <dh...@hotmail.com>
> Date: Friday, May 6, 2022 at 11:33
> To: dev@shenyu.apache.org <de...@shenyu.apache.org>
> Subject: Re: [Question] The data saved in zookeeper looks strange, it has strange number because of serialization method.
> Hi Yu, sure! After these changes, I can create PR to mention it in documentation.
>
> Since there’re too many changes, including registration in server and client, registration instance, data sync, I will use 2 PRs to finish the change.
>
> The first PR includes registration in server and client, registration instance and the UTs. I’ve already finished it. https://github.com/apache/incubator-shenyu/pull/3388
>
> The second PR will include data sync between admin and bootstrap.
>
> Thanks,
> Han
>
>
> From: XiaoYu <xi...@apache.org>
> Date: Thursday, May 5, 2022 at 18:02
> To: dev@shenyu.apache.org <de...@shenyu.apache.org>
> Subject: Re: [Question] The data saved in zookeeper looks strange, it has strange number because of serialization method.
> Hi
>
> we should used Curator 5.X
>
> if user want to this fuction，We should tell users in the documentation that
> they should use a higher version of zk（3.6.x）
>
> Han G <dh...@hotmail.com> 于2022年5月2日周一 12:29写道：
>
> > Hi community,
> >
> >
> >
> > Based on the issue mentioned in earlier mail, I’d like to fix it using
> > plane byte without magic header, which will make it easy for other clients
> > to register to ShenYu. But please pay attention that after this change,
> > customers who are using zookeeper as register center or data sync center
> > need to re-register to zookeeper.
> >
> >
> >
> > I also plan to take this change to replace current zk client
> > (com.101tec.zkclient) with Curator 5.2.1. But Curator 5.2.1 is not
> > compatible with zookeeper 3.4.x version and before. Please let me know if
> > any concern. Thanks a lot!
> >
> >
> >
> > Related issue:
> >
> > https://github.com/apache/incubator-shenyu/issues/3362
> >
> >
> >
> > Thanks,
> >
> > Han
> >
> >
> >
> >
> >
> > *From: *Han G <dh...@hotmail.com>
> > *Date: *Thursday, April 28, 2022 at 17:45
> > *To: *dev <de...@shenyu.apache.org>
> > *Subject: *[Question] The data saved in zookeeper looks strange, it has
> > strange number because of serialization method.
> >
> > Hi folks,
> >
> >
> >
> > I also created an ISSUE for this question, see
> > https://github.com/apache/incubator-shenyu/issues/3360.
> >
> >
> >
> > I'm working on the zookeeper client registration for dotnet client, and I
> > found the serialization for zkClient looks strange, see below.
> > [image: Pasted image 20220428170506]
> > <https://user-images.githubusercontent.com/11908658/165719204-df04cfc5-80a6-4bc5-b602-6a62d0f3dfab.png>
> >
> >
> >
> > And admin also use the same serializer which is default one
> > SerializableSerializer in package org.I0Itec.zkclient, which makes the
> > format below cannot be deserialized correctly.
> > [image: image]
> > <https://user-images.githubusercontent.com/11908658/165719634-c5b8d3ea-7ffd-4e03-a09a-26e9abffe6bf.png>
> >
> >
> >
> > And will cause below exception.
> > [image: Pasted image 20220428170748]
> > <https://user-images.githubusercontent.com/11908658/165719822-27e7992c-3132-44c5-843d-438812301432.png>
> >
> > org.I0Itec.zkclient also has a byte serializer, and I've tested that it
> > works well and has no strange number when writing in zookeeper.
> >
> >
> >
> > I'm wondering if it's by design or a bug. Because I can't find the
> > similar serializer in zookeeper official package or sdks in other language.
> >
> > If it's a bug, I can have a try to fix it and using the byte serializer
> > instead. It will change all the places where ZkClient was used. The
> > changes mainly looks like below, and also including some other small
> > changes.
> >
> >
> >
> > -this.zkClient = new ZkClient(config.getServerLists(), sessionTimeout,
> > connectionTimeout);
> >
> > +this.zkClient = new ZkClient(config.getServerLists(), sessionTimeout,
> > connectionTimeout, new BytesPushThroughSerializer());
> >
> >
> >
> > If it's by design, I will try to investigate if any mitigation to solve
> > this strange number issue.
> >
> >
> >
> > Thanks,
> >
> > Han
> >
> >
> >
Re: [Question] The data saved in zookeeper looks strange, it has strange number because of serialization method.
Posted by Han G <dh...@hotmail.com>.
Hi folks, I also implemented the data sync part in this PR. Please help to review. Thank you for your time!
Thanks,
Han
From: Han G <dh...@hotmail.com>
Date: Friday, May 6, 2022 at 11:33
To: dev@shenyu.apache.org <de...@shenyu.apache.org>
Subject: Re: [Question] The data saved in zookeeper looks strange, it has strange number because of serialization method.
Hi Yu, sure! After these changes, I can create PR to mention it in documentation.
Since there’re too many changes, including registration in server and client, registration instance, data sync, I will use 2 PRs to finish the change.
The first PR includes registration in server and client, registration instance and the UTs. I’ve already finished it. https://github.com/apache/incubator-shenyu/pull/3388
The second PR will include data sync between admin and bootstrap.
Thanks,
Han
From: XiaoYu <xi...@apache.org>
Date: Thursday, May 5, 2022 at 18:02
To: dev@shenyu.apache.org <de...@shenyu.apache.org>
Subject: Re: [Question] The data saved in zookeeper looks strange, it has strange number because of serialization method.
Hi
we should used Curator 5.X
if user want to this fuction，We should tell users in the documentation that
they should use a higher version of zk（3.6.x）
Han G <dh...@hotmail.com> 于2022年5月2日周一 12:29写道：
> Hi community,
>
>
>
> Based on the issue mentioned in earlier mail, I’d like to fix it using
> plane byte without magic header, which will make it easy for other clients
> to register to ShenYu. But please pay attention that after this change,
> customers who are using zookeeper as register center or data sync center
> need to re-register to zookeeper.
>
>
>
> I also plan to take this change to replace current zk client
> (com.101tec.zkclient) with Curator 5.2.1. But Curator 5.2.1 is not
> compatible with zookeeper 3.4.x version and before. Please let me know if
> any concern. Thanks a lot!
>
>
>
> Related issue:
>
> https://github.com/apache/incubator-shenyu/issues/3362
>
>
>
> Thanks,
>
> Han
>
>
>
>
>
> *From: *Han G <dh...@hotmail.com>
> *Date: *Thursday, April 28, 2022 at 17:45
> *To: *dev <de...@shenyu.apache.org>
> *Subject: *[Question] The data saved in zookeeper looks strange, it has
> strange number because of serialization method.
>
> Hi folks,
>
>
>
> I also created an ISSUE for this question, see
> https://github.com/apache/incubator-shenyu/issues/3360.
>
>
>
> I'm working on the zookeeper client registration for dotnet client, and I
> found the serialization for zkClient looks strange, see below.
> [image: Pasted image 20220428170506]
> <https://user-images.githubusercontent.com/11908658/165719204-df04cfc5-80a6-4bc5-b602-6a62d0f3dfab.png>
>
>
>
> And admin also use the same serializer which is default one
> SerializableSerializer in package org.I0Itec.zkclient, which makes the
> format below cannot be deserialized correctly.
> [image: image]
> <https://user-images.githubusercontent.com/11908658/165719634-c5b8d3ea-7ffd-4e03-a09a-26e9abffe6bf.png>
>
>
>
> And will cause below exception.
> [image: Pasted image 20220428170748]
> <https://user-images.githubusercontent.com/11908658/165719822-27e7992c-3132-44c5-843d-438812301432.png>
>
> org.I0Itec.zkclient also has a byte serializer, and I've tested that it
> works well and has no strange number when writing in zookeeper.
>
>
>
> I'm wondering if it's by design or a bug. Because I can't find the
> similar serializer in zookeeper official package or sdks in other language.
>
> If it's a bug, I can have a try to fix it and using the byte serializer
> instead. It will change all the places where ZkClient was used. The
> changes mainly looks like below, and also including some other small
> changes.
>
>
>
> -this.zkClient = new ZkClient(config.getServerLists(), sessionTimeout,
> connectionTimeout);
>
> +this.zkClient = new ZkClient(config.getServerLists(), sessionTimeout,
> connectionTimeout, new BytesPushThroughSerializer());
>
>
>
> If it's by design, I will try to investigate if any mitigation to solve
> this strange number issue.
>
>
>
> Thanks,
>
> Han
>
>
>
Re: [Question] The data saved in zookeeper looks strange, it has strange number because of serialization method.
Posted by Han G <dh...@hotmail.com>.
Hi Yu, sure! After these changes, I can create PR to mention it in documentation.
Since there’re too many changes, including registration in server and client, registration instance, data sync, I will use 2 PRs to finish the change.
The first PR includes registration in server and client, registration instance and the UTs. I’ve already finished it. https://github.com/apache/incubator-shenyu/pull/3388
The second PR will include data sync between admin and bootstrap.
Thanks,
Han
From: XiaoYu <xi...@apache.org>
Date: Thursday, May 5, 2022 at 18:02
To: dev@shenyu.apache.org <de...@shenyu.apache.org>
Subject: Re: [Question] The data saved in zookeeper looks strange, it has strange number because of serialization method.
Hi
we should used Curator 5.X
if user want to this fuction，We should tell users in the documentation that
they should use a higher version of zk（3.6.x）
Han G <dh...@hotmail.com> 于2022年5月2日周一 12:29写道：
> Hi community,
>
>
>
> Based on the issue mentioned in earlier mail, I’d like to fix it using
> plane byte without magic header, which will make it easy for other clients
> to register to ShenYu. But please pay attention that after this change,
> customers who are using zookeeper as register center or data sync center
> need to re-register to zookeeper.
>
>
>
> I also plan to take this change to replace current zk client
> (com.101tec.zkclient) with Curator 5.2.1. But Curator 5.2.1 is not
> compatible with zookeeper 3.4.x version and before. Please let me know if
> any concern. Thanks a lot!
>
>
>
> Related issue:
>
> https://github.com/apache/incubator-shenyu/issues/3362
>
>
>
> Thanks,
>
> Han
>
>
>
>
>
> *From: *Han G <dh...@hotmail.com>
> *Date: *Thursday, April 28, 2022 at 17:45
> *To: *dev <de...@shenyu.apache.org>
> *Subject: *[Question] The data saved in zookeeper looks strange, it has
> strange number because of serialization method.
>
> Hi folks,
>
>
>
> I also created an ISSUE for this question, see
> https://github.com/apache/incubator-shenyu/issues/3360.
>
>
>
> I'm working on the zookeeper client registration for dotnet client, and I
> found the serialization for zkClient looks strange, see below.
> [image: Pasted image 20220428170506]
> <https://user-images.githubusercontent.com/11908658/165719204-df04cfc5-80a6-4bc5-b602-6a62d0f3dfab.png>
>
>
>
> And admin also use the same serializer which is default one
> SerializableSerializer in package org.I0Itec.zkclient, which makes the
> format below cannot be deserialized correctly.
> [image: image]
> <https://user-images.githubusercontent.com/11908658/165719634-c5b8d3ea-7ffd-4e03-a09a-26e9abffe6bf.png>
>
>
>
> And will cause below exception.
> [image: Pasted image 20220428170748]
> <https://user-images.githubusercontent.com/11908658/165719822-27e7992c-3132-44c5-843d-438812301432.png>
>
> org.I0Itec.zkclient also has a byte serializer, and I've tested that it
> works well and has no strange number when writing in zookeeper.
>
>
>
> I'm wondering if it's by design or a bug. Because I can't find the
> similar serializer in zookeeper official package or sdks in other language.
>
> If it's a bug, I can have a try to fix it and using the byte serializer
> instead. It will change all the places where ZkClient was used. The
> changes mainly looks like below, and also including some other small
> changes.
>
>
>
> -this.zkClient = new ZkClient(config.getServerLists(), sessionTimeout,
> connectionTimeout);
>
> +this.zkClient = new ZkClient(config.getServerLists(), sessionTimeout,
> connectionTimeout, new BytesPushThroughSerializer());
>
>
>
> If it's by design, I will try to investigate if any mitigation to solve
> this strange number issue.
>
>
>
> Thanks,
>
> Han
>
>
>
Re: [Question] The data saved in zookeeper looks strange, it has strange number because of serialization method.
Posted by XiaoYu <xi...@apache.org>.
Hi
we should used Curator 5.X
if user want to this fuction，We should tell users in the documentation that
they should use a higher version of zk（3.6.x）
Han G <dh...@hotmail.com> 于2022年5月2日周一 12:29写道：
> Hi community,
>
>
>
> Based on the issue mentioned in earlier mail, I’d like to fix it using
> plane byte without magic header, which will make it easy for other clients
> to register to ShenYu. But please pay attention that after this change,
> customers who are using zookeeper as register center or data sync center
> need to re-register to zookeeper.
>
>
>
> I also plan to take this change to replace current zk client
> (com.101tec.zkclient) with Curator 5.2.1. But Curator 5.2.1 is not
> compatible with zookeeper 3.4.x version and before. Please let me know if
> any concern. Thanks a lot!
>
>
>
> Related issue:
>
> https://github.com/apache/incubator-shenyu/issues/3362
>
>
>
> Thanks,
>
> Han
>
>
>
>
>
> *From: *Han G <dh...@hotmail.com>
> *Date: *Thursday, April 28, 2022 at 17:45
> *To: *dev <de...@shenyu.apache.org>
> *Subject: *[Question] The data saved in zookeeper looks strange, it has
> strange number because of serialization method.
>
> Hi folks,
>
>
>
> I also created an ISSUE for this question, see
> https://github.com/apache/incubator-shenyu/issues/3360.
>
>
>
> I'm working on the zookeeper client registration for dotnet client, and I
> found the serialization for zkClient looks strange, see below.
> [image: Pasted image 20220428170506]
> <https://user-images.githubusercontent.com/11908658/165719204-df04cfc5-80a6-4bc5-b602-6a62d0f3dfab.png>
>
>
>
> And admin also use the same serializer which is default one
> SerializableSerializer in package org.I0Itec.zkclient, which makes the
> format below cannot be deserialized correctly.
> [image: image]
> <https://user-images.githubusercontent.com/11908658/165719634-c5b8d3ea-7ffd-4e03-a09a-26e9abffe6bf.png>
>
>
>
> And will cause below exception.
> [image: Pasted image 20220428170748]
> <https://user-images.githubusercontent.com/11908658/165719822-27e7992c-3132-44c5-843d-438812301432.png>
>
> org.I0Itec.zkclient also has a byte serializer, and I've tested that it
> works well and has no strange number when writing in zookeeper.
>
>
>
> I'm wondering if it's by design or a bug. Because I can't find the
> similar serializer in zookeeper official package or sdks in other language.
>
> If it's a bug, I can have a try to fix it and using the byte serializer
> instead. It will change all the places where ZkClient was used. The
> changes mainly looks like below, and also including some other small
> changes.
>
>
>
> -this.zkClient = new ZkClient(config.getServerLists(), sessionTimeout,
> connectionTimeout);
>
> +this.zkClient = new ZkClient(config.getServerLists(), sessionTimeout,
> connectionTimeout, new BytesPushThroughSerializer());
>
>
>
> If it's by design, I will try to investigate if any mitigation to solve
> this strange number issue.
>
>
>
> Thanks,
>
> Han
>
>
>