Skip to content

Test SHOW TABLES after relation creation#16993

Merged
findepi merged 3 commits intotrinodb:masterfrom
findepi:findepi/test-show-tables-after-relation-creation-e47c9c
Apr 14, 2023
Merged

Test SHOW TABLES after relation creation#16993
findepi merged 3 commits intotrinodb:masterfrom
findepi:findepi/test-show-tables-after-relation-creation-e47c9c

Conversation

@findepi
Copy link
Member

@findepi findepi commented Apr 12, 2023

Move AbstractTestQueries.testShowTables to
AbstractTestEngineOnlyQueries.
Add BaseConnectorTest.testShowTables testing SHOW TABLES with a
connector after a table or view is created in the connector.

Move them from `QueryAssert` to `MaterializedResult`.
@findepi findepi force-pushed the findepi/test-show-tables-after-relation-creation-e47c9c branch from cdab9de to 7a81cd5 Compare April 13, 2023 11:50
@findepi findepi force-pushed the findepi/test-show-tables-after-relation-creation-e47c9c branch from 7a81cd5 to 0d764f5 Compare April 13, 2023 13:25
Copy link
Member

@hashhar hashhar left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks Piotr.

@findepi findepi force-pushed the findepi/test-show-tables-after-relation-creation-e47c9c branch from e86e9cb to 8c86936 Compare April 14, 2023 11:46
Move `AbstractTestQueries.testShowTables` to
`AbstractTestEngineOnlyQueries`.
Add `BaseConnectorTest.testShowTables` testing `SHOW TABLES` with a
connector after a table or view is created in the connector.
@findepi findepi force-pushed the findepi/test-show-tables-after-relation-creation-e47c9c branch from 8c86936 to 55bd4d6 Compare April 14, 2023 12:07
@findepi findepi merged commit 5c520c4 into trinodb:master Apr 14, 2023
@findepi findepi deleted the findepi/test-show-tables-after-relation-creation-e47c9c branch April 14, 2023 20:52
@github-actions github-actions bot added this to the 414 milestone Apr 15, 2023
@colebow colebow added the no-release-notes This pull request does not require release notes entry label Apr 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed no-release-notes This pull request does not require release notes entry

Development

Successfully merging this pull request may close these issues.

5 participants