Support snapshot queries on Iceberg system tables - IcebergSystemTableHandle is a ConnectorTableHandle with custom instanceof handling in IcebergMetadata

[findinpath]

Description

Use IcebergSystemTableHandle for dealing with system tables
because it encodes for the table being queried also:

• snapshot id
• table type

NOTE that this change reuses IcebergMetadata columns for performing queries related to system tables.

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

Bugfix

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?

The metadata tables exposed by the Iceberg connector for its tables should support snapshot/versioned queries in the same fashion as the Iceberg tables.

SELECT * FROM table1$files FOR VERSION AS OF 1242141
SELECT * FROM table1$files FOR TIMESTAMP AS OF TIMESTAMP '2022-01-01'

Related issues, pull requests, and links

Fixes #12736

This PR is built on top of #12737 and uses IcebergSystemTableHandle for dealing with system tables, instead of IcebergTableHandle.

This PR could pave the way to the functionality of having distributed $files calculation. See #4840

Having a distributed retrieval for $files could be beneficial in debugging scenarios when dealing with tables with a lot of files. Probably other scenarios could follow.

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

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

# Iceberg
* Support snapshot queries on Iceberg system tables

[findinpath]

In io.trino.connector.system.SystemTablesMetadata#applyFilter there is made the call

trino/core/trino-main/src/main/java/io/trino/connector/system/SystemTablesMetadata.java

Line 159 in 3e06ac2

SystemTable systemTable = checkAndGetTable(session, table);

to verify whether the table has disappeared in the meantime.

Should we do it here as well?