Skip to content

Conversation

@opensearch-trigger-bot
Copy link
Contributor

Backport ff9ac16 from #1278

* Add benchmark module and test implicit cast

Signed-off-by: Chen Dai <daichen@amazon.com>

* Benchmark the second approach from Yury

Signed-off-by: Chen Dai <daichen@amazon.com>

* Refactor code to make benchmark module for future use

Signed-off-by: Chen Dai <daichen@amazon.com>

* Add readme

Signed-off-by: Chen Dai <daichen@amazon.com>

* Move benchmark class to right source location

Signed-off-by: Chen Dai <daichen@amazon.com>

* Change depedency accessiblity from api to impl

Signed-off-by: Chen Dai <daichen@amazon.com>

Signed-off-by: Chen Dai <daichen@amazon.com>
(cherry picked from commit ff9ac16)
@opensearch-trigger-bot opensearch-trigger-bot bot requested a review from a team as a code owner January 23, 2023 19:11
@dai-chen dai-chen added the infrastructure Changes to infrastructure, testing, CI/CD, pipelines, etc. label Jan 23, 2023
@codecov-commenter
Copy link

codecov-commenter commented Jan 23, 2023

Codecov Report

Merging #1291 (3c4a1b2) into 2.x (a5931e0) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##                2.x    #1291   +/-   ##
=========================================
  Coverage     98.35%   98.35%           
  Complexity     3605     3605           
=========================================
  Files           345      345           
  Lines          8914     8914           
  Branches        567      567           
=========================================
  Hits           8767     8767           
  Misses          142      142           
  Partials          5        5           
Flag Coverage Δ
sql-engine 98.35% <ø> (ø)

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

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

@dai-chen dai-chen merged commit db9be18 into 2.x Jan 23, 2023
@github-actions github-actions bot deleted the backport/backport-1278-to-2.x branch January 23, 2023 21:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

infrastructure Changes to infrastructure, testing, CI/CD, pipelines, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants