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] fix detector writeTo() method missing fields #698

Merged
merged 1 commit into from
Oct 26, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 7a45203 from #695

* fix detector writeTo() method missing fields

Signed-off-by: Surya Sashank Nistala <snistala@amazon.com>

* fix test

Signed-off-by: Surya Sashank Nistala <snistala@amazon.com>

---------

Signed-off-by: Surya Sashank Nistala <snistala@amazon.com>
(cherry picked from commit 7a45203)
@codecov
Copy link

codecov bot commented Oct 26, 2023

Codecov Report

Merging #698 (b93df46) into 2.x (d14fd60) will increase coverage by 0.09%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##                2.x     #698      +/-   ##
============================================
+ Coverage     25.02%   25.11%   +0.09%     
- Complexity      941      946       +5     
============================================
  Files           255      255              
  Lines         11162    11167       +5     
  Branches       1251     1251              
============================================
+ Hits           2793     2805      +12     
+ Misses         8115     8111       -4     
+ Partials        254      251       -3     
Files Coverage Δ
...g/opensearch/securityanalytics/model/Detector.java 66.66% <100.00%> (+2.33%) ⬆️

... and 1 file with indirect coverage changes

@eirsep eirsep merged commit ef92b4f into 2.x Oct 26, 2023
13 of 15 checks passed
@github-actions github-actions bot deleted the backport/backport-695-to-2.x branch October 26, 2023 17:17
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.

None yet

1 participant