Skip to content

Fix Config Validation#16164

Merged
arhimondr merged 1 commit intoprestodb:masterfrom
darrenfu:fix-config-validation
Jun 1, 2021
Merged

Fix Config Validation#16164
arhimondr merged 1 commit intoprestodb:masterfrom
darrenfu:fix-config-validation

Conversation

@darrenfu
Copy link
Contributor

== RELEASE NOTES ==

General Changes
* Fix config validation. Based on JSR 303, validation should be applied on getter methods only. This fix will move all setter-side javax validation annotations to getter methods. This applies to all three Config Bean files.

Reference:

Similar PR as the one applied to 0.228: #16162

@darrenfu darrenfu changed the title Fix Config Validation issue Fix Config Validation May 26, 2021
@darrenfu
Copy link
Contributor Author

darrenfu commented Jun 1, 2021

Thanks for approval @arhimondr
I would like to see this PR included in release 0.255.

@arhimondr arhimondr merged commit 02bb379 into prestodb:master Jun 1, 2021
@jainxrohit jainxrohit mentioned this pull request Jun 5, 2021
4 tasks
@ajaygeorge ajaygeorge mentioned this pull request Jun 9, 2021
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants