Hi Jun,

Q2:
For the group.coordinator.rebalance.protocols and process.roles configurations, 
setting them to an 
empty value is invalid—the broker/controller will fail to start in such cases. 
Therefore, I believe it's safe 
to assume that no users would configure them this way, and we can reasonably 
choose not to handle 
this edge case.

> Regarding "The none policy will not delete or compact any segments", we
> should be more accurate. We won't delete segments based on
> log.retention.bytes/log.retention.ms, but we should continue to delete
> segments based on log.local.retention.bytes/log.retention.ms. Otherwise, we
> risk running out of local disk space when remote storage is enabled.

Update the document in KIP.

Best Regards,
Jiunn-Yang


> Jun Rao <j...@confluent.io.INVALID> 於 2025年4月23日 清晨6:32 寫道:
> 
> Hi, Jiunn-Yang,
> 
> Regarding "The none policy will not delete or compact any segments", we
> should be more accurate. We won't delete segments based on
> log.retention.bytes/log.retention.ms, but we should continue to delete
> segments based on log.local.retention.bytes/log.retention.ms. Otherwise, we
> risk running out of local disk space when remote storage is enabled.
> 
> Thanks,
> 
> Jun
> 
> On Tue, Apr 22, 2025 at 9:45 AM Jun Rao <j...@confluent.io> wrote:
> 
>> Hi, Jiunn-Yang,
>> 
>> Thanks for the reply.
>> 
>> Q2. What about existing empty values for
>> group.coordinator.rebalance.protocols and process.roles during upgrade?
>> 
>> Jun
>> 
>> On Tue, Apr 22, 2025 at 7:29 AM 黃竣陽 <s7133...@gmail.com> wrote:
>> 
>>> Hello Jun,
>>> 
>>> Thanks for review this KIP.
>>> 
>>> Q1 & Q3:
>>> I’ve updated the method name accordingly and revised the cleanup.policy
>>> documentation
>>> to clarify that the none policy cannot be used with any other policy.
>>> 
>>> Q2:
>>> For users currently using an empty cleanup.policy, the approach is to
>>> apply the none policy
>>> during the preProcessParsedConfig step. Additionally, a warning message
>>> will be emitted to inform users
>>> of the upcoming change.
>>> 
>>> Best Regards,
>>> Jiunn-Yang
>>> 
>>>> Jun Rao <j...@confluent.io.INVALID> 於 2025年4月22日 凌晨4:52 寫道:
>>>> 
>>>> Hi, Jiunn-Yang,
>>>> 
>>>> Thanks for the KIP. A few comments.
>>>> 
>>>> 1. It's fine to introduce a new value None for cleanup.policy. But now
>>> not
>>>> all value combinations are valid. For example, None can't be used with
>>>> Delete or Compact. It would be useful to document that.
>>>> 2. What's the behavior during upgrade when an existing config has an
>>> empty
>>>> list.
>>>> 3. inWithEmptyCheck: It's not clear what the empty check does. How about
>>>> sth like inNonEmpty ?
>>>> 
>>>> Thanks,
>>>> 
>>>> Jun
>>>> 
>>>> On Tue, Apr 15, 2025 at 8:25 AM 黃竣陽 <s7133...@gmail.com> wrote:
>>>> 
>>>>> Hello everyone,
>>>>> 
>>>>> I would like to start a discussion on KIP-1161: cleanup.policy
>>> shouldn't
>>>>> be empty
>>>>> <https://cwiki.apache.org/confluence/x/HArXF>
>>>>> 
>>>>> This proposal aims to improve the cleanup.policy configuration.
>>> Currently,
>>>>> this setting should not be left empty.
>>>>> Therefore, there are two proposed improvements:
>>>>> 1. Update ValidList to validate whether an empty list is allowed.
>>>>> 2. Introduce a new 'none' value for cleanup.policy.
>>>>> 
>>>>> Best Regards,
>>>>> Jiunn-Yang
>>> 
>>> 

Reply via email to