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

2022-01-25 Thread PengHui Li
+1 (binding) Regards, Penghui On Wed, Jan 26, 2022 at 12:17 PM Michael Marshall wrote: > +1 (non binding) - this proposal looks great! Thank you for a good > discussion of this feature! > > Thanks, > Michael > > On Tue, Jan 25, 2022 at 8:20 PM Hang Chen wrote: > > > > +1 (binding) > > > > Than

[GitHub] [pulsar-helm-chart] lhotari closed issue #85: Change Pulsar Manager docker image version

2022-01-25 Thread GitBox
lhotari closed issue #85: URL: https://github.com/apache/pulsar-helm-chart/issues/85 -- 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-unsubscr.

[GitHub] [pulsar-helm-chart] lhotari commented on issue #85: Change Pulsar Manager docker image version

2022-01-25 Thread GitBox
lhotari commented on issue #85: URL: https://github.com/apache/pulsar-helm-chart/issues/85#issuecomment-1021917774 The docker image version was upgraded as part of #206. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [pulsar-helm-chart] michaeljmarshall commented on pull request #205: Tiered Storage config

2022-01-25 Thread GitBox
michaeljmarshall commented on pull request #205: URL: https://github.com/apache/pulsar-helm-chart/pull/205#issuecomment-1021885592 @sijie - PTAL -- 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

[GitHub] [pulsar-helm-chart] michaeljmarshall merged pull request #210: Remove references to tag: 2.6.0 in examples

2022-01-25 Thread GitBox
michaeljmarshall merged pull request #210: URL: https://github.com/apache/pulsar-helm-chart/pull/210 -- 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-mai

Re: [DISCUSS] [Transaction] Clear namespace backlog can't clear system topic and system sub backlog.

2022-01-25 Thread Michael Marshall
> Currently, we don't have some system topics that users can consume, I think > it's a good chance to normalise the system topic naming rule. > but "__" is more common, we need more complex naming rules to reduce > restrictions on users. Maybe we can use some prefix like "__pulsar_sys_" > etc. If

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

2022-01-25 Thread Michael Marshall
+1 (non binding) - this proposal looks great! Thank you for a good discussion of this feature! Thanks, Michael On Tue, Jan 25, 2022 at 8:20 PM Hang Chen wrote: > > +1 (binding) > > Thanks, > Hang > > Jia Zhai 于2022年1月26日周三 10:17写道: > > > > +1 > > > > > > > > On Tue, Jan 25, 2022 at 8:28 PM matt

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

2022-01-25 Thread Hang Chen
+1 (binding) Thanks, Hang Jia Zhai 于2022年1月26日周三 10:17写道: > > +1 > > > > On Tue, Jan 25, 2022 at 8:28 PM mattison chao > wrote: > > > +1 (non-binding) > > > > > On Jan 25, 2022, at 5:34 PM, Zike Yang > > wrote: > > > > > > Hi Pulsar Community, > > > > > > This is the voting thread for PIP-124.

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

2022-01-25 Thread Jia Zhai
+1 On Tue, Jan 25, 2022 at 8:28 PM mattison chao wrote: > +1 (non-binding) > > > On Jan 25, 2022, at 5:34 PM, Zike Yang > wrote: > > > > Hi Pulsar Community, > > > > This is the voting thread for PIP-124. It will stay open for at least > > 48 hours. Pasted below for quoting convenience. > > >

Re: Architecture of function authorization for process mode

2022-01-25 Thread Niclas Hedhman
On 2022-01-25 08:57, Matteo Merli wrote: The only recommended way to run a multi-tenant Pulsar functions clusters is to run it with Kubernetes runtime. In thread or process runtime, there is no reliable way to restrict the access to the credentials of each function instance (since it needs to be

Re: Architecture of function authorization for process mode

2022-01-25 Thread Jerry Peng
Devin, You can customize how each function authenticates with brokers by creating a implementation of this interface: https://github.com/apache/pulsar/blob/master/pulsar-functions/runtime/src/main/java/org/apache/pulsar/functions/auth/FunctionAuthProvider.java and setting the class name of your

[GitHub] [pulsar-manager] michaeljmarshall commented on issue #441: Support HerdDB in the standard docker image.

2022-01-25 Thread GitBox
michaeljmarshall commented on issue #441: URL: https://github.com/apache/pulsar-manager/issues/441#issuecomment-1020387920 @mattisonchao - yes, that makes sense to me for docker-compose. Ideally, we will also make it configurable in the pulsar helm chart. -- This is an automated message

[GitHub] [pulsar-client-node] Matt-Esch commented on issue #99: Authenticating using a Token

2022-01-25 Thread GitBox
Matt-Esch commented on issue #99: URL: https://github.com/apache/pulsar-client-node/issues/99#issuecomment-1020882307 Segfault issue tracked here: https://github.com/apache/pulsar-client-node/issues/191 -- This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] [pulsar-helm-chart] lhotari merged pull request #206: Upgrade default images for Grafana & Pulsar Manager

2022-01-25 Thread GitBox
lhotari merged pull request #206: URL: https://github.com/apache/pulsar-helm-chart/pull/206 -- 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] ericsyh commented on pull request #209: fix(charts): missing eof on bk sts

2022-01-25 Thread GitBox
ericsyh commented on pull request #209: URL: https://github.com/apache/pulsar-helm-chart/pull/209#issuecomment-1020897852 > @ericsyh This problem is already fixed by #208 . I have merged it now. I'll close this PR. OK, sure. -- This is an automated message from the Apache Git Ser

[GitHub] [pulsar-helm-chart] lhotari merged pull request #208: Fix ci error caused by wrong block of if clause

2022-01-25 Thread GitBox
lhotari merged pull request #208: URL: https://github.com/apache/pulsar-helm-chart/pull/208 -- 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 #209: fix(charts): missing eof on bk sts

2022-01-25 Thread GitBox
lhotari commented on pull request #209: URL: https://github.com/apache/pulsar-helm-chart/pull/209#issuecomment-1020832918 @ericsyh This problem is already fixed by #208 . I have merged it now. I'll close this PR. -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [pulsar-helm-chart] lhotari closed pull request #209: fix(charts): missing eof on bk sts

2022-01-25 Thread GitBox
lhotari closed pull request #209: URL: https://github.com/apache/pulsar-helm-chart/pull/209 -- 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

[VOTE] Pulsar Release 2.9.2 Candidate 2

2022-01-25 Thread Ran Gao
Sorry, the 2.9.2 release candidate-1 has a wrong sign certificate. This is the second release candidate for Apache Pulsar, version 2.9.2. *** Please download, test, and vote on this release. This vote will stay open for at least 72 hours *** Note that we are voting upon the source (tag), binarie

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

2022-01-25 Thread mattison chao
+1 (non-binding) > On Jan 25, 2022, at 5:34 PM, Zike Yang wrote: > > Hi Pulsar Community, > > This is the voting thread for PIP-124. It will stay open for at least > 48 hours. Pasted below for quoting convenience. > > Here is the issue for this PIP: https://github.com/apache/pulsar/issues/1340

Re: Note for pulsar-client checkstyle PR

2022-01-25 Thread Enrico Olivelli
Yunze, Il giorno mar 25 gen 2022 alle ore 13:14 Yunze Xu ha scritto: > > Hi all (especially committers), > > I opened a PR to enable checkstyle plugin for pulsar-client module just now. > See https://github.com/apache/pulsar/pull/13940 > . Great wor

Note for pulsar-client checkstyle PR

2022-01-25 Thread Yunze Xu
Hi all (especially committers), I opened a PR to enable checkstyle plugin for pulsar-client module just now. See https://github.com/apache/pulsar/pull/13940 . It’s really a huge PR but I hope it could be included in Pulsar 2.10 release because pulsar-

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

2022-01-25 Thread Zike Yang
Hi Pulsar Community, This is the voting thread for PIP-124. It will stay open for at least 48 hours. Pasted below for quoting convenience. Here is the issue for this PIP: https://github.com/apache/pulsar/issues/13408 ## Motivation If we enable the DLQ when consuming messages. For some mes

[GitHub] [pulsar-helm-chart] lhotari merged pull request #206: Upgrade default images for Grafana & Pulsar Manager

2022-01-25 Thread GitBox
lhotari merged pull request #206: URL: https://github.com/apache/pulsar-helm-chart/pull/206 -- 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 opened a new pull request #211: Add -XX:+ExitOnOutOfMemoryError to Zookeeper's PULSAR_GC parameters

2022-01-25 Thread GitBox
lhotari opened a new pull request #211: URL: https://github.com/apache/pulsar-helm-chart/pull/211 ### Motivation - `-XX:+ExitOnOutOfMemoryError` is used for other components - it's better to exit the process when an OOME happens since an OOME can leave Zookeeper in a bad state

[GitHub] [pulsar-helm-chart] lhotari opened a new pull request #210: Remove references to tag: 2.6.0 in examples

2022-01-25 Thread GitBox
lhotari opened a new pull request #210: URL: https://github.com/apache/pulsar-helm-chart/pull/210 ### Motivation It's better to not maintain out-dated examples referencing the 2.6.0 tag version. ### Modifications - remove out-dated examples -- This is an automated me