Re: [DISCUSSION] PIP-132: Support Pulsar system event.

2022-01-07 Thread PengHui Li
We have a pluggable broker interceptor, > I think we need to trigger some event as bellow: - [ ] Client connected - [ ] Client disconnected - [ ] Consumer subscribe - [ ] Consumer unsubscribe - [ ] Producer publish - [ ] Message delivered - [ ] Message acked For these events, the broker intercept

Re: Lifting Kubernetes minimum version requirement for Apache Pulsar Helm Charts from k8s 1.14 to 1.18

2022-01-07 Thread Lari Hotari
Yes, the change in the PR [1] is mainly about CI. The documentation will also be updated that the minimum required version is 1.18 . BR, Lari [1] - https://github.com/apache/pulsar-helm-chart/pull/192 pe 7. tammik. 2022 klo 2.43 Sijie Guo wrote: > Can you clarify what is the main change? I s

[GitHub] [pulsar-client-node] Hoskins355 commented on issue #188: When installing the Pulsar Node.js client, the operation steps need to be optimized

2022-01-07 Thread GitBox
Hoskins355 commented on issue #188: URL: https://github.com/apache/pulsar-client-node/issues/188#issuecomment-1007542953 I got Libpulsar.so in usr/lib https://user-images.githubusercontent.com/3084440/148573912-1e9bfce4-68e4-4d7a-9f88-edccd5d7e527.png";> and I copied that directory in

Re: Voting time period minimum

2022-01-07 Thread Dave Fisher
> On Jan 6, 2022, at 4:51 PM, Sijie Guo wrote: > > I believe 48 hours vote is only for PIP, which was agreed in the dev@. I would like to understand why Matteo chose 48 hours for this process. What’s the hurry? In this discussion there is a mixture of discussion and +1. [1] Did the project

[GitHub] [pulsar-client-node] Hoskins355 removed a comment on issue #188: When installing the Pulsar Node.js client, the operation steps need to be optimized

2022-01-07 Thread GitBox
Hoskins355 removed a comment on issue #188: URL: https://github.com/apache/pulsar-client-node/issues/188#issuecomment-1007542953 I got Libpulsar.so in usr/lib https://user-images.githubusercontent.com/3084440/148573912-1e9bfce4-68e4-4d7a-9f88-edccd5d7e527.png";> and I copied that dire

[GitHub] [pulsar-client-node] Hoskins355 commented on issue #188: When installing the Pulsar Node.js client, the operation steps need to be optimized

2022-01-07 Thread GitBox
Hoskins355 commented on issue #188: URL: https://github.com/apache/pulsar-client-node/issues/188#issuecomment-1007669919 @ zhaoyajun2009, Thanks I got it working. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [DISCUSSION] PIP-122: Change loadBalancer default loadSheddingStrategy to ThresholdShedder

2022-01-07 Thread Frank Kelly
This is very cool. How is `ThresholdShedder` chosen - I don't see if referenced here? https://pulsar.apache.org/docs/en/reference-configuration/ -F On Thu, Jan 6, 2022 at 7:45 PM Sijie Guo wrote: > +1 ThresholdShedder is a much better policy > > On Tue, Jan 4, 2022 at 6:24 AM Hang Chen wrote:

[GitHub] [pulsar-helm-chart] csthomas1 opened a new issue #197: -Dlog4j2.formatMsgNoLookups=true not set for pulsar manager

2022-01-07 Thread GitBox
csthomas1 opened a new issue #197: URL: https://github.com/apache/pulsar-helm-chart/issues/197 **Describe the bug** The Pulsar Manager java process is being launched without the inclusion of the "-Dlog4j2.formatMsgNoLookups=true" switch, despite the fact that it includes an older versio

Re: Voting time period minimum

2022-01-07 Thread Matteo Merli
On Fri, Jan 7, 2022 at 9:27 AM Dave Fisher wrote: > > I believe 48 hours vote is only for PIP, which was agreed in the dev@. > > I would like to understand why Matteo chose 48 hours for this process. What’s > the hurry? Since we went from no formal process to "some" process, and since any change

Re: Voting time period minimum

2022-01-07 Thread Dave Fisher
> On Jan 7, 2022, at 2:59 PM, Matteo Merli wrote: > > On Fri, Jan 7, 2022 at 9:27 AM Dave Fisher wrote: >>> I believe 48 hours vote is only for PIP, which was agreed in the dev@. >> >> I would like to understand why Matteo chose 48 hours for this process. >> What’s the hurry? > > Since we

Re: [DISCUSSION] PIP-122: Change loadBalancer default loadSheddingStrategy to ThresholdShedder

2022-01-07 Thread Michael Marshall
Hi Frank, You can configure the Load Shedding Strategy in the `broker.conf` file using `loadBalancerLoadSheddingStrategy`. The configuration is missing from the document you referenced--it should be there, though. There was a recent PR to add documentation describing the different configuration

[GitHub] [pulsar-helm-chart] csthomas1 opened a new pull request #198: Added -Dlog4j2.formatMsgNoLookups=true to PULSAR_MANAGER_OPTS

2022-01-07 Thread GitBox
csthomas1 opened a new pull request #198: URL: https://github.com/apache/pulsar-helm-chart/pull/198 Fixes #197 ### Motivation Adds the -Dlog4j2.formatMsgNoLookups=true switch to the pulsar manager commandline. Necessary for pulsar manager v0.1.0, which includes a dependenc

[GitHub] [pulsar-manager] nicknezis opened a new pull request #438: Updated to fix Log4J security vulnerabilities

2022-01-07 Thread GitBox
nicknezis opened a new pull request #438: URL: https://github.com/apache/pulsar-manager/pull/438 ### Motivation Details at https://logging.apache.org/log4j/2.x/security.html Description of CVE-2021-45105: > Apache Log4j2 versions 2.0-alpha1 through 2.16.0 did not protect

Re: [DISCUSSION] PIP-124: Create init subscription before sending message to DLQ

2022-01-07 Thread Michael Marshall
Thanks for your response, Penghui. I support simplifying message loss prevention for DLQ topics. However, it's not clear to me why we should only simplify it for DLQ topics. As a Pulsar user, I encountered many of the challenges you mention when producing to auto created topics. In my architectur

Re: [PR] Pulsar non root docker image

2022-01-07 Thread Michael Marshall
> Is Functions being verified? I discussed this a bit in PR 13376's description and comments, please let me know if you have additional questions. I haven't done any extra verification. Note that since [0] is in both 2.8.x and 2.9.x, the upgrade scenario that led us to revert the first non-root w

Re: Lifting Kubernetes minimum version requirement for Apache Pulsar Helm Charts from k8s 1.14 to 1.18

2022-01-07 Thread Michael Marshall
I noticed today that we document some Kubernetes requirements on the website [0]. We'll need to coordinate appropriate updates to our docs, too. - Michael [0] - https://pulsar.apache.org/docs/en/kubernetes-helm/#prerequisite On Fri, Jan 7, 2022 at 9:30 AM Lari Hotari wrote: > > Yes, the change