[GitHub] [spark] cloud-fan commented on pull request #32365: [SPARK-35228][SQL] Add expression ToPrettyString for keep consistent between hive/spark format in df.show and transform

[GitBox <gi...@apache.org>]

cloud-fan commented on pull request #32365:
URL: https://github.com/apache/spark/pull/32365#issuecomment-867636282


   It's good to be consistent, but we also need to make a decision that we want to change `df.show` to match the Hive format.
   
   For TRANSFORM, we have to match Hive as TRANSFORM is a Hive feature. For spark-sql shell, I don't think we have to but the ship has already sailed. For `df.show`, I think it makes sense to match the spark-sql shell, but I'd like to hear more opinions. cc @dongjoon-hyun @viirya @maropu @yaooqinn @HyukjinKwon (how about `df.show` in pyspark?)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org