Skip to content

Use a common constant to specify the version for log type mappings #708#734

Merged
jowg-amazon merged 3 commits intoopensearch-project:mainfrom
jowg-amazon:bug_708
Dec 8, 2023
Merged

Use a common constant to specify the version for log type mappings #708#734
jowg-amazon merged 3 commits intoopensearch-project:mainfrom
jowg-amazon:bug_708

Conversation

@jowg-amazon
Copy link
Copy Markdown
Collaborator

@jowg-amazon jowg-amazon commented Nov 27, 2023

Description

Sets the log_type_mapping_version in LogTypeService.java to the same schema version values used in log_type_config_mapping.json

Issues Resolved

#708

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.

@codecov
Copy link
Copy Markdown

codecov bot commented Nov 27, 2023

Codecov Report

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

Comparison is base (c0f7bd9) 24.73% compared to head (aad562e) 24.77%.
Report is 1 commits behind head on main.

Files Patch % Lines
...arch/securityanalytics/logtype/LogTypeService.java 83.33% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main     #734      +/-   ##
============================================
+ Coverage     24.73%   24.77%   +0.03%     
- Complexity     1021     1022       +1     
============================================
  Files           275      275              
  Lines         12662    12668       +6     
  Branches       1389     1389              
============================================
+ Hits           3132     3138       +6     
  Misses         9264     9264              
  Partials        266      266              

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

Signed-off-by: Joanne Wang <jowg@amazon.com>
Signed-off-by: Joanne Wang <jowg@amazon.com>
Signed-off-by: Joanne Wang <jowg@amazon.com>
@jowg-amazon jowg-amazon merged commit 7066923 into opensearch-project:main Dec 8, 2023
opensearch-trigger-bot bot pushed a commit that referenced this pull request Dec 8, 2023
… (#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)
opensearch-trigger-bot bot pushed a commit that referenced this pull request Dec 8, 2023
… (#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)
opensearch-trigger-bot bot pushed a commit that referenced this pull request Dec 8, 2023
… (#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)
jowg-amazon added a commit that referenced this pull request Dec 8, 2023
… (#734) (#777)

* 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)

Co-authored-by: Joanne Wang <jowg@amazon.com>
jowg-amazon added a commit that referenced this pull request Dec 8, 2023
… (#734) (#776)

* 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)

Co-authored-by: Joanne Wang <jowg@amazon.com>
jowg-amazon added a commit that referenced this pull request Dec 8, 2023
… (#734) (#778)

* 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)

Co-authored-by: Joanne Wang <jowg@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants