KAFKA-9353: Add groupInstanceId to DescribeGroup for better visibility

[dengziming]

Kafka-8538(#6957) has already added group.instance.id to MemberDescription but didn't print it, so we just print it.
before the change, the describe command print as follows:

GROUP           CONSUMER-ID                                                  HOST            CLIENT-ID               #PARTITIONS     
DemoConsumer    consumer-DemoConsumer-2-89251f12-f0ae-4dc1-a118-bda49f2a6e86 /127.0.0.1      consumer-DemoConsumer-2 0               
DemoConsumer    consumer-DemoConsumer-1-72221c6b-f3d9-4c68-96db-ffffa12ddf93 /127.0.0.1      consumer-DemoConsumer-1 1               

after the change, the describe command print as follows:

GROUP           CONSUMER-ID                                    GROUP-INSTANCE-ID HOST            CLIENT-ID                       #PARTITIONS     
DemoConsumer    groupIns2-f050379c-9c0d-433c-bbe0-44de6177b60d groupIns2         /127.0.0.1      consumer-DemoConsumer-groupIns2 0               
DemoConsumer    groupIns1-44805ba9-ae6f-49d3-89af-44a4b95aff8d groupIns1         /127.0.0.1      consumer-DemoConsumer-groupIns1 1                        

if all the GROUP-INSTANCE-ID is null, just as the previous:

GROUP           CONSUMER-ID                                                  HOST            CLIENT-ID               #PARTITIONS     
DemoConsumer    consumer-DemoConsumer-2-89251f12-f0ae-4dc1-a118-bda49f2a6e86 /127.0.0.1      consumer-DemoConsumer-2 0               
DemoConsumer    consumer-DemoConsumer-1-72221c6b-f3d9-4c68-96db-ffffa12ddf93 /127.0.0.1      consumer-DemoConsumer-1 1               

[dengziming]

@abbccdda PTAL.

[ghost]

The columns don't line up nicely for me when there is no group instance id:

GROUP                  CONSUMER-ID                                     GROUP-INSTANCE-ID HOST            CLIENT-ID       #PARTITIONS
console-consumer-71167 consumer-1-7969ebf4-6c42-402b-a3e3-ed5d66596ac7                 /127.0.0.1      consumer-1      1

[abbccdda]

Thanks for the PR @dengziming. I think the change makes sense, could you also address @WheresAlice comment?

[dengziming]

@WheresAlice thank you. this is because of the default column width is 15, but len("GROUP-INSTANCE-ID")=17, so I just change it to 17, and it's better now:

GROUP           CONSUMER-ID                                                  GROUP-INSTANCE-ID  HOST            CLIENT-ID               #PARTITIONS     
DemoConsumer    consumer-DemoConsumer-1-5b54f273-a784-4bc2-9f17-46495b01917b                    /192.168.53.21  consumer-DemoConsumer-1 1               

@abbccdda take a look again.

[ghost]

Yes, that's looking better to me too:

GROUP                  CONSUMER-ID                                     GROUP-INSTANCE-ID HOST            CLIENT-ID       #PARTITIONS
console-consumer-45143 consumer-1-d4eb6170-8aca-46fd-b3cb-f6fc2dc8ceba                   /127.0.0.1      consumer-1      1

LGTM

[abbccdda]

Thanks for the patch, LGTM! Will ping @hachikuji if he has time to take another look

[dengziming]

@ijuma Hi, PTLA.
when groupInsId all null

GROUP           CONSUMER-ID                                                  HOST            CLIENT-ID               #PARTITIONS     
DemoConsumer    consumer-DemoConsumer-2-89251f12-f0ae-4dc1-a118-bda49f2a6e86 /127.0.0.1      consumer-DemoConsumer-2 0               
DemoConsumer    consumer-DemoConsumer-1-72221c6b-f3d9-4c68-96db-ffffa12ddf93 /127.0.0.1      consumer-DemoConsumer-1 1               

when at least one column is not null

GROUP           CONSUMER-ID                                                  GROUP-INSTANCE-ID HOST            CLIENT-ID                       #PARTITIONS     
DemoConsumer    consumer-DemoConsumer-1-ff056df8-f2da-4ec5-af7d-0f1b5a8ea74c                   /127.0.0.1      consumer-DemoConsumer-1         0               
DemoConsumer    groupIns1-63ef1547-3ae8-4d1f-89c7-5d42570c8029               groupIns1         /127.0.0.1      consumer-DemoConsumer-groupIns1 1               

[dengziming]

@hachikuji hi, resolved, PTAL.

[dengziming]

cc @hachikuji , PTAL.

[hachikuji]

LGTM. Thanks for the patch!

[hachikuji]

retest this please

[mjsax]

Retest this please.

[mjsax]

LGTM.