You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@spark.apache.org by "Yihong He (Jira)" <ji...@apache.org> on 2025/04/24 11:40:00 UTC
[jira] [Updated] (SPARK-51899) Implement an error allowlist for spark.catalog.listTables()
[ https://issues.apache.org/jira/browse/SPARK-51899?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Yihong He updated SPARK-51899:
------------------------------
Description:
Swallowing all non-fatal errors and unconditionally returning partial results can lead to inconsistent outcomes, especially in cases involving transient errors (e.g., a table may return full results in {{{}listTables(){}}}, but then return partial results in a subsequent call due to a transient error).
Additionally, the {{tableType}} field in the {{Table}} class is not nullable. While it is technically possible to set {{tableType}} to {{{}null{}}}, doing so violates the expectations of the {{Table}} class. This may surprise people.
To address these issues, we can implement an error allowlist based on SQL states for non-transient errors and avoid returning results for those tables, rather than returning partial results when exceptions occur.
was:
Swallowing all non-fatal errors and unconditionally returning partial results can lead to inconsistent outcomes, especially in cases involving transient errors (e.g., a table may return full results in {{{}listTables(){}}}, but then return partial results in a subsequent call due to a transient error).
Additionally, the {{tableType}} field in the {{Table}} class is not nullable. While it is technically possible to set {{tableType}} to {{{}null{}}}, doing so violates the expectations of the {{Table}} class. This may surprise people.
To address these issues, we can implement an exception allowlist based on SQL states for non-transient errors and avoid returning results for those tables, rather than returning partial results when exceptions occur.
Summary: Implement an error allowlist for spark.catalog.listTables() (was: Implement an exception allowlist for spark.catalog.listTables())
> Implement an error allowlist for spark.catalog.listTables()
> -----------------------------------------------------------
>
> Key: SPARK-51899
> URL: https://issues.apache.org/jira/browse/SPARK-51899
> Project: Spark
> Issue Type: Improvement
> Components: Connect
> Affects Versions: 4.1.0
> Reporter: Yihong He
> Priority: Minor
>
> Swallowing all non-fatal errors and unconditionally returning partial results can lead to inconsistent outcomes, especially in cases involving transient errors (e.g., a table may return full results in {{{}listTables(){}}}, but then return partial results in a subsequent call due to a transient error).
> Additionally, the {{tableType}} field in the {{Table}} class is not nullable. While it is technically possible to set {{tableType}} to {{{}null{}}}, doing so violates the expectations of the {{Table}} class. This may surprise people.
> To address these issues, we can implement an error allowlist based on SQL states for non-transient errors and avoid returning results for those tables, rather than returning partial results when exceptions occur.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@spark.apache.org
For additional commands, e-mail: issues-help@spark.apache.org