khakhlyuk commented on code in PR #54422:
URL: https://github.com/apache/spark/pull/54422#discussion_r2839976965
##########
sql/connect/server/src/main/scala/org/apache/spark/sql/connect/planner/SparkConnectPlanner.scala:
##########
@@ -1440,12 +1440,13 @@ class SparkConnectPlanner(
else Deduplicate(allColumns, queryExecution.analyzed)
} else {
val toGroupColumnNames = rel.getColumnNamesList.asScala.toSeq
+ val fieldNames = allColumns.map(_.name).mkString(", ")
Review Comment:
nitty nit: this line can be moved between 1448 and 1449 since it's not used
in the outer scope
--
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]
