fix(snowflake): look up last modified timestamps in each table's database

[DG47]

Description

Fixes #6131.

SnowflakeEngineAdapter.get_table_last_modified_ts read LAST_ALTERED from the unqualified INFORMATION_SCHEMA.TABLES view. In Snowflake that view only lists tables in the session's current database, so tables in any other database were never returned. The freshness() signal then raised MissingSourceError for models that read external tables from a different database than the gateway's (for example RAW vs ANALYTICS).

This change groups the requested tables by database and queries each database's own INFORMATION_SCHEMA.TABLES ("RAW".INFORMATION_SCHEMA.TABLES, and so on). The TABLE_CATALOG predicate is no longer needed because each view is already scoped to one database. The BigQuery implementation groups per dataset in the same way. The query is now built with sqlglot rather than string formatting, so table and schema names are escaped as string literals. Tables without a catalog still go to the current database's information schema.

Test Plan

• Added test_get_table_last_modified_ts_multiple_catalogs to tests/core/engine_adapter/test_snowflake.py. It requests three tables across two databases and checks that one query per database is issued and the timestamps are combined. It fails on main (one unqualified query) and passes with this change.
• pytest tests/core/engine_adapter -m "not docker and not remote" -n 8: 714 passed, 3 skipped.
• pytest -n 8 -m "fast and not web and not github and not dbt and not jupyter and not cicdonly and not isolated": 2436 passed, 6 failed. The 6 failures are the pyodbc connection-string tests, which fail locally on main too because unixODBC is not installed.
• SKIP=prettier,eslint pre-commit run --all-files: ruff, ruff-format, mypy and valid migrations all pass.

The Snowflake freshness integration tests are skipped for Snowflake (_skip_snowflake in test_freshness.py), and I did not run this against a live Snowflake account.

Checklist

• I have run make style and fixed any issues
• I have added tests for my changes (if applicable)
• All existing tests pass (make fast-test)
• My commits are signed off (git commit -s) per the DCO

[nicksspirit]

Hey @DG47 — thanks for fixing this. Querying each database's own INFORMATION_SCHEMA is the right fix, and the new test covers the RAW vs ANALYTICS case from the issue.

One edge case: the tables are grouped by table.catalog, which drops quoting. In Snowflake, unquoted raw is database RAW, and quoted "raw" is a different database. If both appear in one call, they get grouped together and the query uses whichever catalog identifier came first.

Could you key the groups on the catalog identifier itself, so quoted and unquoted names stay separate? A small test that passes both "raw" and raw in one call would show they hit two different information schemas.

Thanks again for taking the time to put this up.

[DG47]

Thanks @nicksspirit, good catch. Updated in c7a1528: the groups are now keyed on the catalog identifier (quoting included) instead of its name, so "raw" and raw go to two different information schemas. Added test_get_table_last_modified_ts_quoted_and_unquoted_catalogs, which passes both in one call and checks the two queries. It fails with the previous grouping and passes now.

[nicksspirit]

One test is failing on windows but it's more so a dependency update thing not a huge blocker