Hi, Colin, Thanks for the KIP. Just a couple of minor comments.
1. Fields like RemovingReplicas are added as tagged fields in PartitionChangeRecord, but as non-tagged fields in PartitionRecord. Should we make them consistent? 2. Should we add BrokerRegistrationChangeRecord later when it has more fields than what's already covered in FenceBrokerRecord and UnfenceBrokerRecord? Jun On Wed, Jun 2, 2021 at 11:02 AM Colin McCabe <[email protected]> wrote: > Hi all, > > I have posted a KIP about updating the KRaft metadata records for 3.0. > > Check it out at : https://cwiki.apache.org/confluence/x/34zOCg > > best, > Colin >
