回复： [DISCUSS] PIP-346: Add a simplified configuration file for Pulsar

[thetumbled <wo...@qq.com.INVALID>]

Hi Lari,
  Based on my understanding, this new lib can merge multiple configuration file with some file having higher priority, which is a nice feature as this pip try to introduce another simplified configuration file.
  But i wonder why we can't move forward with PIP-346 without introducing Gestalt. Whether introducing Gestalt or not, we will introduce a new simplified configuration file. I don't get why we should block this PIP?

Regards,
thetumbled.






------------------ 原始邮件 ------------------
发件人:                                                                                                                        "dev"                                                                                    <lhotari@apache.org&gt;;
发送时间:&nbsp;2024年3月15日(星期五) 晚上11:16
收件人:&nbsp;"dev"<dev@pulsar.apache.org&gt;;

主题:&nbsp;Re: [DISCUSS] PIP-346: Add a simplified configuration file for Pulsar



Before making this change, I hope that there could be discussion about
improving the configuration solution in Pulsar.
The mailing list thread is
https://lists.apache.org/thread/8splwyrn25pt5rb6ph0yz0tfgknn3hqh .

With a proper configuration library such as Gestalt Config [1], it's
possible to have a solution that is expressed in this PIP-346.
In Gestalt Config, the feature "Merge Multiple Sources: Seamlessly
merge configurations from different sources to create comprehensive
settings." support this.

This is the reason why I don't think that we should move forward with
PIP-346 without addressing the proposal in the other mailing list
thread
"[DISCUSSION] Proposal to Replace Pulsar's Homegrown Configuration
Framework with Gestalt Config" [2].

-Lari

1 - https://github.com/gestalt-config/gestalt#features
2 - https://lists.apache.org/thread/8splwyrn25pt5rb6ph0yz0tfgknn3hqh

On Fri, 15 Mar 2024 at 04:30, thetumbled <wof567@qq.com.invalid&gt; wrote:
&gt;
&gt; Hi, pulsar community.
&gt; &amp;nbsp; &amp;nbsp; I open a new PIP for improving the configuration of Pulsar.
&gt; [improve][pip] PIP-346: Add a simplified configuration file for Pulsar by thetumbled · Pull Request #22274 · apache/pulsar (github.com)
&gt;
&gt;
&gt; Thanks.
&gt; thetumbled.