hi Kamal

Thanks for the response. I will file a jira and KIP for it.

Best,
Chia-Ping

> Kamal Chandraprakash <kamal.chandraprak...@gmail.com> 於 2025年5月3日 晚上10:54 寫道:
> 
> 1. Yes, it was considered during the implementation of KIP-950 but the
> config was not proposed in the KIP and
> the KIP/Feature freeze date was passed by that time.
> 
> 2. Having the RLMFollowerTask thread dynamically configurable is a useful
> feature.
> 
> I'm +1 on adding the config to maintain consistency. We need to propose the
> KIP for it.
> 
> PR: https://github.com/apache/kafka/pull/16502
> 
> Thanks,
> Kamal
> 
> 
> 
>> On Sat, May 3, 2025 at 2:51 PM Chia-Ping Tsai <chia7...@gmail.com> wrote:
>> 
>> hi Kamal
>> 
>> Thanks for your response. I have two minor questions.
>> 
>> 1. Have we considered adding a new config for RLMFolloweTask, such as
>> remote.log.manager.follower.thread.pool.size, for consistency?
>> 
>> 2. Additionally, with the specific config, the size of RLMFolloweTask can
>> be dynamically configured
>> 
>> WDYT?
>> 
>> Best,
>> Chia-Ping
>> 
>> 
>> 
>>> Kamal Chandraprakash <kamal.chandraprak...@gmail.com> 於 2025年5月3日
>> 下午2:29 寫道:
>>> 
>>> Hi Chia-Ping,
>>> 
>>> Thanks for following up on this!
>>> 
>>> The `remote.log.manager.thread.pool.size` was initially planned to be
>>> deprecated as part of KIP-950
>>> <
>> https://cwiki.apache.org/confluence/display/KAFKA/KIP-950%3A++Tiered+Storage+Disablement
>>> ,
>>> 
>>> then it was repurposed to use it as followerThreadPool in KIP-1030
>>> <
>> https://cwiki.apache.org/confluence/display/KAFKA/KIP-1030%3A+Change+constraints+and+default+values+for+various+configurations#KIP1030:Changeconstraintsanddefaultvaluesforvariousconfigurations-#9#10#11Config(s):remote.log.manager.copier.thread.pool.size/remote.log.manager.expiration.thread.pool.size/remote.log.manager.thread.pool.size
>> <https://cwiki.apache.org/confluence/display/KAFKA/KIP-1030%3A+Change+constraints+and+default+values+for+various+configurations#KIP1030:Changeconstraintsanddefaultvaluesforvariousconfigurations-%239%2310%2311Config(s):remote.log.manager.copier.thread.pool.size/remote.log.manager.expiration.thread.pool.size/remote.log.manager.thread.pool.size>
>>> 
>>> .
>>> 
>>> PR: https://github.com/apache/kafka/pull/18137
>>> 
>>> --
>>> Kamal
>>> 
>>> 
>>>> On Sat, May 3, 2025 at 11:09 AM Chia-Ping Tsai <chia7...@apache.org>
>> wrote:
>>>> 
>>>> hi Kamal
>>>> 
>>>> Apologies for posting on this older thread. I have a question regarding
>>>> the configuration parameter remote.log.manager.thread.pool.size.
>>>> 
>>>> Is there a Jira ticket associated with the deprecation of this setting?
>>>> Additionally, is there a replacement configuration available for
>>>> followerThreadPool?
>>>> 
>>>> thanks,
>>>> Chia-Ping
>>>> 
>>>>> On 2024/11/07 16:07:36 Kamal Chandraprakash wrote:
>>>>> Hi Satish,
>>>>> 
>>>>> Thanks for the review! Yes, we won't be enabling the deprecated config
>> as
>>>>> dynamic.
>>>>> Removed the remote.log.manager.thread.pool.size config from the KIP.
>>>>> 
>>>>> Currently, the config is not marked as deprecated which was
>>>>> already proposed in
>>>>> KIP-950, we can mark that config as deprecated in the source code.
>>>>> 
>>>>> Thanks,
>>>>> Kamal
>>>>> 
>>>>> On Thu, Nov 7, 2024 at 9:23 PM Satish Duggana <
>> satish.dugg...@gmail.com>
>>>>> wrote:
>>>>> 
>>>>>> Thanks Kamal for the KIP. This is useful for dynamically changing the
>>>>>> thread pool configurations, especially in production environments. We
>>>>>> can skip remote.log.manager.thread.pool.size as it is already
>>>>>> deprecated, and remote.log.manager.copier.thread.pool.size,
>>>>>> remote.log.manager.expiration.thread.pool.size  are derived from that.
>>>>>> 
>>>>>> ~Satish.
>>>>>> 
>>>>>> 
>>>>>> On Thu, 7 Nov 2024 at 10:07, Kamal Chandraprakash
>>>>>> <kamal.chandraprak...@gmail.com> wrote:
>>>>>>> 
>>>>>>> Hi all,
>>>>>>> 
>>>>>>> If there are no more comments, then I'll start a voting thread as the
>>>>>>> change is minor.
>>>>>>> 
>>>>>>> --
>>>>>>> Kamal
>>>>>>> 
>>>>>>> On Thu, Nov 7, 2024 at 9:00 AM Kamal Chandraprakash <
>>>>>>> kamal.chandraprak...@gmail.com> wrote:
>>>>>>> 
>>>>>>>> Hi Federico,
>>>>>>>> 
>>>>>>>> Updated the KIP by replacing the `isInitialized` to `isReady` in
>>>> the
>>>>>> KIP.
>>>>>>>> 
>>>>>>>> 
>>>>>>>> 
>>>>>>>> On Wed, Nov 6, 2024 at 12:47 PM Federico Valeri <
>>>> fedeval...@gmail.com>
>>>>>>>> wrote:
>>>>>>>> 
>>>>>>>>> Thanks Kamal, LGTM, but you should replace all instances of
>>>>>>>>> isInitialized to isReady in the rest of the KIP.
>>>>>>>>> 
>>>>>>>>> On Wed, Nov 6, 2024 at 5:22 AM Kamal Chandraprakash
>>>>>>>>> <kamal.chandraprak...@gmail.com> wrote:
>>>>>>>>>> 
>>>>>>>>>> Hi Federico,
>>>>>>>>>> 
>>>>>>>>>> Thanks for the review!
>>>>>>>>>> 
>>>>>>>>>> 1. Changed the API name to `isReady`
>>>>>>>>>> 2. Added an example of stacktrace in the KIP.
>>>>>>>>>> 
>>>>>>>>>> PTAL.
>>>>>>>>>> 
>>>>>>>>>> Thanks,
>>>>>>>>>> Kamal
>>>>>>>>>> 
>>>>>>>>>> On Mon, Nov 4, 2024 at 2:37 PM Federico Valeri <
>>>>>> fedeval...@gmail.com>
>>>>>>>>> wrote:
>>>>>>>>>> 
>>>>>>>>>>> Hi Kamal, these changes make sense to me. Thanks.
>>>>>>>>>>> 
>>>>>>>>>>> In this case, I wonder if "isReady" could be a better name,
>>>>>> instead of
>>>>>>>>>>> "isInitialized". Wdyt?
>>>>>>>>>>> 
>>>>>>>>>>> Could you please add an example of the stack trace that the
>>>> RLMM
>>>>>> can
>>>>>>>>>>> raise during the initialization phase?
>>>>>>>>>>> 
>>>>>>>>>>> On Sun, Nov 3, 2024 at 4:50 PM Kamal Chandraprakash
>>>>>>>>>>> <kamal.chandraprak...@gmail.com> wrote:
>>>>>>>>>>>> 
>>>>>>>>>>>> Hi all,
>>>>>>>>>>>> 
>>>>>>>>>>>> I would like to start a discussion thread on KIP-1105
>>>>>>>>>>>> <
>>>>>>>>>>> 
>>>>>>>>> 
>>>>>> 
>>>> 
>> https://cwiki.apache.org/confluence/display/KAFKA/KIP-1105%3A+Make+remote+log+manager+thread-pool+configs+dynamic
>>>>>>>>>>>> .
>>>>>>>>>>>> This KIP is about
>>>>>>>>>>>> 
>>>>>>>>>>>> 1. Configuring the thread-pool used by the remote-log
>>>> manager
>>>>>>>>> dynamically
>>>>>>>>>>>> and
>>>>>>>>>>>> 2. Graceful handling of remote-log components during server
>>>>>> startup.
>>>>>>>>>>>> 
>>>>>>>>>>>> 
>>>>>>>>>>> 
>>>>>>>>> 
>>>>>> 
>>>> 
>> https://cwiki.apache.org/confluence/display/KAFKA/KIP-1105%3A+Make+remote+log+manager+thread-pool+configs+dynamic
>>>>>>>>>>>> 
>>>>>>>>>>>> Please take a look and suggest your thoughts.
>>>>>>>>>>>> 
>>>>>>>>>>>> Thanks,
>>>>>>>>>>>> Kamal
>>>>>>>>>>> 
>>>>>>>>> 
>>>>>>>> 
>>>>>> 
>>>>> 
>>>> 
>> 

Reply via email to