> 1. The querystate exit with `Reason: key 'hello' doesn't exist`, which the
old version will not exit

there is a bug for it, a 500 error is returned instead of a 404 error,
there is the fix: https://github.com/apache/pulsar/pull/21921

> The output missed the `version` field

In PIP-312 <https://github.com/apache/pulsar/pull/21597>, I used the
`StateStore` instead of bookkeeper's `StorageAdminClient` to manage state
keys in `ComponentImpl`, this is to make functions support
`StateStoreImpls` other than the bookkeeper
but for now the `StateStore.get` method doesn't return `version` yet, so
there is no `version` field in the output, we may expose the `version`
field to the `StateStore.get` in the future

Regards,
Jiang Pengcheng

PengHui Li <peng...@apache.org> 于2024年1月18日周四 11:29写道:

> It looks like something broke the behavior of the querystate from Pulsar
> Functions.
>
> 1. The querystate exit with `Reason: key 'hello' doesn't exist`, which the
> old version will not exit
>
> ```
> lipenghui@lipenghuis-MacBook-Pro apache-pulsar-3.2.0 % bin/pulsar-admin
> functions querystate --tenant test --namespace test-namespace --name
> word_count -k hello -w
> key 'hello' doesn't exist.
>
> Reason: key 'hello' doesn't exist.
> ```
>
> 2. The output missed the `version` field
>
> ```
>   "key": "hello",
>   "numberValue": 20,
>   "version": 19
> ```
>
> ```
>   "key": "hello",
>   "stringValue": "\u0000\u0000\u0000\u0000\u0000\u0000\u0000\n",
>   "numberValue": 10
> ```
>
> Regards,
> Penghui
>
> On Tue, Jan 16, 2024 at 6:34 PM Zixuan Liu <node...@gmail.com> wrote:
>
> > +1 (non-binding)
> >
> > - Checked the checksums and signatures
> > - Built with Temurin-17.0.6+10
> > - Run standalone
> > - Checked producer and consumer
> >
> > Thanks,
> > Zixuan
> >
> > guo jiwei <techno...@apache.org> 于2024年1月16日周二 15:57写道:
> >
> > > This is the second release candidate for Apache Pulsar version 3.2.0.
> > >
> > > It fixes the following issues:
> > > https://github.com/apache/pulsar/milestone/36?closed=1
> > >
> > > *** Please download, test and verify on this release. This release
> > > candidate verification will stay open until Jan 15 ***
> > >
> > > Note that we are verifying upon the source (tag), binaries are provided
> > for
> > > convenience.
> > >
> > > Source and binary files:
> > >
> https://dist.apache.org/repos/dist/dev/pulsar/pulsar-3.2.0-candidate-2/
> > >
> > > SHA-512 checksums:
> > >
> > >
> > >
> >
> 57af4de0531baada79ff3eb5e51659ae3d2b6732840a27861636d3538d8cca2b3aeb77d3f5cecf37dc5d8a0de11a05d0775ae2d2ca9398f9747fbe662e1cdeae
> > >
> > > apache-pulsar-3.2.0-bin.tar.gz
> > >
> > >
> > >
> >
> 7ddd8df261f4ffeed2e4f029aa475f523bb9505869870db681a57b7df25af01d5670ba00bc7e062ddae0b962eaed9ee828a4cd7f4744a965afd77e9ae1239d6a
> > >
> > > apache-pulsar-3.2.0-src.tar.gz
> > >
> > > Maven staging repo:
> > >
> https://repository.apache.org/content/repositories/orgapachepulsar-1262/
> > >
> > > The tag to verify:
> > > v3.2.0-candidate-2 (5348e1b9124052a454f66769ae3e9f54ee0a75d4)
> > > https://github.com/apache/pulsar/commits/v3.2.0-candidate-2/
> > >
> > > Pulsar's KEYS file containing PGP keys you use to sign the release:
> > > https://dist.apache.org/repos/dist/dev/pulsar/KEYS
> > >
> > > Docker images:
> > >
> > > pulsar images:
> > >
> > >
> >
> https://hub.docker.com/layers/technoboy8/pulsar/3.2.0-5348e1b/images/sha256-cf02a08e588ca493d10dbe472b72f5e64e4f90b8b5ae01e91321f336776a5a4e?context=repo
> > > <
> > >
> >
> https://hub.docker.com/layers/mattison/pulsar/3.1.0-candidate-1/images/sha256-0efbaad7d893cc5041a46a2d4d56432bda855ae4068a38349777d1be6e98d27d?context=explore
> > > >
> > > pulsar-all images:
> > >
> > >
> >
> https://hub.docker.com/layers/technoboy8/pulsar-all/3.2.0-5348e1b/images/sha256-801e395fc26257e2beb5ea0574fe27f5a4ff956c85cbcd6b362432d6dda28b95?context=repo
> > >
> > > Please download the source package, and follow the README to build
> > > and run the Pulsar standalone service.
> > >
> > > Note that this RC doesn't require a formal vote, but we would also
> > > appreciate your feedback with +1/-1. And please provide specific
> > > comments if your feedback is not +1.
> > >
> > >
> > > Regards
> > > Jiwei Guo (Tboy)
> > >
> >
>

Reply via email to