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

2022-01-23 Thread GitBox
dragonls commented on pull request #208: URL: https://github.com/apache/pulsar-helm-chart/pull/208#issuecomment-1019435385 @michaeljmarshall PTAL, thanks a lot. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

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

2022-01-23 Thread mattison chao
HI, Michael Marshall > I am concerned that this design will not scale and that it will be very expensive on installations with many topics. I very much agree with your concern, but currently we already have some topic names to use ``endWith()`` and ``startWith()`` to determine. > It would also i

[VOTE] Pulsar Release 2.9.2 Candidate 1

2022-01-23 Thread Ran Gao
This is the first release candidate for Apache Pulsar, version 2.9.2. It fixes the many issues and the log4j security problem. *** 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), binaries are provi

Re: [DISCUSS] Apache Pulsar 2.9.2 release

2022-01-23 Thread Ran Gao
Thanks for your reminder. Currently, the new added PR will be added the label `branch-x.x`, the label indicate the PR should be added to which branch. After the PR is merged, the PR will be cherry-picked to the specific branch. After cherry-pick, a label `cherry-picked/branch-x.x` will be added

Re: [DISCUSS] Apache Pulsar 2.9.2 release

2022-01-23 Thread Ran Gao
Sorry, late response, please take a look at the previous reply. If you have good ideas, we could start a discussion. On 2022/01/05 18:31:38 Michael Marshall wrote: > +1 - thanks for starting this thread. > > >171 commits? > > Why? > > This is too much in my opinion for a point release. > > I ag

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

2022-01-23 Thread GitBox
mattisonchao commented on issue #441: URL: https://github.com/apache/pulsar-manager/issues/441#issuecomment-1019492858 @michaeljmarshall Yes, That's exactly what I thought. I think we can add ``herddb image`` in **docker-compose** and remove raw database logic from pulsar-man

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

2022-01-23 Thread Zike Yang
> This is true of the DLQ producer, but by using the metadata map, won't we be exposing this feature to end users implicitly because they could add the field to the metadata map for other producers? (I am okay with giving users this functionality, but I know that Matteo said in a community meeting