[Backport 2.x] Fix null response from pow/power and added missing integration testing by opensearch-trigger-bot[bot] · Pull Request #1459 · opensearch-project/sql · GitHub
Skip to content

[Backport 2.x] Fix null response from pow/power and added missing integration testing#1459

Merged
Yury-Fridlyand merged 1 commit into
2.xfrom
backport/backport-1457-to-2.x
Mar 22, 2023
Merged

[Backport 2.x] Fix null response from pow/power and added missing integration testing#1459
Yury-Fridlyand merged 1 commit into
2.xfrom
backport/backport-1457-to-2.x

Conversation

@opensearch-trigger-bot

@opensearch-trigger-bot opensearch-trigger-bot Bot commented Mar 21, 2023

Copy link
Copy Markdown
Contributor

Backport a35f963 from #1457

@matthewryanwells @github-actions
Added missing integration tests for the pow and power functions (#247) (
c973902 
#1457)

* Changed pow and power to return null when an invalid response would be returned and added missing integration tests for the pow and power function.

Signed-off-by: Matthew Wells <matthew.wells@improving.com>
(cherry picked from commit a35f963)
@opensearch-trigger-bot opensearch-trigger-bot Bot requested a review from a team as a code owner March 21, 2023 22:57
@codecov-commenter

codecov-commenter commented Mar 21, 2023 •
edited by codecov Bot
Loading

Copy link
Copy Markdown

@dai-chen dai-chen added the enhancement New feature or request label Mar 22, 2023
@Yury-Fridlyand Yury-Fridlyand merged commit 49a82ae into 2.x Mar 22, 2023
@github-actions github-actions Bot deleted the backport/backport-1457-to-2.x branch March 22, 2023 16:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dai-chen dai-chen dai-chen approved these changes
+1 more reviewer
@Yury-Fridlyand Yury-Fridlyand Yury-Fridlyand approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@codecov-commenter @dai-chen @Yury-Fridlyand @matthewryanwells