Skip to content

Add BaseConnectorTest test to verify failure when creating tables in a schema where it doesn't exist#11808

Merged
ebyhr merged 5 commits intomasterfrom
ebi/test-create-table-schema-not-found
Apr 6, 2022
Merged

Add BaseConnectorTest test to verify failure when creating tables in a schema where it doesn't exist#11808
ebyhr merged 5 commits intomasterfrom
ebi/test-create-table-schema-not-found

Conversation

@ebyhr
Copy link
Copy Markdown
Member

@ebyhr ebyhr commented Apr 6, 2022

Description

Add BaseConnectorTest test to verify failure when creating tables in a schema where it doesn't exist. Also, add support for creating and dropping schemas in Accumulo connector.
Fixes #11794

Documentation

( ) No documentation is needed.
(x) Sufficient documentation is included in this PR.
( ) Documentation PR is available with #prnumber.
( ) Documentation issue #issuenumber is filed, and can be handled later.

Release notes

( ) No release notes entries required.
(x) Release notes entries required with the following suggested text:

# Section
* Fix some things. ({issue}`issuenumber`)

@cla-bot cla-bot bot added the cla-signed label Apr 6, 2022
@github-actions github-actions bot added the docs label Apr 6, 2022
@ebyhr ebyhr changed the title Ebi/test create table schema not found Add BaseConnectorTest test to verify failure when creating tables in a schema where it doesn't exist Apr 6, 2022
Additionally, deny creating tables in a schema where it doesn't exist.
@ebyhr ebyhr force-pushed the ebi/test-create-table-schema-not-found branch 2 times, most recently from 86bf16f to 768b6cb Compare April 6, 2022 05:39
@ebyhr
Copy link
Copy Markdown
Member Author

ebyhr commented Apr 6, 2022

CI hit #11764

@ebyhr ebyhr marked this pull request as ready for review April 6, 2022 06:59
@ebyhr ebyhr requested review from findepi and hashhar April 6, 2022 08:02
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

remove the test override, this should be handed by the check of the conenctor behavior

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

SUPPORTS_CREATE_TABLE_WITH_DATA

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

remove (it's kind of obvious this is not "the test" for create table)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

ok to remove

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

document why override?

@ebyhr ebyhr force-pushed the ebi/test-create-table-schema-not-found branch from 768b6cb to 844d935 Compare April 6, 2022 08:33
@ebyhr ebyhr merged commit b0bf19c into master Apr 6, 2022
@ebyhr ebyhr deleted the ebi/test-create-table-schema-not-found branch April 6, 2022 12:23
@ebyhr ebyhr mentioned this pull request Apr 6, 2022
@github-actions github-actions bot added this to the 376 milestone Apr 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

Add BaseConnectorTest test to verify failure when creating tables in a schema where it doesn't exist

3 participants