[
https://issues.apache.org/jira/browse/BEAM-9430?focusedWorklogId=406364&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-406364
]
ASF GitHub Bot logged work on BEAM-9430:
----------------------------------------
Author: ASF GitHub Bot
Created on: 19/Mar/20 16:18
Start Date: 19/Mar/20 16:18
Worklog Time Spent: 10m
Work Description: mxm commented on pull request #11126: [BEAM-9430,
BEAM-2939] Migrate from ProcessContext#updateWatermark to WatermarkEstimators
URL: https://github.com/apache/beam/pull/11126#discussion_r395145834
##########
File path:
sdks/java/core/src/main/java/org/apache/beam/sdk/transforms/DoFn.java
##########
@@ -265,18 +265,6 @@
* data has been explicitly requested. See {@link Window} for more
information.
*/
public abstract PaneInfo pane();
-
- /**
- * Gives the runner a (best-effort) lower bound about the timestamps of
future output associated
- * with the current element.
- *
- * <p>If the {@link DoFn} has multiple outputs, the watermark applies to
all of them.
- *
- * <p>Only splittable {@link DoFn DoFns} are allowed to call this method.
It is safe to call
- * this method from a different thread than the one running {@link
ProcessElement}, but all
- * calls must finish before {@link ProcessElement} returns.
- */
- public abstract void updateWatermark(Instant watermark);
Review comment:
This is a backwards-incompatible change. It probably does not affect many
users but we should make sure to announce it, e.g. via the release notes.
----------------------------------------------------------------
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]
Issue Time Tracking
-------------------
Worklog Id: (was: 406364)
Time Spent: 1h 40m (was: 1.5h)
> Migrate from ProcessContext#updateWatermark to WatermarkEstimators
> ------------------------------------------------------------------
>
> Key: BEAM-9430
> URL: https://issues.apache.org/jira/browse/BEAM-9430
> Project: Beam
> Issue Type: Sub-task
> Components: sdk-java-core
> Reporter: Luke Cwik
> Assignee: Luke Cwik
> Priority: Major
> Time Spent: 1h 40m
> Remaining Estimate: 0h
>
> Current discussion underway in
> [https://lists.apache.org/thread.html/r5d974b6a58bc04ff4c02682fda4ef68608121f1bf23a86e9d592ca6e%40%3Cdev.beam.apache.org%3E]
>
> Proposed API: [https://github.com/apache/beam/pull/10992]
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
