Skip to content

Fix broken Iceberg Glue access count test#17185

Merged
ebyhr merged 1 commit intomasterfrom
ebi/glue-fix-test
Apr 24, 2023
Merged

Fix broken Iceberg Glue access count test#17185
ebyhr merged 1 commit intomasterfrom
ebi/glue-fix-test

Conversation

@ebyhr
Copy link
Copy Markdown
Member

@ebyhr ebyhr commented Apr 22, 2023

No description provided.

@cla-bot cla-bot bot added the cla-signed label Apr 22, 2023
@ebyhr ebyhr requested review from electrum and findepi April 22, 2023 06:59
@ebyhr ebyhr added the no-release-notes This pull request does not require release notes entry label Apr 22, 2023
@github-actions github-actions bot added the iceberg Iceberg connector label Apr 22, 2023
@ebyhr ebyhr merged commit 21608c7 into master Apr 24, 2023
@ebyhr ebyhr deleted the ebi/glue-fix-test branch April 24, 2023 04:55
@github-actions github-actions bot added this to the 415 milestone Apr 24, 2023
.add(GET_DATABASE)
.add(CREATE_TABLE)
.addCopies(GET_TABLE, 6)
.addCopies(GET_TABLE, 5)
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.

nice!

do we know which change introduced this improvement?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

#16557 reduced the access count.

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.

thanks @ebyhr for the link.
thanks @electrum for the improvement.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Development

Successfully merging this pull request may close these issues.

3 participants