ankitsultana commented on PR #12538: URL: https://github.com/apache/pinot/pull/12538#issuecomment-2029083877
@chenboat @satishd : I was thinking about this today and I think appending a null-byte is not the right approach, since it can lead to collisions (albeit on sub-optimally defined primary-keys). I have instead resorted to prepending the length of the byte-array for each element in the `concatenate` method. Note that this gets called only on invalid UUID value scenarios, which should be rare. -- 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-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
