leoyvens opened a new pull request, #10876:
URL: https://github.com/apache/datafusion/pull/10876

   ## Which issue does this PR close?
   
   Closes #10875.
   
   ## Rationale for this change
   
   This logic is extracted from `SessionState::resolve_table_references`, but 
can be used without a context making it more flexible as a helper function for 
catalog management.
   
   ## What changes are included in this PR?
   
   The `resolve_table_references` function will only return the tables from the 
information schema if it actually implicitly requires them due to a `SHOW` 
statement.
   
   A new public function `catalog::resolve_table_references` is added, 
containing the same logic as `SessionState::resolve_table_references` but in a 
more flexible interface.
   
   ## Are these changes tested?
   
   Yes there is a doctest and it's well covered by sqllogictests as it's 
executed as part of SQL planning.
   
   ## Are there any user-facing changes?
   
   A new public function is added.
   
   The behaviour of `SessionState::resolve_table_references` around the 
information schema is changed, which could theoretically break users relying on 
that directly, but it's quite an edge case.


-- 
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: github-unsubscr...@datafusion.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org

Reply via email to