Hi Anton, I replied on the JIRA. I do not think this is a bug, you just failed to account for implicit defaults in your protocol code. That is, 0 is the default of numeric fields if no other default is specified, etc.
best, Colin On Mon, Oct 21, 2024, at 08:07, Anton Agestam wrote: > Hi everyone, > > I have found a protocol serialization bug that surfaces only with one of > the entities introduced for KIP-853 (UpdateRaftVoterResponse). > > Due to the irreversible implications this might have once merged, I'd argue > that this needs to be considered a release blocker. > > https://issues.apache.org/jira/browse/KAFKA-17845 > > BR, > Anton > > Den tors 10 okt. 2024 kl 23:16 skrev Colin McCabe <cmcc...@apache.org>: > >> This is the second candidate for the release of Apache Kafka 3.9.0. I have >> titled it rc2 since I had an rc1 which got very far, even to the point of >> pushing tags and docker images, before I spotted an issue. So rather than >> mutate the tags, I decided to skip over rc1. >> >> - This is a major release, the final one in the 3.x line. (There may of >> course be other minor releases in this line, such as 3.9.1.) >> - Tiered storage will be considered production-ready in this release. >> - This will be the final major release to feature the deprecated ZooKeeper >> mode. >> >> This release includes the following KIPs: >> - KIP-853: Support dynamically changing KRaft controller membership >> - KIP-1057: Add remote log metadata flag to the dump log tool >> - KIP-1049: Add config log.summary.interval.ms to Kafka Streams >> - KIP-1040: Improve handling of nullable values in InsertField, >> ExtractField, and other transformations >> - KIP-1031: Control offset translation in MirrorSourceConnector >> - KIP-1033: Add Kafka Streams exception handler for exceptions occurring >> during processing >> - KIP-1017: Health check endpoint for Kafka Connect >> - KIP-1025: Optionally URL-encode clientID and clientSecret in >> authorization header >> - KIP-1005: Expose EarliestLocalOffset and TieredOffset >> - KIP-950: Tiered Storage Disablement >> - KIP-956: Tiered Storage Quotas >> >> Release notes for the 3.9.0 release: >> https://dist.apache.org/repos/dist/dev/kafka/3.9.0-rc2/RELEASE_NOTES.html >> >> *** Please download, test and vote by October 16, 2024. >> >> Kafka's KEYS file containing PGP keys we use to sign the release: >> https://kafka.apache.org/KEYS >> >> * Release artifacts to be voted upon (source and binary): >> https://dist.apache.org/repos/dist/dev/kafka/3.9.0-rc2/ >> >> * Docker release artifacts to be voted upon: >> apache/kafka:3.9.0-rc2 >> apache/kafka-native:3.9.0-rc2 >> >> * Maven artifacts to be voted upon: >> https://repository.apache.org/content/groups/staging/org/apache/kafka/ >> >> * Javadoc: >> https://dist.apache.org/repos/dist/dev/kafka/3.9.0-rc2/javadoc/ >> >> * Documentation: >> https://kafka.apache.org/39/documentation.html >> >> * Protocol: >> https://kafka.apache.org/39/protocol.html >> >> * Tag to be voted upon (off 3.9 branch) is the 3.9.0-rc2 tag: >> https://github.com/apache/kafka/releases/tag/3.9.0-rc2 >> >> * Successful Docker Image Github Actions Pipeline for 3.9 branch: >> Docker Build Test Pipeline (JVM): >> https://github.com/apache/kafka/actions/runs/11281563007 >> Docker Build Test Pipeline (Native): >> https://github.com/apache/kafka/actions/runs/11281608809 >> >> Thanks to everyone who helped with this release candidate, either by >> contributing code, testing, or documentation. >> >> Regards, >> Colin >>