danny0405 commented on a change in pull request #2189:
URL: https://github.com/apache/calcite/pull/2189#discussion_r523864711
##########
File path:
core/src/main/java/org/apache/calcite/rel/metadata/RelMetadataQuery.java
##########
@@ -338,25 +338,23 @@ public Double getPercentageOriginalRows(RelNode rel) {
}
/**
- * Determines the origin of a column, provided the column maps to a single
- * column that isn't derived.
+ * Determines the origin of a column.
*
* @see #getColumnOrigins(org.apache.calcite.rel.RelNode, int)
*
* @param rel the RelNode of the column
* @param column the offset of the column whose origin we are trying to
* determine
*
- * @return the origin of a column provided it's a simple column; otherwise,
- * returns null
+ * @return the origin of a column;
*/
Review comment:
Remove `;`.
----------------------------------------------------------------
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]
