jsancio commented on PR #16669:
URL: https://github.com/apache/kafka/pull/16669#issuecomment-2265713681

   @cmccabe it looks like the tool is not grabbing the hostname correctly:
   ```bash
   $ git diff
   diff --git a/config/kraft/server.properties b/config/kraft/server.properties
   index dfa9489cbc..f34b18c889 100644
   --- a/config/kraft/server.properties
   +++ b/config/kraft/server.properties
   @@ -28,6 +28,7 @@ node.id=1
   
    # The connect string for the controller quorum
    controller.quorum.voters=1@localhost:9093
   +unstable.feature.versions.enable=true
   
    ############################# Socket Server Settings 
#############################
   ```
   
   ```bash
   bin/kafka-dump-log.sh --cluster-metadata-decoder --files 
/tmp/kraft-combined-logs/__cluster_metadata-0/00000000000000000000-0000000000.checkpoint
   Dumping 
/tmp/kraft-combined-logs/__cluster_metadata-0/00000000000000000000-0000000000.checkpoint
   Snapshot end offset: 0, epoch: 0
   baseOffset: 0 lastOffset: 2 count: 3 baseSequence: -1 lastSequence: -1 
producerId: -1 producerEpoch: -1 partitionLeaderEpoch: 0 isTransactional: false 
isControl: true deleteHorizonMs: OptionalLong.empty position: 0 CreateTime: 
1722614412657 size: 160 magic: 2 compresscodec: none crc: 1500076082 isvalid: 
true
   | offset: 0 CreateTime: 1722614412657 keySize: 4 valueSize: 11 sequence: -1 
headerKeys: [] SnapshotHeader 
{"version":0,"lastContainedLogTimestamp":1722614412648}
   | offset: 1 CreateTime: 1722614412657 keySize: 4 valueSize: 5 sequence: -1 
headerKeys: [] KRaftVersion {"version":0,"kRaftVersion":1}
   | offset: 2 CreateTime: 1722614412657 keySize: 4 valueSize: 50 sequence: -1 
headerKeys: [] KRaftVoters 
{"version":0,"voters":[{"voterId":1,"voterDirectoryId":"Xf-d77HEShOwt6TTMEVApQ","endpoints":[{"name":"CONTROLLER","host":"null","port":9093}],"kRaftVersionFeature":{"minSupportedVersion":0,"maxSupportedVersion":1}}]}
   baseOffset: 3 lastOffset: 3 count: 1 baseSequence: -1 lastSequence: -1 
producerId: -1 producerEpoch: -1 partitionLeaderEpoch: 0 isTransactional: false 
isControl: true deleteHorizonMs: OptionalLong.empty position: 160 CreateTime: 
1722614412661 size: 75 magic: 2 compresscodec: none crc: 1648935618 isvalid: 
true
   | offset: 3 CreateTime: 1722614412661 keySize: 4 valueSize: 3 sequence: -1 
headerKeys: [] SnapshotFooter {"version":0}
   ```
   
   Notice that the host field in the voter set control records is set to 
`null`. I should change the schema to no allow null values for that field.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to