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

[Feature/extensions] Updated imports for xcontent #817

Conversation

owaiskazi19
Copy link
Member

@owaiskazi19 owaiskazi19 commented Feb 22, 2023

Description

This PR updates xcontent imports changed by opensearch-project/OpenSearch#5902

Issues Resolved

List any issues this PR will resolve, e.g. Closes [...].

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.

Signed-off-by: owaiskazi19 <owaiskazi19@gmail.com>
@owaiskazi19 owaiskazi19 changed the title Updated imports for xcontent [Feature/extensions] Updated imports for xcontent Feb 22, 2023
Signed-off-by: owaiskazi19 <owaiskazi19@gmail.com>
Signed-off-by: owaiskazi19 <owaiskazi19@gmail.com>
@owaiskazi19 owaiskazi19 mentioned this pull request Feb 22, 2023
Signed-off-by: owaiskazi19 <owaiskazi19@gmail.com>
Signed-off-by: owaiskazi19 <owaiskazi19@gmail.com>
Signed-off-by: owaiskazi19 <owaiskazi19@gmail.com>
@owaiskazi19 owaiskazi19 marked this pull request as ready for review February 24, 2023 17:36
@owaiskazi19 owaiskazi19 requested a review from a team February 24, 2023 17:36
@codecov-commenter
Copy link

Codecov Report

Merging #817 (1a37be9) into feature/extensions (ce6b955) will decrease coverage by 3.57%.
The diff coverage is n/a.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

Impacted file tree graph

@@                   Coverage Diff                    @@
##             feature/extensions     #817      +/-   ##
========================================================
- Coverage                 49.17%   45.60%   -3.57%     
+ Complexity                 2630     2458     -172     
========================================================
  Files                       297      297              
  Lines                     17115    17115              
  Branches                   1843     1843              
========================================================
- Hits                       8416     7806     -610     
- Misses                     8089     8739     +650     
+ Partials                    610      570      -40     
Flag Coverage Δ
plugin 45.60% <ø> (-3.57%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...va/org/opensearch/ad/AnomalyDetectorExtension.java 0.00% <ø> (ø)
...va/org/opensearch/ad/AnomalyDetectorJobRunner.java 0.00% <ø> (ø)
.../java/org/opensearch/ad/AnomalyDetectorPlugin.java 86.56% <ø> (ø)
...rg/opensearch/ad/AnomalyDetectorProfileRunner.java 7.42% <ø> (ø)
...in/java/org/opensearch/ad/EntityProfileRunner.java 4.00% <ø> (ø)
.../main/java/org/opensearch/ad/NodeStateManager.java 63.28% <ø> (ø)
...main/java/org/opensearch/ad/auth/UserIdentity.java 54.78% <ø> (ø)
.../org/opensearch/ad/feature/CompositeRetriever.java 0.00% <ø> (ø)
...va/org/opensearch/ad/feature/SearchFeatureDao.java 6.45% <ø> (ø)
...opensearch/ad/indices/AnomalyDetectionIndices.java 11.76% <ø> (ø)
... and 106 more

'org.opensearch.ad.transport.AnomalyResultAction',
'org.opensearch.ad.transport.CronNodeResponse',
'org.opensearch.ad.transport.CronResponse',
'org.opensearch.ad.transport.AnomalyResultResponse'
Copy link
Member

Choose a reason for hiding this comment

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

Are all these exclusions part of this PR or just left over from a merge commit (i.e., fixed by rebase)?

Copy link
Member Author

Choose a reason for hiding this comment

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

They are the part of this PR. Updating imports caused failure for few test cases, the functionality of which isn't implemented anyway.

Copy link
Member

@dbwiddis dbwiddis left a comment

Choose a reason for hiding this comment

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

LGTM, a few stray questions that look like they're part of a merge commit rather than a rebase commit. If that's the case, go ahead and merge!

@dbwiddis dbwiddis merged commit 5c2c65e into opensearch-project:feature/extensions Feb 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants