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.

Sign up for GitHub

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

Jump to bottom

Replace BusinessKey with declared natural IDs #1239

Open
arteymix opened this issue Oct 3, 2024 · 1 comment
Open

Replace BusinessKey with declared natural IDs #1239

arteymix opened this issue Oct 3, 2024 · 1 comment
Labels
cleanup Code reorganization and changes that should not impact functionality enhancement Enhance the code or user experience low priority

Comments

@arteymix
Copy link
Member

arteymix commented Oct 3, 2024

We centralize the definition of "business keys" in BusinessKey. This can in most cases be replaced by a native Hibernate natural ID.

I think it would also allow us to have natural-ID-based lazy proxies. For example, an ExpressionExperiment lazy proxy based on its shortName attribute.

This is related to #809.
@arteymix arteymix added enhancement Enhance the code or user experience cleanup Code reorganization and changes that should not impact functionality low priority labels Oct 3, 2024
@arteymix
Copy link
Member Author

Also, if it does not fit with a natural key definition, we should consider relocating the logic in the specific DAO.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
cleanup Code reorganization and changes that should not impact functionality enhancement Enhance the code or user experience low priority
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@arteymix