cloud-fan commented on issue #24768: [SPARK-27919][SQL] Add v2 session catalog
URL: https://github.com/apache/spark/pull/24768#issuecomment-505260318
 
 
   > and eventually separate out that functionality to an internal class to 
handle it
   
   Isn't `SessionCatalog` the internal class we want? Temp views are 
per-session so this internal class must have one instance per session. Temp 
views may have the same names as tables, and Spark prioritizes temp view over 
tables. `SessionCatalog` will assume an identifier is a temp view and look it 
up. If no temp view is found, assume it's a table and look it up from 
`ExternalCatalog`. If I'm asked to re-design an internal class to do these 
things, I may end up something similar to `SessionCatalog`. We can rename it if 
we think the name doesn't match its functionalities very well.
   
   
   
   

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to