-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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 unnecessary use of Long.toString from BlobStoreRepository. #5833
Conversation
Signed-off-by: Mani <singh.mani1231@gmail.com>
Gradle Check (Jenkins) Run Completed with:
|
Gradle Check (Jenkins) Run Completed with:
|
Codecov Report
@@ Coverage Diff @@
## main #5833 +/- ##
============================================
+ Coverage 70.81% 70.90% +0.08%
- Complexity 58686 58760 +74
============================================
Files 4768 4768
Lines 280575 280575
Branches 40514 40514
============================================
+ Hits 198685 198935 +250
+ Misses 65636 65349 -287
- Partials 16254 16291 +37
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
@@ -1820,7 +1820,7 @@ private RepositoryData getRepositoryData(long indexGen) { | |||
return RepositoryData.EMPTY; | |||
} | |||
try { | |||
final String snapshotsIndexBlobName = INDEX_FILE_PREFIX + Long.toString(indexGen); | |||
final String snapshotsIndexBlobName = INDEX_FILE_PREFIX + indexGen; |
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.
I'm curious whether underneath it generates the same code? Do you know?
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.
Hi @dblock , most probably, yes, I've tested that out.
And thanks for merging this.
…pensearch-project#5833) Signed-off-by: Mani <singh.mani1231@gmail.com>
…5833) (#5911) Signed-off-by: Mani <singh.mani1231@gmail.com> (cherry picked from commit db30439) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Signed-off-by: Mani <singh.mani1231@gmail.com> 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>
…5833) (#5911) Signed-off-by: Mani <singh.mani1231@gmail.com> (cherry picked from commit db30439) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Signed-off-by: Mani <singh.mani1231@gmail.com> 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>
Signed-off-by: Mani singh.mani1231@gmail.com
Description
Removed unnecessary use of Long.toString from BlobStoreRepository.
Check List
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.