Dear all,
If no one has further comments.  I want to start a vote thread next week.

Thank you,
Wenbing

wenbing shen <oliver.shen...@gmail.com> 于2021年4月1日周四 上午2:13写道:

> Hi Tom,
> Thank you so much for your good suggestions.
> Based on your suggestions above,  I've updated in the KIP-727.
>
> I used the following options for testing:
> *--non-preferred-leader *for the more human readable output.
> *--non-preferred-leader-json *for the JSON output would be used by 
> kafka-preferred-replica-election.sh
> and kafka-leader-election.sh tools.
>
> Thank you,
> Wenbing
>
> Tom Bentley <tbent...@redhat.com> 于2021年3月30日周二 下午7:01写道:
>
>> Hi Wenbing,
>>
>> Thanks for the KIP, I can see this being useful.
>>
>> 1. Could you include an example command line and sample output of the
>> command in the KIP?
>>
>> 2. I think a better name for the option would be --non-preferred-leader
>> rather than --under-preferred-replica-partitions
>>
>> 3. One of the things a user might do after running this command would be
>> to
>> use `kafka-preferred-replica-election.sh` to try electing the preferred
>> leader for (some of) those partitions.
>> `kafka-preferred-replica-election.sh` reads a JSON file containing the
>> partitions which should have the preferred leader elected. It would be
>> tedious if the user had to reformat the output of `kafka-topics.sh
>> --non-preferred-leader` in order to pass it to
>> `kafka-preferred-replica-election.sh`. The tools would need to use a
>> common
>> format to do that, most obviously by `kafka-topics.sh
>> --non-preferred-leader` emiting the existing JSON format. It's a little
>> awkward because the other outputs from kafka-topics.sh don't naturally get
>> used as inputs to the other tools, so I don't think a top level option,
>> such as `--output=json` would be a good fit. I did wonder if
>> `--non-preferred-leader=json` could be used for the JSON output, and plain
>> `--non-preferred-leader` for the more human readable output, but maybe
>> `--non-preferred-leader-json` (instead of --non-preferred-leader) or
>> `--non-preferred-leader-format=json` (as well as --non-preferred-leader)
>> is
>> a more obvious way to do it.
>>
>> Kind regards,
>>
>> Tom
>>
>> On Sat, Mar 27, 2021 at 2:10 PM wenbing shen <oliver.shen...@gmail.com>
>> wrote:
>>
>> > Hi everyone,
>> >
>> > I'd like to discuss the following proposal to add
>> > --under-preferred-replica-partitions option to describe topics command.
>> >
>> >
>> >
>> https://cwiki.apache.org/confluence/display/KAFKA/KIP-727%3A+Add+--under-preferred-replica-partitions+option+to+describe+topics+command
>> >
>> > Many thanks,
>> >
>> > Wenbing
>> >
>> >
>>
>

Reply via email to