Gimini-3 opened a new pull request, #23083:
URL: https://github.com/apache/kafka/pull/23083

   [KAFKA-20894](https://issues.apache.org/jira/browse/KAFKA-20894)
   
   `DescribeStreamsGroupsOptions` omits authorized operations by default. The 
omitted response value is converted to `null`, which is a documented valid 
state for `StreamsGroupDescription.authorizedOperations()`. However, 
`StreamsGroupDescription.toString()` unconditionally called `stream()` on that 
nullable set, so logging or otherwise rendering a successfully described group 
could throw `NullPointerException`.
   
   This change renders omitted authorized operations as `null` while preserving 
the existing output format for populated sets. It also extends the existing 
AdminClient test for an omitted authorized-operations response to verify that 
the resulting description can be converted to a string and includes 
`authorizedOperations=null`.
   
   Testing:
   - `DOCKER_API_VERSION=1.40 ./gradlew clients:test`
   - `./gradlew clients:test --tests 
org.apache.kafka.clients.admin.KafkaAdminClientTest.testDescribeStreamsGroupsWithAuthorizedOperationsOmitted
 --tests 
org.apache.kafka.clients.admin.internals.DescribeStreamsGroupsHandlerTest`
   - `./gradlew spotlessCheck`
   - `./gradlew clients:spotbugsMain clients:spotbugsTest -x test`
   
   The full clients test suite, targeted regression tests, Checkstyle, SpotBugs 
main analysis, and Spotless checks passed. `clients:spotbugsTest` was skipped 
by the build configuration.


-- 
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