You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2020/06/18 16:45:25 UTC

[GitHub] [spark] sap1ens commented on pull request #28852: [SPARK-30616][SQL] Introduce TTL config option for SQL Metadata Cache

sap1ens commented on pull request #28852:
URL: https://github.com/apache/spark/pull/28852#issuecomment-646160221


   @maropu regarding testing, I don't see a dedicated test suite for the `FileStatusCache`. It's implicitly tested in `HiveSchemaInferenceSuite` though.
   
   Do you think I should create a new suite for `FileStatusCache`? Or try to extend `HiveSchemaInferenceSuite`?


----------------------------------------------------------------
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org