[DISCUSS] Pulsar Maven Builds w/ Unit Tests

2023-07-07 Thread Dave Fisher
Hi - I challenge the dev community about the state of building these branches 2.11, 3,0, and Master when you actually perform all of the unit tests. In the website it seems that we always tell everyone to use `mvn clean install -DskipTests` we do this because tests take too long and are flaky.

Re: [DISCUSS] Cherry-pick PR-16059 to 2.10 to prevent infinite unloading

2023-07-07 Thread Heesung Sohn
Hi dev, I think we also need to consider the namespace anit-affinity-group logic too. These logics seem to do similar things. https://pulsar.apache.org/docs/3.0.x/administration-load-balance/#distribute-anti-affinity-namespaces-across-failure-domains PengHui We got three biding votes here. Do

Re: Failover Subscription - consumer assignment logic discussion

2023-07-07 Thread Yu
Hi Penghui and Girish, Thanks for your discussions and technical guidance! I've optimized the docs in https://github.com/apache/pulsar-site/pull/633, could you PTAL? Anyone else who wants to take a look and comment, feel free! Thanks! Yu On Tue, Jul 4, 2023 at 8:31 AM PengHui Li wrote: > >

Re: [VOTE] PIP-280 : Refactor CLI Argument Parsing Logic for Measurement Units using JCommander's custom converter

2023-07-07 Thread Yunze Xu
+1 (binding) Thanks, Yunze On Fri, Jul 7, 2023 at 5:25 PM Joo Hyuk Kim wrote: > > Hi community, > > This PIP has received a couple of approvals in github PR link [1] > So I thought it's time to vote. > > ## Motivation > > In the current Pulsar codebase, the logic to parse CLI arguments for >

Re: [VOTE] Pulsar Node.js Client Release 1.9.0 Candidate 1

2023-07-07 Thread Yunze Xu
+1 (binding) - Verified checksum and signature - Build from source on macOS m1 with Node.js v20.4.0 - Run basic e2e examples Two additional notes: 1. You should use https://downloads.apache.org/pulsar/KEYS instead of https://dist.apache.org/repos/dist/dev/pulsar/KEYS 2. When building from the

[VOTE] PIP-280 : Refactor CLI Argument Parsing Logic for Measurement Units using JCommander's custom converter

2023-07-07 Thread Joo Hyuk Kim
Hi community, This PIP has received a couple of approvals in github PR link [1] So I thought it's time to vote. ## Motivation In the current Pulsar codebase, the logic to parse CLI arguments for measurement units like time and bytes is scattered across various CLI classes. Each value read has

[DISCUSS] Move superuser/tenantAdmin checks to AuthorizationService from AuthorizationProvider

2023-07-07 Thread Zixuan Liu
Hello everyone, When a role wants to use the resource, the role needs to have resource permissions. The process is to first check whether the role is the superuser or tenant administrator. If yes, operations are allowed. Otherwise, check the policies stored in zk. Right now, we have the