Re: [DISCUSS] [PIP-165] Auto release client useless connections

2022-05-26 Thread Enrico Olivelli
Make sense to me Enrico Il giorno gio 26 mag 2022 alle ore 08:31 Yubiao Feng ha scritto: > > I open a pip to discuss Auto release client useless connections, could you > help me review > > > ## Motivation > Currently, the Pulsar client keeps the connection even if no producers or > consumers use

Re: [DISCUSS] [PIP-165] Auto release client useless connections

2022-05-26 Thread Gavin Gao
+1 On 2022/05/26 06:31:37 Yubiao Feng wrote: > I open a pip to discuss Auto release client useless connections, could you > help me review > > > ## Motivation > Currently, the Pulsar client keeps the connection even if no producers or > consumers use this connection. > If a client produces mess

[GitHub] [pulsar-site] urfreespace merged pull request #99: backup old website next version docs to website directory

2022-05-26 Thread GitBox
urfreespace merged PR #99: URL: https://github.com/apache/pulsar-site/pull/99 -- 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...@pulsar.apa

Re: [DISCUSS] [PIP-165] Auto release client useless connections

2022-05-26 Thread mattison chao
+1 Best, Mattison On Thu, 26 May 2022 at 15:25, Gavin Gao wrote: > +1 > > > On 2022/05/26 06:31:37 Yubiao Feng wrote: > > I open a pip to discuss Auto release client useless connections, could > you > > help me review > > > > > > ## Motivation > > Currently, the Pulsar client keeps the connecti

[VOTE] PIP-168: Support zero-copy of NIC to NIC on Proxy

2022-05-26 Thread zhaocong
Hi Pulsar Community, I would like to start a VOTE on "Support zero-copy of NIC to NIC on Proxy" (PIP-168). The proposal can be read at https://github.com/apache/pulsar/issues/15631 and the discussion thead is available at https://lists.apache.org/thread/gjys9tvbd5hy28mbkbcq7wkqfldycn7v Voti

Re: Discuss about closing Pulsar resource

2022-05-26 Thread Nicolò Boschi
Hi, the initial intention of https://github.com/apache/pulsar/pull/15638 was about fixing thread leaks in tests which lead to Mockito weird behaviours. Now a few executors do not wait for thread termination and it makes the tests unstable, mainly with mocked objects. The AsyncCloseable addresses a

[GitHub] [pulsar-helm-chart] oguzhantortop commented on issue #132: kop

2022-05-26 Thread GitBox
oguzhantortop commented on issue #132: URL: https://github.com/apache/pulsar-helm-chart/issues/132#issuecomment-1138883377 > Here is a chart which contains kop and mop https://github.com/streamnative/charts/blob/master/charts/pulsar/values.yaml#L94 hi, isn't this chart based on strea

Re: [VOTE] [PIP-153] Optimize metadataPositions in MLPendingAckStore

2022-05-26 Thread Neng Lu
Hi All, +1 (non-binding) On Mon, May 23, 2022 at 5:09 AM Enrico Olivelli wrote: > +1 (binding) > it looks like the patch has already been committed > > > https://github.com/apache/pulsar/commit/ebca19b522fd9f4496689ca7d32ede345d28511a > > Enrico > > Il giorno lun 16 mag 2022 alle ore 07:18 Hang

Re: [DISCUSS] PIP-166: Function add NONE delivery semantics

2022-05-26 Thread Neng Lu
Some suggestions: 1. Instead of deleting the `autoAck`, keep it but not use it in the code. And documented clearly it's deprecated for the following 2~3 release. And then delete it. 2. For `PulsarSinkAtLeastOnceProcessor` and `PulsarSinkEffectivelyOnceProcessor`, if `NONE` is configured, it defaul

Re: [DISCUSS] PIP-157 Exclusive Producer: new mode ExclusiveWithFencing

2022-05-26 Thread Neng Lu
Hi All, Would "Preemptive" Mode make sense? On Wed, May 11, 2022 at 8:56 AM Matteo Merli wrote: > +1 > > > On Tue, May 10, 2022 at 5:56 AM Enrico Olivelli > wrote: > > > Hello, > > I created a new PIP about a new AccessMode for the Producer. > > https://github.com/apache/pulsar/issues/15528 >

Re: [Discuss] Update Helm Chart to Support 2.10 Docker Image

2022-05-26 Thread Neng Lu
Hi All, I'm curious to learn once the image is run as non-root, how can we debug or investigate production issues inside a running cluster? On Thu, May 19, 2022 at 12:14 PM Michael Marshall wrote: > Hello Pulsar Community, > > With the 2.10.0 release, our Pulsar Docker images default to run as

Re: [DISCUSS] [PIP-165] Auto release client useless connections

2022-05-26 Thread Neng Lu
This is a good idea. Also one thing i realize is people are replying with +1, is this considered as a vote? On Thu, May 26, 2022 at 6:26 AM mattison chao wrote: > +1 > > Best, > Mattison > > On Thu, 26 May 2022 at 15:25, Gavin Gao wrote: > > > +1 > > > > > > On 2022/05/26 06:31:37 Yubiao Feng

[GitHub] [pulsar-site] Anonymitaet commented on pull request #97: Sync language tab choices

2022-05-26 Thread GitBox
Anonymitaet commented on PR #97: URL: https://github.com/apache/pulsar-site/pull/97#issuecomment-1139147882 @SignorMercurio yes we need to make the texts for options consistent. Is inconsistency a blocker for your PR? I guess not? @urfreespace can you help review the PR? -- This i

[GitHub] [pulsar-site] urfreespace merged pull request #98: Add committer 'zike'

2022-05-26 Thread GitBox
urfreespace merged PR #98: URL: https://github.com/apache/pulsar-site/pull/98 -- 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...@pulsar.apa

[GitHub] [pulsar-site] urfreespace commented on pull request #97: Sync language tab choices

2022-05-26 Thread GitBox
urfreespace commented on PR #97: URL: https://github.com/apache/pulsar-site/pull/97#issuecomment-1139194263 > @SignorMercurio yes we need to make the texts for options consistent. Is inconsistency a blocker for your PR? I guess not? > > @urfreespace can you help review the PR?

[GitHub] [pulsar-site] Anonymitaet commented on pull request #97: Sync language tab choices

2022-05-26 Thread GitBox
Anonymitaet commented on PR #97: URL: https://github.com/apache/pulsar-site/pull/97#issuecomment-1139195829 @SignorMercurio yes, source doc files live in https://github.com/apache/pulsar/tree/master/site2, the website build process is done in https://github.com/apache/pulsar-site -- This

[GitHub] [pulsar-site] Anonymitaet commented on pull request #97: Sync language tab choices

2022-05-26 Thread GitBox
Anonymitaet commented on PR #97: URL: https://github.com/apache/pulsar-site/pull/97#issuecomment-1139195830 @SignorMercurio yes, source doc files live in https://github.com/apache/pulsar/tree/master/site2, the website build process is done in https://github.com/apache/pulsar-site -- This

[GitHub] [pulsar-site] Anonymitaet commented on pull request #97: Sync language tab choices

2022-05-26 Thread GitBox
Anonymitaet commented on PR #97: URL: https://github.com/apache/pulsar-site/pull/97#issuecomment-1139196817 @SignorMercurio for this improvement, can you open an issue in the Pulsar repo and label it w/ `doc-required`? Thanks > Note that some pages use options like pulsar-admin, REST

[GitHub] [pulsar-site] Anonymitaet commented on pull request #97: Sync language tab choices

2022-05-26 Thread GitBox
Anonymitaet commented on PR #97: URL: https://github.com/apache/pulsar-site/pull/97#issuecomment-1139196816 @SignorMercurio for this improvement, can you open an issue in the Pulsar repo and label it w/ `doc-required`? Thanks > Note that some pages use options like pulsar-admin, REST

Re: [DISCUSS] [PIP-165] Auto release client useless connections

2022-05-26 Thread Haiting Jiang
Hi Yubiao, It seems one possible downside of this pip is increasing produce latency when topic ownership changed, any way to avoid that? Thanks, Haiting On 2022/05/26 06:31:37 Yubiao Feng wrote: > I open a pip to discuss Auto release client useless connections, could you > help me review > >

Re: [DISCUSS] [PIP-165] Auto release client useless connections

2022-05-26 Thread Yubiao Feng
We can make sure there is at least one connection in the pool for lookup, but we can't reduce the time it takes to reconnect to ownership. Do you have a good idea. On Fri, May 27, 2022 at 10:15 AM Haiting Jiang wrote: > Hi Yubiao, > > It seems one possible downside of this pip is increasing prod

Re: [Discuss] Update Helm Chart to Support 2.10 Docker Image

2022-05-26 Thread Michael Marshall
Hi Neng Lu, I put together a doc [0] that includes some tips for troubleshooting a non-root docker image. Some of the details depend on how you're deploying Pulsar. If you can ssh to the host as the root user, you can run `docker exec --user 0 ...` to get a shell in the container as the root user

[GitHub] [pulsar-site] SignorMercurio commented on pull request #97: Sync language tab choices

2022-05-26 Thread GitBox
SignorMercurio commented on PR #97: URL: https://github.com/apache/pulsar-site/pull/97#issuecomment-1139323157 > b, but you need make pr with these docs changes Sure, shall I close this issue? -- This is an automated message from the Apache Git Service. To respond to the message, pl