trino icon indicating copy to clipboard operation
trino copied to clipboard

Handle empty Iceberg tables while executing procedures

Open alexjo2144 opened this issue 3 years ago • 0 comments

Description

If a table was just created it may not contain any snapshots. Procedures run on tables that do not contain any snapshots can safely do nothing.

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

Fix

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

Iceberg connector

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

Prevent query failure in the edge case where a table is empty and has no history.

Related issues, pull requests, and links

Related to: https://github.com/trinodb/trino/pull/13576

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:

alexjo2144 avatar Aug 09 '22 22:08 alexjo2144

All set, thanks for the reviews

alexjo2144 avatar Aug 10 '22 16:08 alexjo2144

@alexjo2144 can you please suggest RN wording?

findepi avatar Aug 10 '22 18:08 findepi

CI hit https://github.com/trinodb/trino/issues/13556

alexjo2144 avatar Aug 10 '22 20:08 alexjo2144

@findepi added a suggestion to the PR description

alexjo2144 avatar Aug 10 '22 20:08 alexjo2144