+1(binding)
Mattison On 10 Jul 2023 at 19:55 +0800, Joo Hyuk Kim <beansk...@gmail.com>, wrote: > Thanks, > JooHyuk. > > On 2023/07/10 00:24:20 Zili Chen wrote: > > +1 (non-binding) > > > > On 2023/07/07 09:25:22 Joo Hyuk Kim wrote: > > > > Hi community, > > > > > > > > This PIP has received a couple of approvals in github PR link [1] > > > > So I thought it's time to vote. > > > > > > > > ## Motivation > > > > > > > > In the current Pulsar codebase, the logic to parse CLI arguments for > > > > measurement units like time and bytes is > > > > > > > > scattered across various CLI classes. Each value read has its distinct > > > > parsing implementation, leading to a lack of code > > > > > > > > reuse. > > > > > > > > > > > > ## Goals > > > > > > > > > > > > This PIP is to refactor the argument parsing logic to leverage the > > > > `@Parameter.converter` > > > > > > > > functionality provided by JCommander [link 3]. This will isolate the > > > > measurement-specific parsing logic and increase > > > > > > > > code > > > > > > > > reusability. > > > > > > > > > > > > ### In Scope > > > > > > > > > > > > - Refactor all `Cmd` classes to utilize the converter functionality of > > > > JCommander. This will streamline the parsing > > > > > > > > logic and simplify the codebase. > > > > > > > > - Refer to bottom section "Concrete Example", before "Links" > > > > > > > > - Or on-going PR with small use case in > > > > https://github.com/apache/pulsar/pull/20663 > > > > > > > > > > > > ## links > > > > > > > > > > > > [1] PR : https://github.com/apache/pulsar/pull/20691 > > > > > > > > > > > > > > > > Best regards, > > > > > > > > JooHyukKim (Vince) > > > > > >
