sql icon indicating copy to clipboard operation
sql copied to clipboard

[Backport 2.x] Pass accountId to EMRServerlessClientFactory.getClient

Open opensearch-trigger-bot[bot] opened this issue 1 year ago • 2 comments

Backport e24b51fd392985bee4224f4c2a60d0dbe2a5e1b4 from #2783.

Codecov Report

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

Project coverage is 95.18%. Comparing base (775aa90) to head (5581203).

Additional details and impacted files
@@             Coverage Diff              @@
##                2.x    #2793      +/-   ##
============================================
+ Coverage     94.24%   95.18%   +0.93%     
+ Complexity     5065     4824     -241     
============================================
  Files           501      444      -57     
  Lines         14520    13160    -1360     
  Branches        945      854      -91     
============================================
- Hits          13685    12526    -1159     
+ Misses          792      620     -172     
+ Partials         43       14      -29     
Flag Coverage Δ
sql-engine 95.18% <ø> (+0.93%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Jun 28 '24 23:06 codecov[bot]

Build failure is caused by integration test. Fix same as https://github.com/opensearch-project/sql/pull/2794 is needed.

ykmr1224 avatar Jun 29 '24 00:06 ykmr1224

Closing as this backport is addressed in https://github.com/opensearch-project/sql/pull/2822

ykmr1224 avatar Jul 11 '24 23:07 ykmr1224