Skip to content

[Backport 2.11] Use a common constant to specify the version for log type mappings #708#778

Merged
jowg-amazon merged 1 commit into2.11from
backport/backport-734-to-2.11
Dec 8, 2023
Merged

[Backport 2.11] Use a common constant to specify the version for log type mappings #708#778
jowg-amazon merged 1 commit into2.11from
backport/backport-734-to-2.11

Conversation

@opensearch-trigger-bot
Copy link
Copy Markdown
Contributor

Backport 7066923 from #734

… (#734)

* json parser

Signed-off-by: Joanne Wang <jowg@amazon.com>

* update gradle json dependency

Signed-off-by: Joanne Wang <jowg@amazon.com>

* use xcontenthelper to parse json file

Signed-off-by: Joanne Wang <jowg@amazon.com>

---------

Signed-off-by: Joanne Wang <jowg@amazon.com>
(cherry picked from commit 7066923)
@codecov
Copy link
Copy Markdown

codecov bot commented Dec 8, 2023

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (e06858d) 24.63% compared to head (578b9c9) 24.67%.

Files Patch % Lines
...arch/securityanalytics/logtype/LogTypeService.java 83.33% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               2.11     #778      +/-   ##
============================================
+ Coverage     24.63%   24.67%   +0.03%     
- Complexity      946      947       +1     
============================================
  Files           255      255              
  Lines         11369    11374       +5     
  Branches       1279     1279              
============================================
+ Hits           2801     2806       +5     
  Misses         8319     8319              
  Partials        249      249              

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

@jowg-amazon jowg-amazon merged commit 48c2cfa into 2.11 Dec 8, 2023
@github-actions github-actions bot deleted the backport/backport-734-to-2.11 branch December 8, 2023 18:10
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.

1 participant