-
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
Fix integer overflow for remaining index stats #960
Conversation
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.
Look at the failure log, it fails on :java-client:spotlessJavaCheck
. It's just a formatting problem, to fix run ./gradlew :java-client:spotlessApply
.
Signed-off-by: oksanay <oksana.yakusheva@hpe.com>
Signed-off-by: oksanay <oksana.yakusheva@hpe.com>
Signed-off-by: oksanay <oksana.yakusheva@hpe.com>
Signed-off-by: oksanay <oksana.yakusheva@hpe.com>
Signed-off-by: oksanay <oksana.yakusheva@hpe.com>
Signed-off-by: oksanay <oksana.yakusheva@hpe.com>
Signed-off-by: oksanay <oksana.yakusheva@hpe.com>
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.
Looks good!
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.
CHANGELOG needs fixing.
CHANGELOG.md
Outdated
@@ -26,7 +26,10 @@ This section is for maintaining a changelog for all breaking changes for the cli | |||
- Delete shape property ([#884](https://github.com/opensearch-project/opensearch-java/pull/885)) | |||
|
|||
### Fixed | |||
- Fix version and build ([#254](https://github.com/opensearch-project/opensearch-java/pull/254)) | |||
- Fix version and build ([#254](https://github.com/opensearch-project/opensearcjava/pull/254)) |
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.
There's a typo in opensearch-java here, so the link checker caught it and that's why the build failed.
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.
This link is still incorrect, #254 is the correct one.
java-client/src/test/java/org/opensearch/client/opensearch/_types/QueryCacheStatsTest.java
Outdated
Show resolved
Hide resolved
java-client/src/test/java/org/opensearch/client/opensearch/_types/QueryCacheStatsTest.java
Show resolved
Hide resolved
Signed-off-by: oksanay <oksana.yakusheva@hpe.com>
Signed-off-by: oksanay <oksana.yakusheva@hpe.com>
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.
One more.
Signed-off-by: oksanay <oksana.yakusheva@hpe.com>
@dblock I pushed change to test-unit.yml |
@dblock now all checks passed |
The backport to
To backport manually, run these commands in your terminal: # Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add ../.worktrees/backport-2.x 2.x
# Navigate to the new working tree
pushd ../.worktrees/backport-2.x
# Create a new branch
git switch --create backport/backport-960-to-2.x
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x --mainline 1 f0797da1ac1715884d7f105b6507012a53a9b374
# Push it to GitHub
git push --set-upstream origin backport/backport-960-to-2.x
# Go back to the original working tree
popd
# Delete the working tree
git worktree remove ../.worktrees/backport-2.x Then, create a pull request where the |
@oksanay Looks like this will need a manual backport to the 2.x branch, please? I believe we release from that (@VachaShah can correct me if I am wrong). |
Thats right @dblock! We release a new minor or patch version from the 2.x branch. |
@dblock and @VachaShah so do I need to open the issue for the release or not? |
and do I need to run this manual backport stuff? |
@oksanay Yes. We have an automated workflow that works most of the time, but it failed here because it caused a conflict that needs to be manually resolved. |
Yes, please. You have to do both the backport and open a release ticket if you need a release faster than "whenever". |
It should be 2.11.1, right? |
@dblock @VachaShah ^ ? |
This is a bug fix on 2.10.1, so it should be 2.10.2. We will also need to move the CHANGELOG entry accordingly when we release. |
@dblock This command is failing with any combination of 2,10 |
@oksanay The easiest way is to just re-apply your commit to the 2.x branch. In your fork:
If you're struggling I can do it quickly. |
@dblock YES, please! I have a deadline today with my own stuff and I AM struggling |
Signed-off-by: dblock <dblock@amazon.com>
I opened a release proposal ticket in #966. |
Thank you so much!!!! |
Description
Changed data types in QueryCacheStats object to be longs
Example of the data which is having problem when datatypes are integers
"query_cache" : {
"memory_size_in_bytes" : 892496036,
"total_count" : 8757938874,
"hit_count" : 347443703,
"miss_count" : 8410495171,
"cache_size" : 18507,
"cache_count" : 169300,
"evictions" : 150793
}
Issues Resolved
#959
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.