spetz commented on code in PR #4076:
URL: https://github.com/apache/iggy/pull/4076#discussion_r3947155773
##########
foreign/java/java-sdk/src/main/java/org/apache/iggy/serde/BytesSerializer.java:
##########
@@ -340,7 +339,9 @@ private static long batchChecksum(
*/
private static byte[] encodedMessageId(MessageId id) {
if (id.toBigInteger().signum() == 0) {
- return readAllBytes(new
UuidMessageId(UUID.randomUUID()).toBytes());
+ byte[] minted = new byte[16];
+ ThreadLocalRandom.current().nextBytes(minted);
Review Comment:
@richardcocks sounds good. I think the other SDKs are good to go, so we
probably don't need to change these, and regarding the Java one, if we could
have it done today, that'd be great, as we'd like to make it a part of the
voting release.
--
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]
