Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21305#discussion_r200639791
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/DataSourceV2Relation.scala
---
@@ -40,17 +44,24 @@ case class DataSourceV2Relation(
source: DataSourceV2,
output: Seq[AttributeReference],
options: Map[String, String],
- userSpecifiedSchema: Option[StructType])
- extends LeafNode with MultiInstanceRelation with
DataSourceV2StringFormat {
+ tableIdent: Option[TableIdentifier] = None,
+ userSpecifiedSchema: Option[StructType] = None)
+ extends LeafNode with MultiInstanceRelation with NamedRelation with
DataSourceV2StringFormat {
import DataSourceV2Relation._
+ override def name: String = {
+ tableIdent.map(_.unquotedString).getOrElse("unknown")
--- End diff --
SGTM
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
