Re: [DISCUSS] Cherry-pick PR-16059 to 2.10 to prevent infinite unloading

2023-06-30 Thread PengHui Li
+1 for cherry-pick to branch-2.10 since users don't have a workaround for this issue, and the change is well-understand, low risk. I agree with making it false for the next major version release by default. Thanks, Penghui On Sat, Jul 1, 2023 at 9:26 AM Heesung Sohn wrote: > Hi dev, > > I real

[DISCUSS] Cherry-pick PR-16059 to 2.10 to prevent infinite unloading

2023-06-30 Thread Heesung Sohn
Hi dev, I realized that `removeMostServicingBrokersForNamespace` func in the broker selection logic can cause infinite unloading. Suppose an overloaded broker unloaded a bundle and only has the minimum number of bundles(in that namespace) among brokers. In that case, the selection logic (`removeM

[DISCUSS] PIP-280 : Refactor CLI Argument Parsing Logic for Measurement Units using JCommander's custom converter

2023-06-30 Thread Joo Hyuk Kim
Hi, community I am proposing to refactor `Cmd*` classes’s measurement parsing logic. ## Background As it stands, the parsing logic for measurement units such as time and bytes in various CLI classes is implemented verbosely, contained in the Cmd converter themselves. Leading to a lack of code