Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/18148#discussion_r119753432 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala --- @@ -126,14 +127,56 @@ class SessionCatalog( } /** - * A cache of qualified table names to table relation plans. - */ + * A cache of qualified table names to table relation plans. + * Accessing tableRelationCache directly is not recommended, + * since it will introduce exposures to guava libraries. + */ val tableRelationCache: Cache[QualifiedTableName, LogicalPlan] = { val cacheSize = conf.tableRelationCacheSize CacheBuilder.newBuilder().maximumSize(cacheSize).build[QualifiedTableName, LogicalPlan]() } /** + * This method provides a way to get a cached plan --- End diff -- The "without exposing components to Guava" part of all these comments is unnecessary.
--- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org