[GitHub] [pulsar-adapters] aditiwari01 commented on a change in pull request #31: [Issue #29] [pulsar-spark] Adding SparkPulsarReliableReceiver

2022-01-17 Thread GitBox
aditiwari01 commented on a change in pull request #31: URL: https://github.com/apache/pulsar-adapters/pull/31#discussion_r786475056 ## File path: pulsar-spark/src/main/scala/org/apache/spark/streaming/pulsar/SparkStreamingReliablePulsarReceiver.scala ## @@ -0,0 +1,181 @@ +/**

[GitHub] [pulsar-helm-chart] lhotari merged pull request #165: useV2WireProtocol for bookkeeper autorecovery

2022-01-17 Thread GitBox
lhotari merged pull request #165: URL: https://github.com/apache/pulsar-helm-chart/pull/165 -- 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. To unsubscribe, e-mail: dev-un

[GitHub] [pulsar-helm-chart] lhotari commented on pull request #61: anti-affinity rules refactor and default image tags

2022-01-17 Thread GitBox
lhotari commented on pull request #61: URL: https://github.com/apache/pulsar-helm-chart/pull/61#issuecomment-1015120620 @EladDolev good work! > use publishNotReadyAddresses correctly, expose it as a parameter, and remove deprecated alpha annotation this change would be valuabl

[GitHub] [pulsar-helm-chart] lhotari merged pull request #201: Add custom labels to all k8s objects in chart

2022-01-17 Thread GitBox
lhotari merged pull request #201: URL: https://github.com/apache/pulsar-helm-chart/pull/201 -- 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. To unsubscribe, e-mail: dev-un

Re: [ANNOUNCE] New PMC Member - Lari Hotari

2022-01-17 Thread Shivji Kumar Jha
Congratulations. Very well deserved. Lari is an inspiration! Regards, Shivji Kumar Jha http://www.shivjijha.com/ +91 8884075512 On Tue, 18 Jan 2022 at 08:20, Haiting Jiang wrote: > Congratulations! > > BR, > Haiting Jiang > > On 2022/01/18 02:26:32 陳智弘 wrote: > > I am so glad to hear this news

Re: [ANNOUNCE] New PMC Member - Lari Hotari

2022-01-17 Thread Haiting Jiang
Congratulations! BR, Haiting Jiang On 2022/01/18 02:26:32 陳智弘 wrote: > I am so glad to hear this news , lots of critical issues were solved by him > and really help us a lot > > Thomas > > Aloys Zhang 於 2022年1月18日 週二 上午10:25寫道: > > > Congratulations! > > > > Devin Bost 于2022年1月18日周二 07:07写道:

Re: [VOTE] PIP 131: Resolve produce chunk messages failed when topic level maxMessageSize is set

2022-01-17 Thread Lin Lin
+1 On 2021/12/29 02:29:21 Haiting Jiang wrote: > This is the voting thread for PIP-131. It will stay open for at least 48h. > > https://github.com/apache/pulsar/issues/13544 > > The discussion thread is > https://lists.apache.org/thread/c63d9s73j9x1m3dkqr3r38gyp8s7cwzf > > ## Motivation > > C

Re: [VOTE] PIP-132: Include message header size when check maxMessageSize of non-batch message on the client side.

2022-01-17 Thread Lin Lin
+1 On 2022/01/12 01:57:59 Haiting Jiang wrote: > This is the voting thread for PIP-132. It will stay open for at least 48 > hours. > > https://github.com/apache/pulsar/issues/13591 > > Pasted below for quoting convenience. > > > > ## Motivation > > Currently, Pulsar client (Java) only c

Re: [ANNOUNCE] New PMC Member - Lari Hotari

2022-01-17 Thread 陳智弘
I am so glad to hear this news , lots of critical issues were solved by him and really help us a lot Thomas Aloys Zhang 於 2022年1月18日 週二 上午10:25寫道: > Congratulations! > > Devin Bost 于2022年1月18日周二 07:07写道: > >> Congrats!! Well done, Lari. Definitely well deserved. >> >> Devin Bost >> (Sent from

Re: [ANNOUNCE] New PMC Member - Lari Hotari

2022-01-17 Thread Aloys Zhang
Congratulations! Devin Bost 于2022年1月18日周二 07:07写道: > Congrats!! Well done, Lari. Definitely well deserved. > > Devin Bost > (Sent from mobile) > > > From: Enrico Olivelli > Sent: Monday, January 17, 2022 3:07:25 PM > To: pulsar-users > Cc: dev > Subject: Re: [

Re: [DISCUSSION] PIP-133 Pulsar Functions Add API For Accessing Other Function States

2022-01-17 Thread Ethan Merrill
Thanks for the feedback. I see your concerns. I've been thinking about some ways to mitigate these concerns without expanding the scope of this too much. First, I think it could be a good idea to limit state access to just functions within the same namespace. This will at least avoid any issues

Re: [ANNOUNCE] New PMC Member - Lari Hotari

2022-01-17 Thread Devin Bost
Congrats!! Well done, Lari. Definitely well deserved. Devin Bost (Sent from mobile) From: Enrico Olivelli Sent: Monday, January 17, 2022 3:07:25 PM To: pulsar-users Cc: dev Subject: Re: [ANNOUNCE] New PMC Member - Lari Hotari Congratulations! Enrico Il Lun

Re: [ANNOUNCE] New PMC Member - Lari Hotari

2022-01-17 Thread frank
Congrats Lari - very well deservedFrank On Mon, 17 Jan 2022 15:50:37 -0500 w...@apache.org wrote Hi - The Apache Pulsar Project Management Committee (PMC) has invited Lari Hotari (https://github.com/lhotari) as a member of the PMC and we are pleased to

Re: [ANNOUNCE] New PMC Member - Lari Hotari

2022-01-17 Thread Enrico Olivelli
Congratulations! Enrico Il Lun 17 Gen 2022, 22:03 frank ha scritto: > Congrats Lari - very well deserved > > Frank > > > > > > > On Mon, 17 Jan 2022 15:50:37 -0500 w...@apache.org > wrote > > Hi - > > The Apache Pulsar Project Management Committee (PMC) has invited Lari > Hotari > (

[ANNOUNCE] New PMC Member - Lari Hotari

2022-01-17 Thread Dave Fisher
Hi - The Apache Pulsar Project Management Committee (PMC) has invited Lari Hotari (https://github.com/lhotari) as a member of the PMC and we are pleased to announce that he has accepted. He is very active in the community on GitHub and on the pulsar slack channel. Welcome Lari to the Apache Pul

[GitHub] [pulsar-client-node] romainbrancourt commented on issue #189: Fail to build `pulsar-client` on M1

2022-01-17 Thread GitBox
romainbrancourt commented on issue #189: URL: https://github.com/apache/pulsar-client-node/issues/189#issuecomment-1014732347 @csauvage Run this before running npm install, this may fix your problem :D ``` export CPLUS_INCLUDE_PATH="$CPLUS_INCLUDE_PATH:$(brew --prefix)/include"

[GitHub] [pulsar-helm-chart] lhotari commented on pull request #204: Upgrade cert-manager to 1.5.4

2022-01-17 Thread GitBox
lhotari commented on pull request #204: URL: https://github.com/apache/pulsar-helm-chart/pull/204#issuecomment-1014618466 upgrades up to 1.5.x are backwards compatible with cert-manager < 1.0. After 1.6.x, there's a breaking change explained in https://cert-manager.io/docs/installation/up

[GitHub] [pulsar-helm-chart] lhotari opened a new pull request #204: Upgrade cert-manager to 1.5.4

2022-01-17 Thread GitBox
lhotari opened a new pull request #204: URL: https://github.com/apache/pulsar-helm-chart/pull/204 ### Motivation - Keep cert-manager version updated ### Modifications - upgrade cert-manager to 1.5.4 version - use apiVersion `cert-manager.io/v1` -- This is an automa

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

2022-01-17 Thread Hang Chen
Thanks for creating this proposal Zike Yang. I have two ideas about it. 1. Instead of modifying the current protocol, we can use producer metadata to carry the init subscription 2. Please add auth for subscription creation when create topic by producer, otherwise, it will be easily attacked. Thank

Re: [VOTE] PIP-135: Include MetadataStore backend for Etcd

2022-01-17 Thread Lan Liang
+1, Thanks for your work! - lan.liang On 01/17/2022 20:50,ZhangJian He wrote: +1 PengHui Li 于2022年1月17日周一 14:01写道: +1 (binding) Regards, Penghui On Sat, Jan 15, 2022 at 9:24 PM Joe F wrote: +1 (binding) On Sat, Jan 15, 2022 at 4:46 AM Enrico Olivelli wrote: Il Sab 15 Gen 2022, 09:10 t

Re: [VOTE] PIP-135: Include MetadataStore backend for Etcd

2022-01-17 Thread ZhangJian He
+1 PengHui Li 于2022年1月17日周一 14:01写道: > +1 (binding) > > Regards, > Penghui > > On Sat, Jan 15, 2022 at 9:24 PM Joe F wrote: > > > +1 (binding) > > > > On Sat, Jan 15, 2022 at 4:46 AM Enrico Olivelli > > wrote: > > > > > Il Sab 15 Gen 2022, 09:10 tamer Abdlatif ha > > > scritto: > > > > > > >

[GitHub] [pulsar-client-node] tuteng merged pull request #190: Feature support oauth2 for node client

2022-01-17 Thread GitBox
tuteng merged pull request #190: URL: https://github.com/apache/pulsar-client-node/pull/190 -- 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. To unsubscribe, e-mail: dev-un

[GitHub] [pulsar-client-node] tuteng commented on a change in pull request #190: Feature support oauth2 for node client

2022-01-17 Thread GitBox
tuteng commented on a change in pull request #190: URL: https://github.com/apache/pulsar-client-node/pull/190#discussion_r785740743 ## File path: package-lock.json ## @@ -1,9204 +1,8 @@ { "name": "pulsar-client", "version": "1.6.0-rc.0", - "lockfileVersion": 2, Review

[GitHub] [pulsar-helm-chart] lhotari commented on pull request #202: Increase default initialDelaySeconds for Zookeeper probes to workaround ZOOKEEPER-3988

2022-01-17 Thread GitBox
lhotari commented on pull request #202: URL: https://github.com/apache/pulsar-helm-chart/pull/202#issuecomment-1014259746 There's also #203 to speed up startup by using `podManagementPolicy=Parallel` for Zookeeper. -- This is an automated message from the Apache Git Service. To respond

[GitHub] [pulsar-helm-chart] lhotari opened a new pull request #203: Change default podManagementPolicy to Parallel for Zookeeper

2022-01-17 Thread GitBox
lhotari opened a new pull request #203: URL: https://github.com/apache/pulsar-helm-chart/pull/203 ### Motivation - For speeding up startup, podManagementPolicy should be set to `Parallel` for Zookeeper ### Modifications - set `zookeeper.podManagementPolicy` to `Parallel

[GitHub] [pulsar-helm-chart] lhotari commented on pull request #190: Bump to Pulsar 2.8.2

2022-01-17 Thread GitBox
lhotari commented on pull request #190: URL: https://github.com/apache/pulsar-helm-chart/pull/190#issuecomment-1014250323 I have created #202 as a workaround for the Zookeeper issue (when TLS is enabled). -- This is an automated message from the Apache Git Service. To respond to the mes

[GitHub] [pulsar-helm-chart] lhotari opened a new pull request #202: Increase default initialDelaySeconds for Zookeeper probes to workaround ZOOKEEPER-3988

2022-01-17 Thread GitBox
lhotari opened a new pull request #202: URL: https://github.com/apache/pulsar-helm-chart/pull/202 ### Motivation - When TLS is enabled for Zookeeper, NettyServerCnxnFactory will be used. It contains the issue https://github.com/apache/pulsar/issues/11070 / https://issues.apa