Re: [DISCUSS] KIP-727 Add --under-preferred-replica-partitions option to describe topics command

2021-04-10 Thread wenbing shen
Dear all,
If no one has further comments.  I want to start a vote thread next week.

Thank you,
Wenbing

wenbing shen  于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  于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 
>> 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
>> >
>> >
>>
>


Re: [DISCUSS] KIP-727 Add --under-preferred-replica-partitions option to describe topics command

2021-03-31 Thread wenbing shen
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  于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 
> 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
> >
> >
>


Re: [DISCUSS] KIP-727 Add --under-preferred-replica-partitions option to describe topics command

2021-03-30 Thread Tom Bentley
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 
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
>
>


[DISCUSS] KIP-727 Add --under-preferred-replica-partitions option to describe topics command

2021-03-27 Thread wenbing shen
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