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] Add experimental SIMD implementation of B-tree to round down dates #11924

Merged
merged 1 commit into from
Jan 18, 2024

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport b424aaf from #11194.

Copy link
Contributor

github-actions bot commented Jan 18, 2024

Compatibility status:

Checks if related components are compatible with change 86f26af

Incompatible components

Incompatible components: [https://github.com/opensearch-project/asynchronous-search.git, https://github.com/opensearch-project/observability.git, https://github.com/opensearch-project/performance-analyzer.git, https://github.com/opensearch-project/reporting.git, https://github.com/opensearch-project/performance-analyzer-rca.git, https://github.com/opensearch-project/custom-codecs.git, https://github.com/opensearch-project/anomaly-detection.git, https://github.com/opensearch-project/sql.git, https://github.com/opensearch-project/cross-cluster-replication.git]

Skipped components

Compatible components

Compatible components: [https://github.com/opensearch-project/security-analytics.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/k-nn.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/ml-commons.git, https://github.com/opensearch-project/neural-search.git, https://github.com/opensearch-project/security.git, https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/index-management.git, https://github.com/opensearch-project/alerting.git]

Copy link
Contributor

❌ Gradle check result for cbcbc0e: 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?

…11194)

* Add experimental SIMD implementation of B-tree to round down dates

Signed-off-by: Ketan Verma <ketan9495@gmail.com>

* Use system properties in favor of feature flags to remove dependency on the server module

Signed-off-by: Ketan Verma <ketan9495@gmail.com>

* Removed Java 20 test sources to simplify builds

Signed-off-by: Ketan Verma <ketan9495@gmail.com>

* Remove the use of forbidden APIs in unit-tests

Signed-off-by: Ketan Verma <ketan9495@gmail.com>

* Migrate to the recommended usage for custom test task classpath

Signed-off-by: Ketan Verma <ketan9495@gmail.com>

* Switch benchmarks module to multi-release one

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>

* Add JMH annotation processing for JDK-20+ sources

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>

* Make JMH annotations consistent across sources

Signed-off-by: Ketan Verma <ketan9495@gmail.com>

* Improve execution of Roundable unit-tests

Signed-off-by: Ketan Verma <ketan9495@gmail.com>

* Revert "Improve execution of Roundable unit-tests"

This reverts commit 2e82d0a.

Signed-off-by: Ketan Verma <ketan9495@gmail.com>

* Add 'forced' as a possible feature flag value to simplify the execution of unit-tests

Signed-off-by: Ketan Verma <ketan9495@gmail.com>

---------

Signed-off-by: Ketan Verma <ketan9495@gmail.com>
Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
Co-authored-by: Andriy Redko <andriy.redko@aiven.io>
Co-authored-by: Andrew Ross <andrross@amazon.com>
(cherry picked from commit b424aaf)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@reta reta force-pushed the backport/backport-11194-to-2.x branch from cbcbc0e to 86f26af Compare January 18, 2024 19:44
Copy link

codecov bot commented Jan 18, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (6e9e9a4) 71.07% compared to head (6e9e9a4) 71.07%.

❗ Current head 6e9e9a4 differs from pull request most recent head 86f26af. Consider uploading reports for the commit 86f26af to get more accurate results

Additional details and impacted files
@@            Coverage Diff            @@
##                2.x   #11924   +/-   ##
=========================================
  Coverage     71.07%   71.07%           
  Complexity    59321    59321           
=========================================
  Files          4891     4891           
  Lines        279915   279915           
  Branches      41050    41050           
=========================================
  Hits         198937   198937           
  Misses        64091    64091           
  Partials      16887    16887           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

❕ Gradle check result for 86f26af: UNSTABLE

  • TEST FAILURES:
      1 org.opensearch.remotestore.RemoteStoreStatsIT.testDownloadStatsCorrectnessSinglePrimaryMultipleReplicaShards
      1 org.opensearch.index.shard.RemoteStoreRefreshListenerTests.testRefreshAfterCommit
      1 org.opensearch.index.shard.RemoteStoreRefreshListenerTests.testAfterRefresh
      1 org.opensearch.index.shard.RemoteStoreRefreshListenerTests.testAfterMultipleCommits
      1 org.opensearch.index.shard.RemoteStoreRefreshListenerTests.testAfterCommit

Please review all flaky tests that succeeded after retry and create an issue if one does not already exist to track the flaky failure.

@reta reta merged commit 4c331b5 into 2.x Jan 18, 2024
27 checks passed
@reta reta added the v2.12.0 Issues and PRs related to version 2.12.0 label Jan 18, 2024
@github-actions github-actions bot deleted the backport/backport-11194-to-2.x branch January 18, 2024 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
v2.12.0 Issues and PRs related to version 2.12.0
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

3 participants