Skip to content

[Backport 2.19] PPL Alerting: Models - #1970

Closed
opensearch-trigger-bot[bot] wants to merge 1 commit into
2.19from
backport-1955-to-2.19
Closed

[Backport 2.19] PPL Alerting: Models#1970
opensearch-trigger-bot[bot] wants to merge 1 commit into
2.19from
backport-1955-to-2.19

Conversation

@opensearch-trigger-bot

Copy link
Copy Markdown
Contributor

Backport 497dea8 from #1955.

* PPL Alerting Models

Signed-off-by: Dennis Toepker <toepkerd@amazon.com>

* correcting readFrom in PPLTriggerRunResult

Signed-off-by: Dennis Toepker <toepkerd@amazon.com>

* adding minutes to duration fields, changing name from PPLMonitor to PPLSQLMonitor

Signed-off-by: Dennis Toepker <toepkerd@amazon.com>

* adjusting comment positions

Signed-off-by: Dennis Toepker <toepkerd@amazon.com>

* adding description field to monitor

Signed-off-by: Dennis Toepker <toepkerd@amazon.com>

* adding description javadoc

Signed-off-by: Dennis Toepker <toepkerd@amazon.com>

* various refactors

Signed-off-by: Dennis Toepker <toepkerd@amazon.com>

* making alert initiate its own parsing pointer

Signed-off-by: Dennis Toepker <toepkerd@amazon.com>

* adding number of results value validations

Signed-off-by: Dennis Toepker <toepkerd@amazon.com>

* adding full stops to error messages

Signed-off-by: Dennis Toepker <toepkerd@amazon.com>

* adding another missing full stop

Signed-off-by: Dennis Toepker <toepkerd@amazon.com>

* renaming customer exposed field name back to ppl_monitor

Signed-off-by: Dennis Toepker <toepkerd@amazon.com>

* renaming PPL to PPL_SQL in certain places

Signed-off-by: Dennis Toepker <toepkerd@amazon.com>

---------

Signed-off-by: Dennis Toepker <toepkerd@amazon.com>
Co-authored-by: Dennis Toepker <toepkerd@amazon.com>
(cherry picked from commit 497dea8)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
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.

1 participant