Skip to content

Refactor assertCreateTableAsSelect() and assertExplainAnalyze() functions#21811

Merged
majetideepak merged 1 commit intoprestodb:masterfrom
pdabre12:refactor-functions
Feb 8, 2024
Merged

Refactor assertCreateTableAsSelect() and assertExplainAnalyze() functions#21811
majetideepak merged 1 commit intoprestodb:masterfrom
pdabre12:refactor-functions

Conversation

@pdabre12
Copy link
Contributor

@pdabre12 pdabre12 commented Jan 29, 2024

Refactoring the assertCreateTableAsSelect() and assertExplainAnalyze() functions to live in AbstractTestQueryFrameWork for code re-usability.
The goal is to use these for the NativeQueryRunner tests.

@pdabre12 pdabre12 requested a review from a team as a code owner January 29, 2024 19:36
@pdabre12 pdabre12 requested a review from presto-oss January 29, 2024 19:36
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Jan 29, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: pdabre12 / name: Pratik Joseph Dabre (c054d03)

@pdabre12 pdabre12 changed the title Refactor assertCreateTableAsSelect() and assertExplainAnalyze() functions [native] Refactor assertCreateTableAsSelect() and assertExplainAnalyze() functions Jan 29, 2024
Copy link
Collaborator

@majetideepak majetideepak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@majetideepak majetideepak changed the title [native] Refactor assertCreateTableAsSelect() and assertExplainAnalyze() functions Refactor assertCreateTableAsSelect() and assertExplainAnalyze() functions Jan 29, 2024
@github-actions
Copy link

github-actions bot commented Jan 29, 2024

Codenotify: Notifying subscribers in CODENOTIFY files for diff 0ddb2c1...c054d03.

No notifications.

@majetideepak majetideepak merged commit 20e7712 into prestodb:master Feb 8, 2024
@wanglinsong wanglinsong mentioned this pull request Feb 12, 2024
64 tasks
@pdabre12 pdabre12 deleted the refactor-functions branch March 1, 2024 20:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants