pan3793 commented on code in PR #55510:
URL: https://github.com/apache/spark/pull/55510#discussion_r3135817324
##########
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/WriteToDataSourceV2Exec.scala:
##########
@@ -320,7 +323,8 @@ case class ReplaceDataExec(
refreshCache: () => Unit,
projections: ReplaceDataProjections,
write: Write,
- rowLevelCommand: RowLevelOperation.Command) extends RowLevelWriteExec {
+ rowLevelCommand: RowLevelOperation.Command,
+ tableName: String) extends RowLevelWriteExec {
Review Comment:
I wonder if we should define a method in `o.a.s.sql.connector.write.Write`
as a contract to allow the connector to report the table name or something else
they want to display as part of the node name, this way, we can display more
things, i.e., `ReplaceData iceberg cat.ns.t (CoW)`
@aokolnychyi do you have any suggestions?
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
