OpenSearch icon indicating copy to clipboard operation
OpenSearch copied to clipboard

Remove @ExperimentalApi annotations from star-tree related code

Open happysubin opened this issue 6 months ago • 2 comments

Description

Removal of ExperimentalApi annotations in star-tree related code files

Related Issues

Resolves #18499

Check List

  • [ ] Functionality includes testing.
  • [ ] API changes companion pull request created, if applicable.
  • [ ] Public documentation issue/PR created, if applicable.

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.

happysubin avatar Jun 14 '25 06:06 happysubin

:white_check_mark: Gradle check result for 1fcabfc68908cef4cd648f9d279e86cef7a5779e: SUCCESS

github-actions[bot] avatar Jun 14 '25 07:06 github-actions[bot]

Codecov Report

:white_check_mark: All modified and coverable lines are covered by tests. :white_check_mark: Project coverage is 72.72%. Comparing base (3078649) to head (1fcabfc). :warning: Report is 411 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main   #18521      +/-   ##
============================================
+ Coverage     72.64%   72.72%   +0.08%     
- Complexity    68070    68111      +41     
============================================
  Files          5537     5537              
  Lines        313404   313381      -23     
  Branches      45476    45472       -4     
============================================
+ Hits         227658   227915     +257     
+ Misses        67241    66900     -341     
- Partials      18505    18566      +61     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

codecov[bot] avatar Jun 14 '25 07:06 codecov[bot]

@sandeshkr419 When you have a moment, could you please take a look at this PR? I'd really appreciate your feedback !!

happysubin avatar Jun 17 '25 14:06 happysubin

Thanks a lot for contributing to this PR @happysubin . We also can remove experimentalApi from star tree indexing side of code - https://github.com/opensearch-project/OpenSearch/tree/main/server/src/main/java/org/opensearch/index/compositeindex

bharath-techie avatar Jun 23 '25 06:06 bharath-techie

This PR is stalled because it has been open for 30 days with no activity.

This PR is stalled because it has been open for 30 days with no activity.

This PR is stalled because it has been open for 30 days with no activity.