Thanks Penghui and PengCheng, I will drop this candidate and raise a new one after #21921 <https://github.com/apache/pulsar/pull/21921> get merged.
Regards Jiwei Guo (Tboy) On Thu, Jan 18, 2024 at 9:45 PM Pengcheng Jiang <pengcheng.ji...@streamnative.io.invalid> wrote: > > 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) > > > > > > > > > >
