sort by keys

apache · Oct 31, 2021 · 8da0022 · 8da0022
1 parent 988187c
commit 8da0022
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 7 deletions.
diff --git a/sql/core/src/main/scala/org/apache/spark/sql/execution/command/ddl.scala b/sql/core/src/main/scala/org/apache/spark/sql/execution/command/ddl.scala
@@ -191,7 +191,7 @@ case class DescribeDatabaseCommand(
         if (properties.isEmpty) {
           ""
         } else {
-          properties.toSeq.mkString("(", ", ", ")")
+          properties.toSeq.sortBy(_._1).mkString("(", ", ", ")")
         }
       result :+ Row("Properties", propertiesStr)
     } else {

diff --git a/.../src/main/scala/org/apache/spark/sql/execution/datasources/v2/DescribeNamespaceExec.scala b/.../src/main/scala/org/apache/spark/sql/execution/datasources/v2/DescribeNamespaceExec.scala
@@ -44,12 +44,13 @@ case class DescribeNamespaceExec(
     }
 
     if (isExtended) {
-      val properties = metadata.asScala.toMap -- CatalogV2Util.NAMESPACE_RESERVED_PROPERTIES
-      val propertiesStr = if (properties.isEmpty) {
-        ""
-      } else {
-        properties.toSeq.mkString("(", ", ", ")")
-      }
+      val properties = metadata.asScala -- CatalogV2Util.NAMESPACE_RESERVED_PROPERTIES
+      val propertiesStr =
+        if (properties.isEmpty) {
+          ""
+        } else {
+          properties.toSeq.sortBy(_._1).mkString("(", ", ", ")")
+        }
       rows += toCatalystRow("Properties", propertiesStr)
     }
     rows.toSeq