Added document _id as param for terms query when searching alerts by …

[stevanbz]

…their ids

Signed-off-by: Stevan Buzejic [email protected]

(#748)

Description of changes:
Changed the terms query used for getting the alerts to use the doc id (_id) instead of the alert id
CheckList:
[ ] 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-commenter]

Codecov Report

Merging #753 (06e6d22) into main (780744f) will decrease coverage by 0.14%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##               main     #753      +/-   ##
============================================
- Coverage     75.43%   75.29%   -0.15%     
  Complexity      116      116              
============================================
  Files           125      125              
  Lines          6828     6828              
  Branches       1024     1024              
============================================
- Hits           5151     5141      -10     
- Misses         1143     1155      +12     
+ Partials        534      532       -2     

Impacted Files	Coverage Δ
...rch/alerting/transport/TransportGetAlertsAction.kt	75.51% <100.00%> (ø)
.../alerting/transport/TransportIndexMonitorAction.kt	54.93% <0.00%> (-2.97%)	⬇️
.../kotlin/org/opensearch/alerting/util/IndexUtils.kt	70.83% <0.00%> (-2.09%)	⬇️
...destinationmigration/DestinationConversionUtils.kt	70.00% <0.00%> (-1.12%)	⬇️
...ing/model/destination/DestinationContextFactory.kt	78.57% <0.00%> (+3.57%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.