I see. Not sure if we need/want this atm -- will introduce some runtime overhead accessing the `Set` twice. I am not opposed, just unsure. \cc @guozhangwang @bbejeck WDYT?
If we keep it as-is, we should add a comment and explain why we do this (otherwise, we might change the later on, because we forget about it). [ Full content available at: https://github.com/apache/kafka/pull/5724 ] This message was relayed via gitbox.apache.org for [email protected]
