Yikf commented on PR #40437: URL: https://github.com/apache/spark/pull/40437#issuecomment-1489933393
Yes. `hiveResultString` is added to ensure compatibility with hive output. `hiveResultString` is only used by the spark-sql CLI. It is used only as the CLI display. `thriftServer` always outputs as spark's schema, 1. `hiveResultString` is not used for the thrift protocol. 2. spark-sql output from the CLI and `thriftServer` is inconsistent. I'm not sure why spark-sql CLI has to be compatible with hive output, personally, I don't think it's necessary. Maybe we should display spark's schema as is, just like thriftSever? -- 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. To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
