Skip to content

THRIFT-5925: Java UUID fix#3334

Open
CJCombrink wants to merge 3 commits intoapache:masterfrom
CJCombrink:java_uuid_fix
Open

THRIFT-5925: Java UUID fix#3334
CJCombrink wants to merge 3 commits intoapache:masterfrom
CJCombrink:java_uuid_fix

Conversation

@CJCombrink
Copy link
Contributor

Fix the UUID implementation in Java to conform to the specification

  • Added a protocol decorator for user/use cases where migration/adoption is not possible

This is a breaking change for those who already use UUID type in Java.

  • Did you create an Apache Jira ticket? THRIFT-5925: UUID implementation in JAVA is not according to the Thrift Specification
  • If a ticket exists: Does your pull request title follow the pattern "THRIFT-NNNN: describe my issue"?
  • Did you squash your changes to a single commit? (not required, but preferred)
  • Did you do your best to avoid breaking changes? If one was needed, did you label the Jira ticket with "Breaking-Change"?
  • If your change does not involve any code, include [skip ci] anywhere in the commit message to free up build resources.

…ecification

- Don't do byte swapping when writing to and reading from the wire.
- This is a breaking change for anyone using UUID between Java servers
- For users that depends on this behaviour a TProtocolDecorator can be used to get back the old behaviour
…dy in Java

- For those who can't migrate to new UUID format
- Also captures the code that was used before
@mergeable mergeable bot added the java Pull requests that update Java code label Mar 10, 2026
- Using gradle spotlessApply
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

java Pull requests that update Java code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant