leanken commented on a change in pull request #30097:
URL: https://github.com/apache/spark/pull/30097#discussion_r511708600



##########
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:
        -Phadoop-3.2 -Phive-2.3 -Phadoop-cloud -Pkinesis-asl 
-Pspark-ganglia-lgpl -Pyarn -Pkubernetes -Phive -Pmesos -Phive-thriftserver 
unidoc




----------------------------------------------------------------
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

Reply via email to