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

[Backport] [2.x] Fix a bug on handling an invalid array value for point type field (#4900) #4917

Merged
merged 1 commit into from
Oct 25, 2022

Conversation

heemin32
Copy link
Contributor

@heemin32 heemin32 commented Oct 25, 2022

With this commit, appropriate exception is thrown when an array of four values are provided for point type field.

Signed-off-by: Heemin Kim heemin@amazon.com
(cherry picked from commit 0f477a2)

Description

Backport of #4900

Issues Resolved

See #4900

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Commits are signed per the DCO using --signoff
  • Commit changes are listed out in CHANGELOG.md file (See: Changelog)

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.

@heemin32 heemin32 requested review from a team and reta as code owners October 25, 2022 17:13
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@heemin32 heemin32 closed this Oct 25, 2022
@heemin32 heemin32 reopened this Oct 25, 2022
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@heemin32 heemin32 changed the title Fix a bug on handling an invalid array value for point type field (#4900) [Backport] [2.x] Fix a bug on handling an invalid array value for point type field (#4900) Oct 25, 2022
@heemin32 heemin32 force-pushed the backport/backport-4900-to-2.x branch from acbf17e to 749f4c7 Compare October 25, 2022 18:22
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@heemin32
Copy link
Contributor Author

[Test Result](https://build.ci.opensearch.org/job/gradle-check/5576/testReport/) (1 failure / +1)
[org.opensearch.search.PitMultiNodeTests.testCreatePitWhileNodeDropWithAllowPartialCreationFalse](https://build.ci.opensearch.org/job/gradle-check/5576/testReport/junit/org.opensearch.search/PitMultiNodeTests/testCreatePitWhileNodeDropWithAllowPartialCreationFalse/)

#4259

…ensearch-project#4900)

With this commit, appropriate exception is thrown when an array of four values are provided for point type field.

Signed-off-by: Heemin Kim <heemin@amazon.com>
(cherry picked from commit 0f477a2)
@heemin32 heemin32 force-pushed the backport/backport-4900-to-2.x branch from 749f4c7 to c07a4ab Compare October 25, 2022 19:49
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@reta reta merged commit 7416b5e into opensearch-project:2.x Oct 25, 2022
@heemin32 heemin32 deleted the backport/backport-4900-to-2.x branch October 25, 2022 21:48
Bukhtawar pushed a commit that referenced this pull request Nov 3, 2022
* Weighted round-robin scheduling policy for shard coordination traffic… (#4241)

* Add PUT api to update shard routing weights (#4272)

* Add GET api to get shard routing weights (#4275)

* Fix weighted routing metadata deserialization error during node restart (#4691)

* Delete API for weighted round robin search routing (#4917 

* Mark apis experimental

Signed-off-by: Anshu Agarwal <anshukag@amazon.com>
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