eclipse-uprotocol / up-java

uProtocol Language Specific Library for Java
Apache License 2.0
9 stars 14 forks source link

Possible issue with UUID Deserializer #174

Closed matthewd0123 closed 4 months ago

matthewd0123 commented 4 months ago

Wanted to bring up a possible problem with the uuid deserializer in up-java, according to a up-tck test. Attaching the results of both python and java's results for this uuid_deserializer test, and up-java is failing. Something is happening with the deserializer where the msb of the uuid is being set differently in java than in python. Can someone with more knowledge of this method in java take a look and see if it's an issue? Screenshot 2024-07-31 112052 Screenshot 2024-07-31 112038

stevenhartley commented 4 months ago

It looks like the printed number for lsb is greater than 2^63-1 so it cannot fit in a long that has a range of -2^63 to 2^63-1 and that is why you get the negative number from up-java that uses a long.

matthewd0123 commented 4 months ago

Looks like root cause was discovered, closing for now