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