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

   ### About
   This PR aims to resolve the bug when we have '/' or '+' in `memberId` within 
share fetch and share acknowledge RPCs. The bug arises primarily because we do 
a `string -> Uuid` conversion and vice-versa in the broker for `memberId`. When 
a string containing '/' or '+' is converted to Uuid, the 
`Base64.getUrlDecoder()` is unable to decode the string to a Uuid and gives an 
exception which is the reason for the bug.
   On analysis, it looks like there is no efficiency added due to this 
inter-conversion. Hence, we have gotten rid of the conversion from `string -> 
Uuid` in `KafkaApis` for `memberId`. Thus, everywhere on the broker memberIs 
always treated as string not a Uuid


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