Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Delta Lake: Remove useless method in BaseDeltaLakeConnectorSmokeTest #14905

Merged

Conversation

findinpath
Copy link
Contributor

The Delta Lake connector supports INSERT operation reason why this operation is useless.

Description

Test infrastructure change

Non-technical explanation

N/A

Release notes

(x) This is not user-visible or docs only and no release notes are required.
( ) Release notes are required, please propose a release note for me.
( ) Release notes are required, with the following suggested text:

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

The Delta Lake connector supports `INSERT` operation reason why
this operation is useless.
@findinpath findinpath changed the title Delta Lake: Remove useless method Delta Lake: Remove useless method in BaseDeltaLakeConnectorSmokeTest Nov 4, 2022
@ebyhr ebyhr added the no-release-notes This pull request does not require release notes entry label Nov 6, 2022
@ebyhr ebyhr merged commit 0e2a1ec into trinodb:master Nov 6, 2022
@ebyhr
Copy link
Member

ebyhr commented Nov 6, 2022

Merged, thanks!

@github-actions github-actions bot added this to the 403 milestone Nov 6, 2022
@alexjo2144
Copy link
Member

We will just need to put it back in if we add any new read only file systems, like GCS was until recently.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-release-notes This pull request does not require release notes entry
Development

Successfully merging this pull request may close these issues.

3 participants