[
https://issues.apache.org/jira/browse/BEAM-14303?focusedWorklogId=756757&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-756757
]
ASF GitHub Bot logged work on BEAM-14303:
-----------------------------------------
Author: ASF GitHub Bot
Created on: 13/Apr/22 22:42
Start Date: 13/Apr/22 22:42
Worklog Time Spent: 10m
Work Description: reuvenlax commented on code in PR #17359:
URL: https://github.com/apache/beam/pull/17359#discussion_r849952642
##########
sdks/java/core/src/main/java/org/apache/beam/sdk/state/Timer.java:
##########
@@ -99,6 +99,12 @@ public interface Timer {
*/
Timer withOutputTimestamp(Instant outputTime);
+ /**
+ * Asserts that there is no output timestamp. The output watermark will not
be held up, and it is
+ * illegal to output messages from this timer.
Review Comment:
To be precise, illegal to output messages using the default timestamp
(outputWithTimestamp is still allowed).
##########
runners/core-java/src/main/java/org/apache/beam/runners/core/SimpleDoFnRunner.java:
##########
@@ -1251,7 +1262,7 @@ private void setAndVerifyOutputTimestamp() {
:
PeriodFormat.getDefault().print(fn.getAllowedTimestampSkew().toPeriod()),
BoundedWindow.TIMESTAMP_MAX_VALUE));
}
- } else if (TimeDomain.EVENT_TIME.equals(spec.getTimeDomain())) {
+ } else if (!noOutputTimestamp &&
TimeDomain.EVENT_TIME.equals(spec.getTimeDomain())) {
Review Comment:
This was a bug - good catch.
Issue Time Tracking
-------------------
Worklog Id: (was: 756757)
Time Spent: 2h (was: 1h 50m)
> Allow a timer to have no watermark hold
> ---------------------------------------
>
> Key: BEAM-14303
> URL: https://issues.apache.org/jira/browse/BEAM-14303
> Project: Beam
> Issue Type: Improvement
> Components: sdk-java-core
> Reporter: Reuven Lax
> Priority: P2
> Time Spent: 2h
> Remaining Estimate: 0h
>
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
