You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by cloud-fan <gi...@git.apache.org> on 2017/11/16 09:58:20 UTC
[GitHub] spark pull request #19757: [SPARK-22529] [SQL] Relation stats should be cons...
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19757#discussion_r151366943
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/SetCommand.scala ---
@@ -95,6 +95,10 @@ case class SetCommand(kv: Option[(String, Option[String])]) extends RunnableComm
s"prefix spark.hadoop (e.g., spark.hadoop.$key) when starting a Spark application. " +
"For details, see the link: https://spark.apache.org/docs/latest/configuration.html#" +
"dynamically-loading-spark-properties.")
+ } else if (key == SQLConf.CBO_ENABLED.key &&
+ value != sparkSession.conf.get(SQLConf.CBO_ENABLED.key)) {
+ // To change CBO config, first we need to invalidate stats for all relation caches.
--- End diff --
what about the stats in hive metastore?
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org