[jira] [Comment Edited] (BEAM-13672) Window.into() without a windowFn not correctly translated to portable representation

["Steve Niemitz (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/BEAM-13672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17479562#comment-17479562 ] 

Steve Niemitz edited comment on BEAM-13672 at 1/20/22, 6:32 PM:
----------------------------------------------------------------

ah ok.  it's possible this was just a side-effect of /b/214613604 and not actually an issue translating the pipeline.

(that actually might not be the internal ID, it's linked in the doc you were looking at though too)


was (Author: steveniemitz):
ah ok.  it's possible this was just a side-effect of /b/214613604 and not actually an issue translating the pipeline.

> Window.into() without a windowFn not correctly translated to portable representation
> ------------------------------------------------------------------------------------
>
>                 Key: BEAM-13672
>                 URL: https://issues.apache.org/jira/browse/BEAM-13672
>             Project: Beam
>          Issue Type: Bug
>          Components: sdk-java-core
>    Affects Versions: 2.35.0
>            Reporter: Steve Niemitz
>            Priority: P1
>
> On the "classic" non-portable runner, you can do something like:
> {{Window.configure().discardingFiredPanes().withAllowedLateness(maxFinalJitter)}}
> to configure the max allowed lateness on a PCollection.  This translates into a Flatten transform [1].  It doesn't seem like the FlattenTranslator uses this information anywhere [2], and the result is that the configuration is lost when translating to a portable pipeline.
> [1] [https://github.com/apache/beam/blob/master/sdks/java/core/src/main/java/org/apache/beam/sdk/transforms/windowing/Window.java#L403]
>  
> [2] https://github.com/apache/beam/blob/master/runners/core-construction-java/src/main/java/org/apache/beam/runners/core/construction/FlattenTranslator.java#L53



--
This message was sent by Atlassian Jira
(v8.20.1#820001)