[GitHub] [pulsar-site] eolivelli merged pull request #10: Fix Copyright Footer

2022-03-05 Thread GitBox
eolivelli merged pull request #10: URL: https://github.com/apache/pulsar-site/pull/10 -- 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

Re: [DISCUSSION][PIP-146] ManagedCursorInfo compression

2022-03-05 Thread Haiting Jiang
This is a global setting now. But I wonder if we should compress it only if the size is over a threshold? Because: 1. It's not easy for us to notice some managed cursor info is too large in advance, normally it would be found only if it have actual impact. But if we enable this compression in

[GitHub] [pulsar-site] dave2wave merged pull request #11: Suggested change to rewrite rules

2022-03-05 Thread GitBox
dave2wave merged pull request #11: URL: https://github.com/apache/pulsar-site/pull/11 -- 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

Re: [DISCUSSION][PIP-146] ManagedCursorInfo compression

2022-03-05 Thread mattison chao
Hi, Zi Xuan After deep think, I have another question: Why don't we combine ledger compaction and cursor compaction into one configuration switch? I’m not sure, do we have users who need to set the compression configuration for ledger and cursor separately? I think if they were to be set, they

Re: [DISCUSSION][PIP-146] ManagedCursorInfo compression

2022-03-05 Thread mattison chao
Great work! I have no other comments other than the compatibility everybody mentioned. Best, Mattison > On Mar 6, 2022, at 4:55 AM, Enrico Olivelli wrote: > > Good proposal. > It is important that this is disabled by default otherwise we cannot easily > support the rollback > > Apart from t

Re: [VOTE] [PIP-143] Support split bundle by specified boundaries

2022-03-05 Thread mattison chao
+1 Best Mattison > On Mar 3, 2022, at 11:08 AM, r...@apache.org wrote: > > +1 > > Thanks > Xiaolong Ran > > PengHui Li 于2022年2月27日周日 13:04写道: > >> +1 >> >> Thanks, >> Penghui >> >> On Sat, Feb 26, 2022 at 11:59 AM Haiting Jiang >> wrote: >> >>> +1 >>> >>> Thanks, >>> Haiting >>> >>>

Re: [DISCUSSION][PIP-146] ManagedCursorInfo compression

2022-03-05 Thread Enrico Olivelli
Good proposal. It is important that this is disabled by default otherwise we cannot easily support the rollback Apart from that I don't have other comments Enrico Il Sab 5 Mar 2022, 11:22 PengHui Li ha scritto: > Hi Zixuan, > > We should add the compatibility part to the proposal. > And shou

[GitHub] [pulsar-site] dave2wave merged pull request #9: Adjust .htaccess

2022-03-05 Thread GitBox
dave2wave merged pull request #9: URL: https://github.com/apache/pulsar-site/pull/9 -- 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-site] dave2wave opened a new pull request #9: Adjust .htaccess

2022-03-05 Thread GitBox
dave2wave opened a new pull request #9: URL: https://github.com/apache/pulsar-site/pull/9 Adjustments to .htaccess. Also going to move the `.htaccess` file to the top level -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitH

Re: [DISCUSSION][PIP-146] ManagedCursorInfo compression

2022-03-05 Thread PengHui Li
Hi Zixuan, We should add the compatibility part to the proposal. And should also provide steps to roll back to the old version which enabled the compression in the new version. I don't have objections to the proposal, and we have done the same enhancement for topic metadata https://github.com/apa

Re: [DISCUSS] Releasing pulsar-client-go 0.8.1

2022-03-05 Thread PengHui Li
+1 Penghui On Sat, Mar 5, 2022 at 4:58 AM Matteo Merli wrote: > +1 Thanks Rui, we should eliminate the GPL dependency ASAP. > > > > -- > Matteo Merli > > > On Thu, Mar 3, 2022 at 2:08 AM Rui Fu wrote: > > > > Hi everyone, > > > > I would like to start a discussion here about starting a new re

Re: [DISCUSS] PIP-139 : Support Broker send command to real close producer/consumer.

2022-03-05 Thread PengHui Li
> Great point. I was focused on deleting namespaces and missed the case where the user wants to delete a topic from a replicated namespace. I agree that we should make it possible to delete these topics without removing the namespace replication. Oh, sorry. I thought again, after https://github.co