[Discussion] Add a new default method to to the ManagedLedgerInterceptor

2023-03-13 Thread Gavin gao
The issue is: https://github.com/apache/pulsar/pull/19727 This issue tries to address the index generator rolled back problem when entries add failed. A new default method `afterFailedAddEntry` is added to interface ManagedLedgerInterceptor. And this change should be also cherry-picked to releas

Re: [VOTE] Pulsar Client Python Release 3.1.0 Candidate 4

2023-03-13 Thread Yunze Xu
Close this vote by 3 binding +1s: * Penghui * Matteo * Bo and 1 non-binding +1: * Zike Thanks, Yunze On Mon, Mar 13, 2023 at 11:14 AM Zike Yang wrote: > > > It's not a regression. This error applies for Python client 3.0.0 as > > well. Before 3.0.0, the Python client cannot be interrupted by a

Re: [DISCUSS] PIP-254: Support configuring client version at SDK level

2023-03-13 Thread Yunze Xu
Hi Zike and Michael, I agree that we can move this method to `ClientBuilderImpl`. Generally users should not set this description, the client version string is used to identify a specific implementation of the Pulsar API. If there is a strong demand to configure the description for the API layer (

[Vote] PIP-248: Add metrics for backlog eviction events

2023-03-13 Thread 太上玄元道君
Hi all, The thread is to start the vote for PIP-248 Discussion thread: https://lists.apache.org/thread/m7khb49pk7vdx5yxvhzhkbo7t48d0rzs PIP link: https://github.com/apache/pulsar/issues/19601 Thanks, Tao Jiuming

Re: [Discussion] Allowing configure if function consumer should skip to latest

2023-03-13 Thread PengHui Li
Got it. Thanks for the explanation. LGTM Penghui On Thu, Mar 9, 2023 at 3:06 AM Neng Lu wrote: > Hi everyone, > > This discussion has been one week old. If there's no objection or > concerns, I'll move forward and close the discussion with the conclusion > that we are good with the proposed ch

Re: [DISCUSS] Merge GitHub Discussions categories "Ideas" and "Show and Tell" into "General"

2023-03-13 Thread Yu
Hi tison, Thanks for bringing this up! >From the official explanations [1], there is a clear distinction line between them, so users can choose desired/correct categories easily. [1] https://docs.github.com/en/discussions/managing-discussions-for-your-community/managing-categories-for-discussion

Re: Data quality problem

2023-03-13 Thread Devin Bost
> Sorry. I do not fully understand here. Is it also related to the "data > quality" problem > that we discussed? For the consumer side, we can use the AUTO_CONSUME schema > to receive GenericObject (For JSON schema, you can deal with JsonObject > directly). > For the producer side, I think yes. We

[Discuss] PIP-256: Building Great Developer Experience with API Content

2023-03-13 Thread Yu
Hi community, Based on the [2022 Report] Pulsar Website Content Analysis (GA) [1], the Pulsar API doc is one of the top-viewed content. However, Pulsar API was not systematically and logically explained previously. To address this issue, I've created a content strategy and designed the informatio

[DISCUSS] Merge GitHub Discussions categories "Ideas" and "Show and Tell" into "General"

2023-03-13 Thread tison
Hi, >From my experience in other projects, there's little difference between all open-ended discussions. I suggest we merge the GitHub Discussions categories "Ideas" and "Show and Tell" into "General" to reduce decidophobia. Thoughts? Best, tison.

[DISCUSS] PIP-258: Deprecation of the consumer subscribeTopicMode configuration

2023-03-13 Thread Baodi Shi
Hi all, I've started a PIP to discuss: PIP-258: Deprecation of the consumer subscribeTopicMode configuration ### Motivation About pattern subscribes of consumers, the `topicsPattern` and `subscribeTopicMode` configurations are contradictory. For example, the `topicsPattern` represents only subs

Re: [VOTE] PIP-250: Add proxyVersion to CommandConnect

2023-03-13 Thread Zixuan Liu
+1 (non-binding) Thanks, Zixuan Yunze Xu 于2023年3月13日周一 17:31写道: > +1 (binding) > > Thanks, > Yunze > > On Mon, Mar 13, 2023 at 3:48 PM Zike Yang wrote: > > > > +1 (non-binding) > > > > Thanks > > Zike Yang > > > > On Sat, Mar 11, 2023 at 12:08 PM Yubiao Feng > > wrote: > > > > > > +1 (non-bin

Re: [VOTE] PIP-250: Add proxyVersion to CommandConnect

2023-03-13 Thread Yunze Xu
+1 (binding) Thanks, Yunze On Mon, Mar 13, 2023 at 3:48 PM Zike Yang wrote: > > +1 (non-binding) > > Thanks > Zike Yang > > On Sat, Mar 11, 2023 at 12:08 PM Yubiao Feng > wrote: > > > > +1 (non-binding) > > > > Thanks > > Yubiao Feng > > > > On Sat, Mar 11, 2023 at 3:31 AM Michael Marshall > >

Re: [DISCUSS] PIP-257: Add Open ID Connect Support to Server Components

2023-03-13 Thread Zixuan Liu
Hi Michael, +1, Thank you for your PIP! That's important for modern authentication! I have a question: > 2. Implement `KubernetesFunctionAuthProvider` with `KubernetesSecretsAuthProvider`. It looks like we add an authentication provider for the Kubernetes environment. Is the OIDC authentication

Re: [VOTE] PIP-250: Add proxyVersion to CommandConnect

2023-03-13 Thread Zike Yang
+1 (non-binding) Thanks Zike Yang On Sat, Mar 11, 2023 at 12:08 PM Yubiao Feng wrote: > > +1 (non-binding) > > Thanks > Yubiao Feng > > On Sat, Mar 11, 2023 at 3:31 AM Michael Marshall > wrote: > > > Hello Pulsar Community, > > > > This thread is to start the vote for PIP 250. > > > > Discussio

Re: [DISCUSS] PIP-254: Support configuring client version at SDK level

2023-03-13 Thread Zike Yang
> I'd rather use the "description" term, which indicates the client version has extra description in addition to the client version string. The `description` term and 64 length limit all make sense to me. > Is there any way we can avoid giving regular users easy access to this field via the Clien