HeartSaVioR commented on code in PR #45709:
URL: https://github.com/apache/spark/pull/45709#discussion_r1540446675
##########
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/TimerStateImpl.scala:
##########
@@ -188,9 +187,12 @@ class TimerStateImpl(
/**
* Function to get all the registered timers for all grouping keys
+ * @param expiryTimestampMs Threshold for expired timestamp in milliseconds,
this function
Review Comment:
This class is not user-facing, right? If it is, I'd suggest avoiding
implementation detail. Looks like as it doesn't seem to be an user facing, but
just to remind.
--
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]
