[
https://issues.apache.org/jira/browse/BEAM-2722?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16114798#comment-16114798
]
ASF GitHub Bot commented on BEAM-2722:
--------------------------------------
GitHub user XuMingmin opened a pull request:
https://github.com/apache/beam/pull/3689
[BEAM-2722] Remove redundant windowing information from the BeamRecord
itself
R: @robertwb
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/XuMingmin/beam BEAM-2722
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/beam/pull/3689.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #3689
----
commit c0b1fed1c7687a1e44e8ee6997be40aa1785ea51
Author: mingmxu <[email protected]>
Date: 2017-08-04T18:12:45Z
Remove redundant windowing information from the BeamRecord itself
element window information `BoundedWindow` is added in `BeamSqlExpression`.
----
> Remove redundant windowing information from the BeamRecord itself
> -----------------------------------------------------------------
>
> Key: BEAM-2722
> URL: https://issues.apache.org/jira/browse/BEAM-2722
> Project: Beam
> Issue Type: Bug
> Components: dsl-sql, sdk-java-extensions
> Reporter: Robert Bradshaw
> Assignee: Xu Mingmin
> Labels: dsl_sql_merge
>
> We should alway be extracting it from the ambient Window.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
