[GitHub] [spark] cloud-fan commented on pull request #38687: [SPARK-41154][SQL] Incorrect relation caching for queries with time travel spec

2022-11-21 Thread GitBox
cloud-fan commented on PR #38687: URL: https://github.com/apache/spark/pull/38687#issuecomment-1321827968 it has conflicts with 3.3, @ulysses-you can you create a backport PR? thanks! -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [spark] cloud-fan commented on pull request #38687: [SPARK-41154][SQL] Incorrect relation caching for queries with time travel spec

2022-11-21 Thread GitBox
cloud-fan commented on PR #38687: URL: https://github.com/apache/spark/pull/38687#issuecomment-1321822247 thanks, merging to master/3.3! -- 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

[GitHub] [spark] cloud-fan commented on pull request #38687: [SPARK-41154][SQL] Incorrect relation caching for queries with time travel spec

2022-11-20 Thread GitBox
cloud-fan commented on PR #38687: URL: https://github.com/apache/spark/pull/38687#issuecomment-1321552064 Oh wait a minute. Due to the spark catalog extension (set via `spark.sql.catalog.spark_catalog`), we can have tables supporting time travel in the v1 catalog as well. I think

[GitHub] [spark] cloud-fan commented on pull request #38687: [SPARK-41154][SQL] Incorrect relation caching for queries with time travel spec

2022-11-20 Thread GitBox
cloud-fan commented on PR #38687: URL: https://github.com/apache/spark/pull/38687#issuecomment-1321550782 thanks, merging to master/3.3! -- 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

[GitHub] [spark] cloud-fan commented on pull request #38687: [SPARK-41154][SQL] Incorrect relation caching for queries with time travel spec

2022-11-18 Thread GitBox
cloud-fan commented on PR #38687: URL: https://github.com/apache/spark/pull/38687#issuecomment-1320010391 there is another cache in `SessionCatalog.tableRelationCache`, shall we update it as well? -- This is an automated message from the Apache Git Service. To respond to the message,