[jira] [Commented] (KAFKA-12161) Raft observers should not require an id to fetch

2021-01-08 Thread Jason Gustafson (Jira)


[ 
https://issues.apache.org/jira/browse/KAFKA-12161?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17261681#comment-17261681
 ] 

Jason Gustafson commented on KAFKA-12161:
-

[~bchen225242] That's not a bad idea. However, we are constrained a little bit 
since Fetch requires an int32 replicaId. I think it might be simpler for the 
moment to make the field optional and treat non-participating observers more 
like consumers.

> Raft observers should not require an id to fetch
> 
>
> Key: KAFKA-12161
> URL: https://issues.apache.org/jira/browse/KAFKA-12161
> Project: Kafka
>  Issue Type: Sub-task
>Reporter: Jason Gustafson
>Assignee: Jason Gustafson
>Priority: Major
>
> It is useful to allow observers to replay the metadata log without requiring 
> a replica id. For example, this can be used by tools in order to inspect the 
> current metadata state. In order to support this, we should modify 
> `KafkaRaftClient` so that the broker id is not required.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (KAFKA-12161) Raft observers should not require an id to fetch

2021-01-07 Thread Boyang Chen (Jira)


[ 
https://issues.apache.org/jira/browse/KAFKA-12161?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17261072#comment-17261072
 ] 

Boyang Chen commented on KAFKA-12161:
-

I was wondering whether we could just get a random UUID for observer when we do 
the tooling?

> Raft observers should not require an id to fetch
> 
>
> Key: KAFKA-12161
> URL: https://issues.apache.org/jira/browse/KAFKA-12161
> Project: Kafka
>  Issue Type: Sub-task
>Reporter: Jason Gustafson
>Assignee: Jason Gustafson
>Priority: Major
>
> It is useful to allow observers to replay the metadata log without requiring 
> a replica id. For example, this can be used by tools in order to inspect the 
> current metadata state. In order to support this, we should modify 
> `KafkaRaftClient` so that the broker id is not required.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)