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

Removed unneeded calls of String.format() #15385

Closed
wants to merge 297 commits into from

Conversation

dk2k
Copy link
Contributor

@dk2k dk2k commented Aug 23, 2024

Removed unneeded calls of String.format()

Copy link
Contributor

❌ Gradle check result for 8ca0e7b: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@dblock dblock added backport 2.x Backport to 2.x branch skip-changelog labels Aug 23, 2024
Copy link
Contributor

❌ Gradle check result for 8ca0e7b: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

Copy link
Member

@dblock dblock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like a legit failure.


* What went wrong:
Execution failed for task ':server:spotlessJavaCheck'.
> The following files had format violations:
      src/main/java/org/opensearch/index/compositeindex/CompositeIndexValidator.java
          @@ -13,8 +13,6 @@
           import·org.opensearch.index.compositeindex.datacube.startree.StarTreeValidator;
           import·org.opensearch.index.mapper.MapperService;
           
          -import·java.util.Locale;
          -
           /**
           ·*·Validation·for·composite·indices·as·part·of·mappings
           ·*
  Run './gradlew :server:spotlessApply' to fix these violations.

imRishN and others added 22 commits October 16, 2024 22:41
…search-project#16296)

* Optimise clone operation for incremental full cluster snapshots

Signed-off-by: Ashish Singh <[email protected]>

* Add UTs

Signed-off-by: Ashish Singh <[email protected]>

* Add CHANGELOG

Signed-off-by: Ashish Singh <[email protected]>

---------

Signed-off-by: Ashish Singh <[email protected]>
…-project#16265)

* Fix multi-search with template doesn't return status code

Signed-off-by: Gao Binlong <[email protected]>

* Modify changelog

Signed-off-by: Gao Binlong <[email protected]>

---------

Signed-off-by: Gao Binlong <[email protected]>
…nsearch-project#16269)

* Move timestamp pinning before cluster state update

Signed-off-by: Gaurav Bafna <[email protected]>

* Address PR Comments

Signed-off-by: Gaurav Bafna <[email protected]>

* Fix IT

Signed-off-by: Gaurav Bafna <[email protected]>

---------

Signed-off-by: Gaurav Bafna <[email protected]>
…es/hdfs-fixture (opensearch-project#16307)

* Bump ch.qos.logback:logback-core in /test/fixtures/hdfs-fixture

Bumps [ch.qos.logback:logback-core](https://github.com/qos-ch/logback) from 1.5.8 to 1.5.10.
- [Commits](qos-ch/logback@v_1.5.8...v_1.5.10)

---
updated-dependencies:
- dependency-name: ch.qos.logback:logback-core
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* Update changelog

Signed-off-by: dependabot[bot] <[email protected]>

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com>
…earch-project#16309)

* Bump me.champeau.gradle.japicmp from 0.4.3 to 0.4.4 in /server

Bumps me.champeau.gradle.japicmp from 0.4.3 to 0.4.4.

---
updated-dependencies:
- dependency-name: me.champeau.gradle.japicmp
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* Update changelog

Signed-off-by: dependabot[bot] <[email protected]>

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com>
…0 in /plugins/discovery-gce (opensearch-project#16306)

* Bump com.google.oauth-client:google-oauth-client

Bumps [com.google.oauth-client:google-oauth-client](https://github.com/googleapis/google-oauth-java-client) from 1.35.0 to 1.36.0.
- [Release notes](https://github.com/googleapis/google-oauth-java-client/releases)
- [Changelog](https://github.com/googleapis/google-oauth-java-client/blob/main/CHANGELOG.md)
- [Commits](googleapis/google-oauth-java-client@v1.35.0...v1.36.0)

---
updated-dependencies:
- dependency-name: com.google.oauth-client:google-oauth-client
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

* Updating SHAs

Signed-off-by: dependabot[bot] <[email protected]>

* Update changelog

Signed-off-by: dependabot[bot] <[email protected]>

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com>
…ct#16310)

* Bump lycheeverse/lychee-action from 1.10.0 to 2.0.2

Bumps [lycheeverse/lychee-action](https://github.com/lycheeverse/lychee-action) from 1.10.0 to 2.0.2.
- [Release notes](https://github.com/lycheeverse/lychee-action/releases)
- [Commits](lycheeverse/lychee-action@v1.10.0...v2.0.2)

---
updated-dependencies:
- dependency-name: lycheeverse/lychee-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com>
…s/repository-azure (opensearch-project#16311)

* Bump com.azure:azure-core-http-netty in /plugins/repository-azure

Bumps [com.azure:azure-core-http-netty](https://github.com/Azure/azure-sdk-for-java) from 1.15.4 to 1.15.5.
- [Release notes](https://github.com/Azure/azure-sdk-for-java/releases)
- [Commits](Azure/azure-sdk-for-java@azure-core-http-netty_1.15.4...azure-core-http-netty_1.15.5)

---
updated-dependencies:
- dependency-name: com.azure:azure-core-http-netty
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* Updating SHAs

Signed-off-by: dependabot[bot] <[email protected]>

* Update changelog

Signed-off-by: dependabot[bot] <[email protected]>

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com>
…sitory-gcs (opensearch-project#16308)

* Bump com.google.code.gson:gson in /plugins/repository-gcs

Bumps [com.google.code.gson:gson](https://github.com/google/gson) from 2.10.1 to 2.11.0.
- [Release notes](https://github.com/google/gson/releases)
- [Changelog](https://github.com/google/gson/blob/main/CHANGELOG.md)
- [Commits](google/gson@gson-parent-2.10.1...gson-parent-2.11.0)

---
updated-dependencies:
- dependency-name: com.google.code.gson:gson
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

* Updating SHAs

Signed-off-by: dependabot[bot] <[email protected]>

* Update changelog

Signed-off-by: dependabot[bot] <[email protected]>

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com>
…16323)

We have guarded the experimental query approximation framework behind
a feature flag. In order to easily measure the impact of approximation
on big5 benchmarks, it would be nice to have a benchmark config.

Signed-off-by: Michael Froh <[email protected]>
)

* Update last seen cluster state on apply commit

Signed-off-by: Sooraj Sinha <[email protected]>
…o null on index creation (opensearch-project#16331)

* Fix wrong value when setting index.number_of_routing_shards to null on index creation

Signed-off-by: Gao Binlong <[email protected]>

* Modify change log

Signed-off-by: Gao Binlong <[email protected]>

---------

Signed-off-by: Gao Binlong <[email protected]>
…ld (opensearch-project#15258)

* fix cluster not able to spin up issue when disk usage exceeds threshold

Signed-off-by: zane-neo <[email protected]>

* Add comment to changes

Signed-off-by: zane-neo <[email protected]>

* Add UT to ensure the keepAliveThread starts before node starts

Signed-off-by: zane-neo <[email protected]>

* remove unused imports

Signed-off-by: zane-neo <[email protected]>

* Fix forbidden API calls check failed issue

Signed-off-by: zane-neo <[email protected]>

* format code

Signed-off-by: zane-neo <[email protected]>

* format code

Signed-off-by: zane-neo <[email protected]>

* change setInstance method to static

Signed-off-by: zane-neo <[email protected]>

* Add countdownlatch in test to coordinate the thread to avoid concureency issue caused test failure

Signed-off-by: zane-neo <[email protected]>

---------

Signed-off-by: zane-neo <[email protected]>
…nated by a newline [\n]' failures (opensearch-project#16337)

* [Streaming Indexing] Fix intermittent 'The bulk request must be terminated by a newline [\n]' failures

Signed-off-by: Andriy Redko <[email protected]>

* Address code review comments

Signed-off-by: Andriy Redko <[email protected]>

---------

Signed-off-by: Andriy Redko <[email protected]>
@dk2k dk2k requested a review from peternied as a code owner October 16, 2024 19:42
@dk2k
Copy link
Contributor Author

dk2k commented Oct 16, 2024

Guys, sorry, rebase didn't go smoothly. I will close this PR and create a new one.
I really think you need to adjust the processes in this repo and make it contributor-friendly. Currently it's a pain in the neck.

@dk2k dk2k closed this Oct 16, 2024
Copy link
Contributor

❌ Gradle check result for 351d3a3: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@sandeshkr419
Copy link
Contributor

@dk2k Yeah, rebasing has been a major pain. Do you have any suggestions that we can look into?

Also, here is the usual process I do to rebasing smoothly:

  1. git remote set-url origin https://github.com/opensearch-project/Opensearch.git
  2. git checkout main
  3. git pull
  4. git checkout
  5. git rebase main
  6. git remote set-url origin
  7. git push origin

Probably I can include these steps in development guide itself.

@dbwiddis
Copy link
Member

@dk2k Yeah, rebasing has been a major pain. Do you have any suggestions that we can look into?

I'm not dk2k, but on other repos I maintain there is a GitHub setting that enables rebasing from the PR itself.
Screenshot 2024-10-16 at 4 45 14 PM

It creates this dialog:
Screenshot 2024-10-16 at 4 47 02 PM

@dbwiddis
Copy link
Member

@dk2k i messed up my first GitHub PR and had to close and resubmit. You're in good company. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 2.x Backport to 2.x branch skip-changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.