shyjsarah opened a new pull request, #747:
URL: https://github.com/apache/paimon-rust/pull/747

   ## Purpose
   
   After external table routing was made fail-closed, DataFusion metadata 
queries such as `information_schema.columns` could fail if a catalog contained 
any external table whose engine was not registered. This also surfaced through 
statements that inspect catalog metadata.
   
   ## Changes
   
   - preserve catalog field metadata when classifying external tables
   - return a metadata-only provider when the declared table engine is 
unavailable
   - expose the real table schema to DataFusion metadata queries
   - keep scans and writes fail-closed with the existing `no table engine is 
registered` error
   - cover `SHOW TABLES`, unrelated and external `information_schema.columns` 
queries, direct reads, and destructive statements
   
   ## Behavior
   
   | Operation | Unregistered external table |
   | --- | --- |
   | `SHOW TABLES` | succeeds and lists the table |
   | `information_schema.columns` | succeeds using catalog metadata |
   | direct `SELECT` | fails when the scan is planned |
   | writes | fail closed |
   
   ## Tests
   
   - `cargo fmt --all -- --check`
   - `cargo clippy --locked -p paimon -p paimon-datafusion --all-targets -- -D 
warnings`
   - `cargo test -p paimon --lib catalog`
   - `cargo test -p paimon-datafusion --test table_type_routing`
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to