Re:Re: Re: [DISCUSS] PIP-270 Add config to set metadata size threshold for compression.

2023-05-29 Thread lifepuzzlefun
fixed : - ) and more git code link provide in the pip. At 2023-05-29 09:31:31, "PengHui Li" wrote: >> managedLedgerInfoCompressionThresholdInBytes >managedLedgerInfoCompressionThresholdInBytes > >It should be > >managedLedgerInfoCompressionThresholdInBytes

Re: Re: [DISCUSS] PIP-270 Add config to set metadata size threshold for compression.

2023-05-28 Thread PengHui Li
> managedLedgerInfoCompressionThresholdInBytes managedLedgerInfoCompressionThresholdInBytes It should be managedLedgerInfoCompressionThresholdInBytes managedCursorInfoCompressionThresholdInBytes And it is better to add a link to the data structure to help reviewers to understand the proposal.

RE: Re: [DISCUSS] PIP-270 Add config to set metadata size threshold for compression.

2023-05-27 Thread lifepuzzlefun
agree. update the pip with more backgroud. On 2023/05/22 12:19:12 Asaf Mesika wrote: > Hi, > > I have a couple of questions: > > current we support to set compression on ManagedLedger and ManagedCursor > > metadata > > by managedLedgerInfoCompressionType and managedCursorInfoCompressionType >

Re: [DISCUSS] PIP-270 Add config to set metadata size threshold for compression.

2023-05-22 Thread Asaf Mesika
Hi, I have a couple of questions: current we support to set compression on ManagedLedger and ManagedCursor > metadata > by managedLedgerInfoCompressionType and managedCursorInfoCompressionType Can you please elaborate what is the metadata of ManagedLedger and ManagedCursor? Where is that

[DISCUSS] PIP-270 Add config to set metadata size threshold for compression.

2023-05-11 Thread lifepuzzlefun
Dear Pulsar community, I create a pip aim to add configuration on size based metadata compresssion. https://github.com/apache/pulsar/issues/20307 We welcome your feedback and suggestions on this proposal.