[ https://issues.apache.org/jira/browse/BEAM-1425?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15947823#comment-15947823 ]
ASF GitHub Bot commented on BEAM-1425: -------------------------------------- GitHub user jkff opened a pull request: https://github.com/apache/beam/pull/2362 [BEAM-1425] Window should comply with PTransform style guide Incompatible changes: - Window.Bound class is now simply Window: matters for users that were extracting the transform into a variable. - Static methods such as Window.triggering(), Window.withAllowedLateness() etc. are now available via Window.configure() - e.g. Window.<String>configure().withAllowedLateness(...). The method Window.into() is left intact as it is the primary entry point for windowing a collection, whereas the other methods are for adjusting the parameters of the windowing function. R: @tgroh You can merge this pull request into a Git repository by running: $ git pull https://github.com/jkff/incubator-beam window-style Alternatively you can review and apply these changes as the patch at: https://github.com/apache/beam/pull/2362.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 #2362 ---- commit 100b65f5b5203e664386192bc390c34a51a8005b Author: Eugene Kirpichov <kirpic...@google.com> Date: 2017-03-29T00:59:11Z Removes unused name parameter commit f981f40e9f7ec48663dfa692bf44ca680cf2528d Author: Eugene Kirpichov <kirpic...@google.com> Date: 2017-03-29T01:04:37Z Fixes a few warnings in Window commit 9a0fa6617d929ff626bb292232327f79597d241d Author: Eugene Kirpichov <kirpic...@google.com> Date: 2017-03-29T01:14:30Z Uses AutoValue in Window commit 9de5bb2fde66fc0738aec770fcfc1846fcd2be95 Author: Eugene Kirpichov <kirpic...@google.com> Date: 2017-03-29T19:58:20Z Replaced static Window.blah() methods with Window.configure().blah() except Window.into() commit 0ec3afc5530df0e5d1f7bbba4093ef16be8e3657 Author: Eugene Kirpichov <kirpic...@google.com> Date: 2017-03-29T20:09:49Z Replaces Window.Bound with simply Window ---- > Window should comply with PTransform style guide > ------------------------------------------------ > > Key: BEAM-1425 > URL: https://issues.apache.org/jira/browse/BEAM-1425 > Project: Beam > Issue Type: Bug > Components: sdk-java-core > Reporter: Eugene Kirpichov > Labels: backward-incompatible, starter > Fix For: First stable release > > > Suggested changes: > - Remove static builder-like methods such as triggering(), > discardingFiredPanes() - the only static entry point should be .into(). > - (optional) use AutoValue -- This message was sent by Atlassian JIRA (v6.3.15#6346)