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

Update the AbstractScopedSettings class to throw OpenSearchExceptions instead of IllegalArgumentException #4745

Closed
xuezhou25 opened this issue Oct 12, 2022 · 0 comments · Fixed by #4792
Closed

Update the AbstractScopedSettings class to throw OpenSearchExceptions instead of IllegalArgumentException #4745

xuezhou25 opened this issue Oct 12, 2022 · 0 comments · Fixed by #4792
Assignees
@xuezhou25
Labels
enhancement Enhancement or improvement to existing feature or request

Comments

@xuezhou25
Copy link
Contributor

xuezhou25 commented Oct 12, 2022 •
edited
Loading

Is your feature request related to a problem? Please describe.
Update the AbstractScopedSettings class to throw OpenSearchException instead of IllegalArgumentException so that the helpful error messages can be reported in all cases.

Describe the solution you'd like
Mapping IllegalArgumentException to OpenSearchException in AbstractScopedSettings class

Additional context
Coming from issue #4545
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@xuezhou25 xuezhou25

Labels
enhancement Enhancement or improvement to existing feature or request
Projects
None yet
Milestone
No milestone
1 participant
@xuezhou25