William1104 opened a new pull request #24168: Feature/spark 27062 URL: https://github.com/apache/spark/pull/24168 ## What changes were proposed in this pull request? If we refresh a cached table, the table cache will be first uncached and then recache (lazily). Currently, the logic is embedded in CatalogImpl.refreshTable method. The current implementation does not save the cache name and storage level. As a result, cache name and cache level could be changed after a REFERSH. IMHO, it is not what user expected. Therefore, this PR would like to fix this behavior by first save the cache name and storage level for recaching the table. ## How was this patch tested? Two new unit tests are added to "CachedTableSuite". To make sure cache name is not changed after table refreshing. Those test would fail without patching CatalogImpl with this PR.
---------------------------------------------------------------- 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]
