Skip to content

[backport 2.x] Fixed compile issues related to replacing ImmutableOpe…#477

Merged
sbcd90 merged 1 commit intoopensearch-project:2.xfrom
sbcd90:2.x_fix
Jul 11, 2023
Merged

[backport 2.x] Fixed compile issues related to replacing ImmutableOpe…#477
sbcd90 merged 1 commit intoopensearch-project:2.xfrom
sbcd90:2.x_fix

Conversation

@sbcd90
Copy link
Copy Markdown
Collaborator

@sbcd90 sbcd90 commented Jul 11, 2023

…nMap with java.util.Map

Description

backport of pr #412

Issues Resolved

[List any issues this PR will resolve]

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Commits are signed per the DCO using --signoff

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.

@petardz @sbcd90
[backport 2.x] Fixed compile issues related to replacing ImmutableOpe…
e2c46b2 
…nMap with java.util.Map

Signed-off-by: Subhobrata Dey <sbcd90@gmail.com>
@codecov
Copy link
Copy Markdown

codecov bot commented Jul 11, 2023

Codecov Report

Merging #477 (e2c46b2) into 2.x (7818810) will decrease coverage by 0.01%.
The diff coverage is 24.00%.

@@             Coverage Diff              @@
##                2.x     #477      +/-   ##
============================================
- Coverage     28.18%   28.18%   -0.01%     
+ Complexity      895      894       -1     
============================================
  Files           231      231              
  Lines          9337     9338       +1     
  Branches       1074     1074              
============================================
  Hits           2632     2632              
  Misses         6469     6469              
- Partials        236      237       +1
Impacted Files Coverage Δ
...curityanalytics/action/GetMappingsViewRequest.java 0.00% <ø> (ø)
...search/securityanalytics/alerts/AlertsService.java 21.90% <ø> (ø)
...rrelation/index/query/CorrelationQueryBuilder.java 0.00% <ø> (ø)
...indexmanagment/DetectorIndexManagementService.java 0.00% <0.00%> (ø)
...search/securityanalytics/mapper/MapperService.java 4.28% <0.00%> (ø)
...ensearch/securityanalytics/mapper/MapperUtils.java 25.00% <ø> (ø)
...rityanalytics/util/SecurityAnalyticsException.java 36.66% <0.00%> (ø)
...rityanalytics/action/GetIndexMappingsResponse.java 35.00% <46.15%> (ø)

... and 2 files with indirect coverage changes

@sbcd90 sbcd90 merged commit 7642e25 into opensearch-project:2.x Jul 11, 2023
@sbcd90 sbcd90 added the backport 2.9 backports to 2.9 label Jul 11, 2023
opensearch-trigger-bot bot pushed a commit that referenced this pull request Jul 11, 2023
@sbcd90 @petardz @github-actions
[backport 2.x] Fixed compile issues related to replacing ImmutableOpe…
8d5601c 
…nMap with java.util.Map (#477)

Signed-off-by: Subhobrata Dey <sbcd90@gmail.com>
Co-authored-by: Petar Dzepina <petar.dzepina@gmail.com>
(cherry picked from commit 7642e25)
@opensearch-trigger-bot opensearch-trigger-bot bot mentioned this pull request Jul 11, 2023
Merged
sbcd90 pushed a commit that referenced this pull request Jul 11, 2023
@opensearch-trigger-bot @petardz
[backport 2.x] Fixed compile issues related to replacing ImmutableOpe…
95c1b6e 
…nMap with java.util.Map (#477) (#478)

Signed-off-by: Subhobrata Dey <sbcd90@gmail.com>
Co-authored-by: Petar Dzepina <petar.dzepina@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

backport 2.9 backports to 2.9

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@sbcd90 @petardz