-
Notifications
You must be signed in to change notification settings - Fork 191
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
Removed null constraints on NodeInfo fields that are returned null from Amazon OpenSearch Service #1132
Removed null constraints on NodeInfo fields that are returned null from Amazon OpenSearch Service #1132
Conversation
…TransportAddress Signed-off-by: Dan Cecoi <[email protected]>
…white space Signed-off-by: Dan Cecoi <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A minor comment on the CHANGELOG entry wording, and just needs a rebase/merge to fix some conflicts
Signed-off-by: Dan Cecoi <[email protected]> # Conflicts: # CHANGELOG.md
CHANGELOG.md
Outdated
@@ -42,6 +42,7 @@ This section is for maintaining a changelog for all breaking changes for the cli | |||
### Removed | |||
|
|||
### Fixed | |||
- Fixed handling of NodeInfo fields that are not returned by Amazon OpenSearch Service ([#1131](https://github.com/opensearch-project/opensearch-java/pull/1131)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry I missed catching this before, it should reference this PR rather than the previous one
- Fixed handling of NodeInfo fields that are not returned by Amazon OpenSearch Service ([#1131](https://github.com/opensearch-project/opensearch-java/pull/1131)) | |
- Fixed handling of NodeInfo fields that are not returned by Amazon OpenSearch Service ([#1132](https://github.com/opensearch-project/opensearch-java/pull/1132)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops, good catch! I made the changes.
Signed-off-by: Dan Cecoi <[email protected]>
…om Amazon OpenSearch Service (#1132) * Removed the null constraints on NodeInfo.host, NodeInfo.Ip, NodeInfo.TransportAddress Signed-off-by: Dan Cecoi <[email protected]> * Added a CHANGELOG entry, made the getters nullable and removed extra white space Signed-off-by: Dan Cecoi <[email protected]> * modified the PR link in the changelog Signed-off-by: Dan Cecoi <[email protected]> --------- Signed-off-by: Dan Cecoi <[email protected]> Co-authored-by: Dan Cecoi <[email protected]> (cherry picked from commit 94171f8) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
…om Amazon OpenSearch Service (#1132) (#1136) * Removed the null constraints on NodeInfo.host, NodeInfo.Ip, NodeInfo.TransportAddress * Added a CHANGELOG entry, made the getters nullable and removed extra white space * modified the PR link in the changelog --------- (cherry picked from commit 94171f8) Signed-off-by: Dan Cecoi <[email protected]> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: Dan Cecoi <[email protected]>
Description
This change removes null constraints on NodeInfo.Host, NodeInfo.Ip, NodeInfo.TransportAddress that are returned null from Amazon OpenSearch Service when performing a NodeInfoRequest
Issues Resolved
#1129
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.