beliefer commented on a change in pull request #29800:
URL: https://github.com/apache/spark/pull/29800#discussion_r513133285
##########
File path:
sql/core/src/main/scala/org/apache/spark/sql/execution/window/WindowExec.scala
##########
@@ -57,8 +57,12 @@ import org.apache.spark.sql.types.{CalendarIntervalType,
DateType, IntegerType,
* 3. CURRENT ROW AND 1 FOLLOWING
* 4. 1 PRECEDING AND 1 FOLLOWING
* 5. 1 FOLLOWING AND 2 FOLLOWING
- * - Offset frame: The frame consist of one row, which is an offset number of
rows away from the
- * current row. Only [[OffsetWindowFunction]]s can be processed in an offset
frame.
+ * - Offset frame: The frame consist of one row, which is an offset number of
rows. There are three
+ * implement of offset frame.
+ * 1. [[FrameLessOffsetWindowFunction]] returns the value of the input
column offset by a number
+ * of rows according to the current row.
+ * 2. [[UnboundedOffsetWindowFunctionFrame]] and
[[UnboundedPrecedingOffsetWindowFunctionFrame]]
+ * returns the value of the input column offset by a number of rows
within the partition.
Review comment:
Thanks!
----------------------------------------------------------------
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]
