Re: [Java][Client] Important behaviour change on empty key from 2.7.x to 2.8.x

2021-05-18 Thread Enrico Olivelli
Il Mar 18 Mag 2021, 18:15 Matteo Merli ha scritto: > I believe this is a by-product of the protobuf changes. Google > Protobuf returns an empty string if you're trying to access an > optional field with no default value associated with it. This can lead > to subtle bugs.. > > Users of `Message` A

[GitHub] [pulsar-client-node] k2la closed pull request #154: [Do Not Merge]This is a test

2021-05-18 Thread GitBox
k2la closed pull request #154: URL: https://github.com/apache/pulsar-client-node/pull/154 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service,

[GitHub] [pulsar-client-node] hrsakai opened a new pull request #156: [Do Not Merge] This is a test

2021-05-18 Thread GitBox
hrsakai opened a new pull request #156: URL: https://github.com/apache/pulsar-client-node/pull/156 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this

Re: Major performance issue uncovered

2021-05-18 Thread Devin Bost
Lari, Thanks for sharing your thoughts. I'll work on making this easier to reproduce. That's a good suggestion. > How did you detect that nothing else is executed? Based on log output? I suppose it may be misleading for me to say that "nothing" else is executed since I'm judging based on debug l

[GitHub] [pulsar-helm-chart] TC-robV opened a new pull request #121: add enableAdminApi for prometheus

2021-05-18 Thread GitBox
TC-robV opened a new pull request #121: URL: https://github.com/apache/pulsar-helm-chart/pull/121 Fixes # ### Motivation would be nice to have this option here so people can run admin commands against the prometheus. ### Modifications added a new value and modif

Re: Major performance issue uncovered

2021-05-18 Thread Lari Hotari
Thanks Devin, very thoughtful observations. Do you think that it would be possible to create a repro of the case with some "dummy" code in the functions? Perhaps messages could be produced to the input topic partitions with pulsar-perf? There's a lot of interesting and useful details in your obse

[GitHub] [pulsar-client-node] Romstar opened a new pull request #155: added functionality for orderingKey in Message.cc

2021-05-18 Thread GitBox
Romstar opened a new pull request #155: URL: https://github.com/apache/pulsar-client-node/pull/155 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this

Re: [Java][Client] Important behaviour change on empty key from 2.7.x to 2.8.x

2021-05-18 Thread Matteo Merli
I believe this is a by-product of the protobuf changes. Google Protobuf returns an empty string if you're trying to access an optional field with no default value associated with it. This can lead to subtle bugs.. Users of `Message` API should always check `hasKey()` before trying to do `getKey()`

[Pulsar Community Weekly Update] 2021-05-10 ~ 2021-05-16

2021-05-18 Thread Huanli Meng
Dear Pulsar enthusiast, This is the Pulsar community weekly update for 2021-05-10 ~ 2021-05-16, with updates on Pulsar client, broker, and so on. This Pulsar community weekly update is also available at https://streamnative.io/weekly/2021/2021-05/2021-05-17-pulsar-weekly. *All Pulsar community w

[GitHub] [pulsar-manager] andikercuku commented on issue #297: Question:how to change the default login account and password

2021-05-18 Thread GitBox
andikercuku commented on issue #297: URL: https://github.com/apache/pulsar-manager/issues/297#issuecomment-843249637 For this issue you can deploy a pulsar-manager-batch-job.yaml file which will fix or change the login access in backend-service in this [link](https://github.com/andikercuku

Re: Pulsar WebSite Builder is failing

2021-05-18 Thread Enrico Olivelli
Hello, we are still stuck. the website builder does not work. This time is because the build image picks up Python 3.5.2 and so the job is not able to build the Python client that apparently cannot be compiled with that version. I tried to tweak the build image in order to force Python 3.9 but wi

[GitHub] [pulsar-helm-chart] klaus-pd opened a new issue #120: pulsar deploy in k8s zookeeper error

2021-05-18 Thread GitBox
klaus-pd opened a new issue #120: URL: https://github.com/apache/pulsar-helm-chart/issues/120 **Describe the bug** The Pulsar cluster was deployed on K8S using the officially provided Helm cluster, but the ZooKeeper cluster deployment could not complete the initialization of the cluster

[GitHub] [pulsar-client-node] k2la opened a new pull request #154: [Do Not Merge]This is a test

2021-05-18 Thread GitBox
k2la opened a new pull request #154: URL: https://github.com/apache/pulsar-client-node/pull/154 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this ser

[Java][Client] Important behaviour change on empty key from 2.7.x to 2.8.x

2021-05-18 Thread Enrico Olivelli
Hello, I have found this behaviour in the Java Client while switching from 2.7 to 2.8. Short version of the story: - on 2.7.2 a null key is received as an empty key - on 2.8.0 a null key is received as a null key The behaviour of 2.8 is better, because it is what you expect. But if you migrate a

[GitHub] [pulsar-client-node] hrsakai commented on issue #135: send delay message, but deliver immediately

2021-05-18 Thread GitBox
hrsakai commented on issue #135: URL: https://github.com/apache/pulsar-client-node/issues/135#issuecomment-842938122 @ilaipi We released v1.3.0, so closed this issue. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [pulsar-client-node] hrsakai closed issue #135: send delay message, but deliver immediately

2021-05-18 Thread GitBox
hrsakai closed issue #135: URL: https://github.com/apache/pulsar-client-node/issues/135 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, pl