AndrewJSchofield opened a new pull request, #17656:
URL: https://github.com/apache/kafka/pull/17656

   The recent test failure documented by 
https://github.com/apache/kafka/pull/17649 seems to be related to the string 
representation of SharePartitionKey. This is a combination of 
group-id:topic-id:partition used by share groups to refer to a 
"share-partition" which is a particular share group's view of a 
topic-partition. The SharePartitionKey is used to find the share coordinator 
responsible for a share-partition.
   
   It looks like some code was using a simple concatenation of the fields, 
while others was using toString which is human-readable. This mismatch seems to 
have caused requests to be sent to the wrong share coordinator, resulting in 
tests timing out.
   
   This PR aligns all lookup-based uses of SharePartitionKey on the simple 
concatenation of its constituent parts.
   
   ### Committer Checklist (excluded from commit message)
   - [ ] Verify design and implementation 
   - [ ] Verify test coverage and CI build status
   - [ ] Verify documentation (including upgrade notes)
   


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