search

opensearch-project / anomaly-detection

Identify atypical data and receive automatic notifications
https://opensearch.org/docs/latest/monitoring-plugins/ad/index/
Apache License 2.0
67 stars 73 forks source link

Revert "Add flatten_result_index_mapping field to Config (#1317)" #1350

Closed jackiehanyang closed 1 month ago

jackiehanyang commented 1 month ago

This reverts commit cbe1f336faf237907e4786d4c7db429a9126954c.

Description

[Describe what this change achieves]

Related Issues

Resolves #[Issue number to be closed when this PR is merged]

Check List

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[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 80.17%. Comparing base (9d3bb72) to head (bb12ebe). Report is 1 commits behind head on 2.18.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/opensearch-project/anomaly-detection/pull/1350/graphs/tree.svg?width=650&height=150&src=pr&token=ZPONorT0bX&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensearch-project)](https://app.codecov.io/gh/opensearch-project/anomaly-detection/pull/1350?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensearch-project) ```diff @@ Coverage Diff @@ ## 2.18 #1350 +/- ## ========================================= Coverage 80.16% 80.17% + Complexity 5687 5684 -3 ========================================= Files 533 533 Lines 23430 23409 -21 Branches 2335 2333 -2 ========================================= - Hits 18783 18768 -15 + Misses 3545 3544 -1 + Partials 1102 1097 -5 ``` | [Flag](https://app.codecov.io/gh/opensearch-project/anomaly-detection/pull/1350/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensearch-project) | Coverage Δ | | |---|---|---| | [plugin](https://app.codecov.io/gh/opensearch-project/anomaly-detection/pull/1350/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensearch-project) | `80.17% <ø> (+<0.01%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensearch-project#carryforward-flags-in-the-pull-request-comment) to find out more. | [Files with missing lines](https://app.codecov.io/gh/opensearch-project/anomaly-detection/pull/1350?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensearch-project) | Coverage Δ | | |---|---|---| | [src/main/java/org/opensearch/ad/model/ADTask.java](https://app.codecov.io/gh/opensearch-project/anomaly-detection/pull/1350?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Forg%2Fopensearch%2Fad%2Fmodel%2FADTask.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensearch-project#diff-c3JjL21haW4vamF2YS9vcmcvb3BlbnNlYXJjaC9hZC9tb2RlbC9BRFRhc2suamF2YQ==) | `98.00% <ø> (-0.01%)` | :arrow_down: | | [.../java/org/opensearch/ad/model/AnomalyDetector.java](https://app.codecov.io/gh/opensearch-project/anomaly-detection/pull/1350?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Forg%2Fopensearch%2Fad%2Fmodel%2FAnomalyDetector.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensearch-project#diff-c3JjL21haW4vamF2YS9vcmcvb3BlbnNlYXJjaC9hZC9tb2RlbC9Bbm9tYWx5RGV0ZWN0b3IuamF2YQ==) | `89.34% <ø> (+0.35%)` | :arrow_up: | | [.../handler/AbstractAnomalyDetectorActionHandler.java](https://app.codecov.io/gh/opensearch-project/anomaly-detection/pull/1350?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Forg%2Fopensearch%2Fad%2Frest%2Fhandler%2FAbstractAnomalyDetectorActionHandler.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensearch-project#diff-c3JjL21haW4vamF2YS9vcmcvb3BlbnNlYXJjaC9hZC9yZXN0L2hhbmRsZXIvQWJzdHJhY3RBbm9tYWx5RGV0ZWN0b3JBY3Rpb25IYW5kbGVyLmphdmE=) | `97.82% <ø> (-0.05%)` | :arrow_down: | | [...va/org/opensearch/forecast/model/ForecastTask.java](https://app.codecov.io/gh/opensearch-project/anomaly-detection/pull/1350?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Forg%2Fopensearch%2Fforecast%2Fmodel%2FForecastTask.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensearch-project#diff-c3JjL21haW4vamF2YS9vcmcvb3BlbnNlYXJjaC9mb3JlY2FzdC9tb2RlbC9Gb3JlY2FzdFRhc2suamF2YQ==) | `97.99% <ø> (-0.01%)` | :arrow_down: | | [...java/org/opensearch/forecast/model/Forecaster.java](https://app.codecov.io/gh/opensearch-project/anomaly-detection/pull/1350?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Forg%2Fopensearch%2Fforecast%2Fmodel%2FForecaster.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensearch-project#diff-c3JjL21haW4vamF2YS9vcmcvb3BlbnNlYXJjaC9mb3JlY2FzdC9tb2RlbC9Gb3JlY2FzdGVyLmphdmE=) | `79.88% <ø> (+1.39%)` | :arrow_up: | | [.../rest/handler/AbstractForecasterActionHandler.java](https://app.codecov.io/gh/opensearch-project/anomaly-detection/pull/1350?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Forg%2Fopensearch%2Fforecast%2Frest%2Fhandler%2FAbstractForecasterActionHandler.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensearch-project#diff-c3JjL21haW4vamF2YS9vcmcvb3BlbnNlYXJjaC9mb3JlY2FzdC9yZXN0L2hhbmRsZXIvQWJzdHJhY3RGb3JlY2FzdGVyQWN0aW9uSGFuZGxlci5qYXZh) | `91.11% <ø> (-0.20%)` | :arrow_down: | | [...n/java/org/opensearch/timeseries/model/Config.java](https://app.codecov.io/gh/opensearch-project/anomaly-detection/pull/1350?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Forg%2Fopensearch%2Ftimeseries%2Fmodel%2FConfig.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensearch-project#diff-c3JjL21haW4vamF2YS9vcmcvb3BlbnNlYXJjaC90aW1lc2VyaWVzL21vZGVsL0NvbmZpZy5qYXZh) | `86.73% <ø> (+0.42%)` | :arrow_up: | ... and [8 files with indirect coverage changes](https://app.codecov.io/gh/opensearch-project/anomaly-detection/pull/1350/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensearch-project)