[GitHub] [pinot] KKcorps edited a comment on issue #7270: Fix the issue with "pinot-pulsar" module (potentially library conflicts)

[GitBox <gi...@apache.org>]

KKcorps edited a comment on issue #7270:
URL: https://github.com/apache/pinot/issues/7270#issuecomment-998785977


   Yeah, I think the string offsets are borrowed from the Kafka implementation. We might need to make this more generic. That's also another reason I don't want to split the message id on `:` and create from there as it can change on pulsar side.
   IMO, what the solution should be here is to store in Hex string format and then parse hex string -> byte array -> message id. 


-- 
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: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org