address comments

Author: yaooqinn

sql/core/src/main/scala/org/apache/spark/sql/api/python/PythonSQLUtils.scala

@@ -44,14 +44,7 @@ private[sql] object PythonSQLUtils {
     val conf = new SQLConf()
     // Force to build static SQL configurations
     StaticSQLConf
-    // set nondeterministic configurations with general meanings
-    conf.getAllDefinedConfs.map {
-      case p @ (SQLConf.SESSION_LOCAL_TIMEZONE.key, _, _, _) =>
-        p.copy(_2 = "value of local timezone")
-      case p @ (StaticSQLConf.WAREHOUSE_PATH.key, _, _, _) =>
-        p.copy(_2 = "value of $PWD/spark-warehouse")
-      case o => o
-    }
+    conf.getAllDefinedConfs
   }
 
   def listRuntimeSQLConfigs(): Array[(String, String, String, String)] = {

sql/gen-sql-config-docs.py

@@ -81,7 +81,11 @@ def generate_sql_configs_table_html(sql_configs, path):
             """
         ))
         for config in sorted(sql_configs, key=lambda x: x.name):
-            if config.default == "<undefined>":
+            if config.name =="spark.sql.session.timeZone":
+                default = "(value of local timezone)"
+            elif config.name == "spark.sql.warehouse.dir":
+                default = "(value of <code>$PWD/spark-warehouse</code>)"
+            elif config.default == "<undefined>":
                 default = "(none)"
             elif config.default.startswith("<value of "):
                 referenced_config_name = value_reference_pattern.match(config.default).group(1)