cloud-fan commented on a change in pull request #31666:
URL: https://github.com/apache/spark/pull/31666#discussion_r587661561
##########
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
##########
@@ -979,7 +979,7 @@ class Analyzer(override val catalogManager: CatalogManager)
*
* References to metadata columns are resolved using columns from
[[LogicalPlan.metadataOutput]],
* but the relation's output does not include the metadata columns until the
relation is replaced
- * using [[DataSourceV2Relation.withMetadataColumns()]]. Unless this rule
adds metadata to the
+ * with a copy adding them to the output. Unless this rule adds metadata to
the relation's output,
* relation's output, the analyzer will detect that nothing produces the
columns.
Review comment:
nit: `relation's output,` is repeated twice.
----------------------------------------------------------------
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]
