Github user dilipbiswal commented on a diff in the pull request:

    https://github.com/apache/spark/pull/15423#discussion_r82720521
  
    --- Diff: 
sql/core/src/test/scala/org/apache/spark/sql/SQLQueryTestSuite.scala ---
    @@ -207,6 +208,7 @@ class SQLQueryTestSuite extends QueryTest with 
SharedSQLContext {
         // Returns true if the plan is supposed to be sorted.
         def isSorted(plan: LogicalPlan): Boolean = plan match {
           case _: Join | _: Aggregate | _: Generate | _: Sample | _: Distinct 
=> false
    +      case _: ShowColumnsCommand => true
    --- End diff --
    
    @viirya So it seemed odd to have the generated output files to have column 
names sorted which didn't reflect the columns. In the test case i had the table 
create like following.
    ```SQL
    CREATE TABLE showcolumn2 (price int, qty int) partitioned by (year int, 
month int)
    ```
    It seemed odd to me to have the generated output file report the columns as 
month, price, qty and year as opposed to price, qty, year and month.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to