Github user rednaxelafx commented on a diff in the pull request:
https://github.com/apache/spark/pull/22355#discussion_r216524666
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/package.scala
---
@@ -86,24 +86,12 @@ package object expressions {
}
/**
- * Converts a [[InternalRow]] to another Row given a sequence of
expression that define each
- * column of the new row. If the schema of the input row is specified,
then the given expression
- * will be bound to that schema.
- *
- * In contrast to a normal projection, a MutableProjection reuses the
same underlying row object
- * each time an input row is added. This significantly reduces the cost
of calculating the
- * projection, but means that it is not safe to hold on to a reference
to a [[InternalRow]] after
- * `next()` has been called on the [[Iterator]] that produced it.
Instead, the user must call
- * `InternalRow.copy()` and hold on to the returned [[InternalRow]]
before calling `next()`.
+ * A helper function to bound given expressions to an input schema.
--- End diff --
Spelling nitpick: s/bound/bind/
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
