yaooqinn commented on a change in pull request #32563:
URL: https://github.com/apache/spark/pull/32563#discussion_r636785978
##########
File path: sql/core/src/test/resources/sql-tests/results/show-tables.sql.out
##########
@@ -120,19 +120,9 @@ show_t3
-- !query
SHOW TABLE EXTENDED LIKE 'show_t*'
-- !query schema
-struct<namespace:string,tableName:string,isTemporary:boolean,information:string>
+struct<namespace:string,tableName:string,isTemporary:boolean,information:map<string,string>>
-- !query output
- show_t3 true Table: show_t3
Review comment:
this is only used in the SQL shell, not by the thrift server. Otherwise,
the JDBC `ResultSet` will get wrong for mapping the metadata and column results.
If we turn on the `spark.sql.cli.print.header=true` for SQL shell
1) w/o this PR, the information column matches the first element of the map,
then EOL. The rest of the map will print line by line, with wrong/no
indentations.
2) w/ this PR, I guess the schema header and the result more inappropriate
matched
--
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:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
