Hi Jiaqi, Thanks for raising up this discussion. This is the wrong behavior for the go client, and it will make it confusing for the go client user. +1 for fixing this wrong implementation even though it will introduce breaking changes.
Thanks, Zike Yang On Wed, Dec 21, 2022 at 5:05 PM Yunze Xu <y...@streamnative.io.invalid> wrote: > > +1 to me. It's a breaking change, but I think it's more like a wrong > implementation in Go client. > > Thanks, > Yunze > > On Wed, Dec 21, 2022 at 5:00 PM Jiaqi Shen <gleiphir2...@gmail.com> wrote: > > > > Hi all. > > > > I noticed that the behavior of go client Seek is different with Java > > client, which may confuse the users. > > > > More details here: https://github.com/apache/pulsar-client-go/issues/919. > > > > Overall, if go client executes `Seek(id)`, it will reset its position in > > `id` instead of `id+1` by default. It's different with Java Client. > > > > I think we should modify it to make sure that Go Client `Seek()` has the > > same behavior with Java Client. Even though it will bring the breaking > > change. > > > > Please feel free to leave your comments. > > > > Thanks, > > Jiaqi Shen
