Skip to content

Fix Iceberg Glue testShowCreateTable#13438

Merged
ebyhr merged 1 commit intotrinodb:masterfrom
alexjo2144:iceberg/glue-show-table-test
Aug 2, 2022
Merged

Fix Iceberg Glue testShowCreateTable#13438
ebyhr merged 1 commit intotrinodb:masterfrom
alexjo2144:iceberg/glue-show-table-test

Conversation

@alexjo2144
Copy link
Member

Description

Fix Iceberg Glue smoke test testShowCreateTable

Is this change a fix, improvement, new feature, refactoring, or other?

Test fix

Is this a change to the core query engine, a connector, client library, or the SPI interfaces? (be specific)

Test only

How would you describe this change to a non-technical end user or system administrator?

n/a

Related issues, pull requests, and links

Failure caused by: #12941

Documentation

(x) No documentation is needed.
( ) 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

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

@cla-bot cla-bot bot added the cla-signed label Aug 1, 2022
@alexjo2144 alexjo2144 added the no-release-notes This pull request does not require release notes entry label Aug 1, 2022
@alexjo2144 alexjo2144 requested a review from findepi August 1, 2022 15:32
@findepi
Copy link
Member

findepi commented Aug 1, 2022

Test PR with secrets: #13439
(@nineinchnick's #12817 will make these PRs easier)

@ebyhr ebyhr merged commit 890cb54 into trinodb:master Aug 2, 2022
@ebyhr
Copy link
Member

ebyhr commented Aug 2, 2022

Merged, thanks!

@github-actions github-actions bot added this to the 392 milestone Aug 2, 2022
@alexjo2144 alexjo2144 deleted the iceberg/glue-show-table-test branch August 2, 2022 13:04
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.

3 participants