cloud-fan commented on a change in pull request #31196:
URL: https://github.com/apache/spark/pull/31196#discussion_r559337333
##########
File path:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/LogicalRelation.scala
##########
@@ -64,7 +64,8 @@ case class LogicalRelation(
}
override def simpleString(maxFields: Int): String = {
- s"Relation[${truncatedString(output, ",", maxFields)}] $relation"
+ s"Relation ${catalogTable.map(_.identifier.unquotedString).getOrElse("")}"
+
+ s"[${truncatedString(output, ",", maxFields)}] $relation"
Review comment:
There are 2 "names" for v2 tables: the name to look up it and the name
`Table` instance returns. It's possible that people use `tbl` to look up a v2
table and `Table.name` returns a file path. I think we should show both.
----------------------------------------------------------------
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]
