rdblue commented on issue #24768: [SPARK-27919][SQL] Add v2 session catalog URL: https://github.com/apache/spark/pull/24768#issuecomment-509425357 @mccheah, as far as I can tell, it doesn't offer a proposal for alternative behavior. It is just an alternative implementation of what is here. The desired behavior is the same as in this PR, which is why I asked. @cloud-fan, I looked at #25077 and you're proposing the same behavior as implemented in this PR, but with a lot of other changes. I think we agree on what I wrote here: > After this PR, the rules to determine the catalog responsible for an identifier are: > 1. If the identifier starts with a known catalog, use it > 2. If there is a configured default v2 catalog, use that catalog > 3. Otherwise, the session catalog is responsible for the identifier I see that it also handles some cases, like temporary tables, more correctly (`ResolveTables` should ignore temporary tables). But that isn't a change to the proposed behavior, that's a bug fix. What behavior are you proposing should change?
---------------------------------------------------------------- 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]
