Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HHH-14055 - Add UUID support for fields annotated with @RevisionNumber #3425

Open
wants to merge 1 commit into
base: master
from

Conversation

@bbanchev
Copy link

bbanchev commented Jun 2, 2020

Add fallback check if the field type is of type java.util.UUID
Update corresponding MappingException message

Add fallback check if the field type is of type java.util.UUID
Update corresponding MappingException message
@sebersole sebersole requested a review from Naros Jun 2, 2020
@sebersole
Copy link
Member

sebersole commented Jun 2, 2020

@RevisionNumber explicitly says that "values of this property should form a strictly-increasing sequence of numbers". A UUID obviously does not fit that description.

@Naros you are much better versed in envers than I... what do you think?

@bbanchev
Copy link
Author

bbanchev commented Jun 2, 2020

@RevisionNumber explicitly says that "values of this property should form a strictly-increasing sequence of numbers". A UUID obviously does not fit that description.

@Naros you are much better versed in envers than I... what do you think?

Actually this is one of the main reasons to ask for this feature - need unpredictable id sequence for revisions. On the other hand fits better in distributed environment.

@sebersole
Copy link
Member

sebersole commented Jun 2, 2020

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.