Fix concurrency issue with load snapshot from DeltaLakeMetadata

[vlad-lyutenko]

Description

If getTableHandle called concurrently for same table, getSnapshot method could produce concurrency issue:

steps to reproduce:

• create some table: CREATE table my_catalog_delta.my_schema.my_table_1 AS SELECT 1 as a;
• run query:

SELECT
  t.table_name,
  t.table_type,
  mv.catalog_name IS NOT NULL as is_materialized_view
FROM
  my_catalog_delta.information_schema.tables t
  LEFT JOIN system.metadata.materialized_views mv 
    ON 
       t.table_catalog = mv.catalog_name AND 
       t.table_schema = mv.schema_name AND 
       t.table_name = mv.name
WHERE
  t.table_schema = 'my_schema'
  AND t.table_name = 'my_table_1'
  AND t.table_type IN ('BASE TABLE', 'VIEW')
ORDER by
  t.table_name

If you repeat it a few times, you will notice that sometime query failed with : latestTableVersions changed concurrently exception.

It happens because there is multiple concurrent calls to getSnapshot and first call updated latestTableVersions map but not yet queriedSnapshots map and second call see these 2 tables in inconsistent state ().

Additional context and related issues

Release notes

( ) This is not user-visible or is 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`)

[ebyhr]

I can't reproduce the issue. Did you use DefaultDeltaLakeQueryRunnerMain?

[chenjian2664]

I am confusing how it can happen, DeltaLakeMetadata creates the latestTableVersions everytime, which means to me is that only the case: there exists multiple threads arrive this method in one transaction(query) ?
Multiple run should not hit this case

I see the code in DeltaLakeTransactionManager:

    public void begin(ConnectorTransactionHandle transaction)
    {
        MemoizedMetadata previousValue = transactions.putIfAbsent(transaction, new MemoizedMetadata());
        checkState(previousValue == null);
    }