leanken commented on a change in pull request #30097: URL: https://github.com/apache/spark/pull/30097#discussion_r511733174
########## File path: sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/PruneHiveTablePartitions.scala ########## @@ -29,22 +29,9 @@ import org.apache.spark.sql.catalyst.rules.Rule import org.apache.spark.sql.execution.datasources.DataSourceStrategy import org.apache.spark.sql.internal.SQLConf -/** - * Prune hive table partitions using partition filters on [[HiveTableRelation]]. The pruned - * partitions will be kept in [[HiveTableRelation.prunedPartitions]], and the statistics of - * the hive table relation will be updated based on pruned partitions. - * - * This rule is executed in optimization phase, so the statistics can be updated before physical - * planning, which is useful for some spark strategy, eg. - * [[org.apache.spark.sql.execution.SparkStrategies.JoinSelection]]. - * - * TODO: merge this with PruneFileSourcePartitions after we completely make hive as a data source. - */ -private[sql] class PruneHiveTablePartitions(session: SparkSession) Review comment: I was trying to add a empty class definition before object, the unidoc build passed. ``` private[sql] class PruneHiveTablePartitions /** * Prune hive table partitions using partition filters on [[HiveTableRelation]]. The pruned * partitions will be kept in [[HiveTableRelation.prunedPartitions]], and the statistics of * the hive table relation will be updated based on pruned partitions. * * This rule is executed in optimization phase, so the statistics can be updated before physical * planning, which is useful for some spark strategy, eg. * [[org.apache.spark.sql.execution.SparkStrategies.JoinSelection]]. * * TODO: merge this with PruneFileSourcePartitions after we completely make hive as a data source. */ private[sql] object PruneHiveTablePartitions extends Rule[LogicalPlan] with CastSupport with PredicateHelper { ``` ---------------------------------------------------------------- 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: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org