Skip to content

[Backport 2.10] Bulk index findings and sequentially invoke auto-correlations#1463

Open
opensearch-trigger-bot[bot] wants to merge 1 commit into
2.10from
backport-1355-to-2.10
Open

[Backport 2.10] Bulk index findings and sequentially invoke auto-correlations#1463
opensearch-trigger-bot[bot] wants to merge 1 commit into
2.10from
backport-1355-to-2.10

Conversation

@opensearch-trigger-bot
Copy link
Copy Markdown
Contributor

Backport b561965 from #1355.

* Bulk index findings and sequentially invoke auto-correlations

Signed-off-by: Megha Goyal <goyamegh@amazon.com>

* Bulk index findings in batches of 10000 and make it configurable

Signed-off-by: Megha Goyal <goyamegh@amazon.com>

* Addressing review comments

Signed-off-by: Megha Goyal <goyamegh@amazon.com>

* Add integ tests to test bulk index findings

Signed-off-by: Megha Goyal <goyamegh@amazon.com>

* Fix ktlint formatting

Signed-off-by: Megha Goyal <goyamegh@amazon.com>

---------

Signed-off-by: Megha Goyal <goyamegh@amazon.com>
(cherry picked from commit b561965)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@riysaxen-amzn
Copy link
Copy Markdown
Collaborator

tests are failing, let's check them

@engechas
Copy link
Copy Markdown
Collaborator

Test failures are legitimate and will need a manual backport to resolve:

> Task :alerting:compileTestKotlin FAILED
e: file:///home/runner/work/alerting/alerting/alerting/src/test/kotlin/org/opensearch/alerting/DocumentMonitorRunnerIT.kt:409:96 Cannot find a parameter with this name: fields
e: file:///home/runner/work/alerting/alerting/alerting/src/test/kotlin/org/opensearch/alerting/DocumentMonitorRunnerIT.kt:409:105 Not enough information to infer type variable T

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants