Re: [ANNOUNCEMENT] New committers!

2016-10-21 Thread Jean-Baptiste Onofré
Just a small thing. If it's not already done, don't forget to sign a ICLA and let us know your apache ID. Thanks, Regards JB On 10/22/2016 12:18 AM, Davor Bonaci wrote: Hi everyone, Please join me and the rest of Beam PPMC in welcoming the following contributors as our newest committers.

Re: [DISCUSS] Deferring (pre) combine for merging windows.

2016-10-21 Thread Robert Bradshaw
Combine.perKey() is defined as GroupByKey() | Combine.values(). A runner is free, in fact encouraged, to take advantage of the associative properties of CombineFn to compute the result of GroupByKey() | Combine.values() as cheaply as possible, but it is incorrect to produce something that could

Re: [ANNOUNCEMENT] New committers!

2016-10-21 Thread Jesse Anderson
Thanks for the welcomes everyone! On Fri, Oct 21, 2016 at 4:02 PM Mark Liu wrote: > Congrats for all of you! > > Mark > > On Fri, Oct 21, 2016 at 3:34 PM, Kenneth Knowles > wrote: > > > Huzzah! > > > > I've personally enjoyed working

Re: [ANNOUNCEMENT] New committers!

2016-10-21 Thread Mark Liu
Congrats for all of you! Mark On Fri, Oct 21, 2016 at 3:34 PM, Kenneth Knowles wrote: > Huzzah! > > I've personally enjoyed working together, and I am glad to extend this > acknowledgement and welcome this addition to the Beam community. > > Kenn > > On Fri, Oct 21,

Re: [DISCUSS] Deferring (pre) combine for merging windows.

2016-10-21 Thread Amit Sela
Please excuse my typos and apply "s/differ/defer/g" ;-). Amit. On Fri, Oct 21, 2016 at 2:59 PM Amit Sela wrote: > I'd like to raise an issue that was discussed in BEAM-696 > . > I won't recap here because it would be

Re: [ANNOUNCEMENT] New committers!

2016-10-21 Thread Frances Perry
Wonderful to see your contributions recognized ;-) On Fri, Oct 21, 2016 at 3:46 PM, Ahmet Altay wrote: > Congratulations to all of you! > > Ahmet > > On Fri, Oct 21, 2016 at 3:35 PM, Ben Chambers > > wrote: > > > Congrats. +3! > > > > On

Re: [ANNOUNCEMENT] New committers!

2016-10-21 Thread Ahmet Altay
Congratulations to all of you! Ahmet On Fri, Oct 21, 2016 at 3:35 PM, Ben Chambers wrote: > Congrats. +3! > > On Fri, Oct 21, 2016 at 3:34 PM Kenneth Knowles > wrote: > > > Huzzah! > > > > I've personally enjoyed working together, and I

Re: [ANNOUNCEMENT] New committers!

2016-10-21 Thread Neelesh Salian
Congratulations folks! On Fri, Oct 21, 2016 at 3:34 PM, Kenneth Knowles wrote: > Huzzah! > > I've personally enjoyed working together, and I am glad to extend this > acknowledgement and welcome this addition to the Beam community. > > Kenn > > On Fri, Oct 21, 2016 at

Re: [ANNOUNCEMENT] New committers!

2016-10-21 Thread Kenneth Knowles
Huzzah! I've personally enjoyed working together, and I am glad to extend this acknowledgement and welcome this addition to the Beam community. Kenn On Fri, Oct 21, 2016 at 3:18 PM Davor Bonaci wrote: > Hi everyone, > Please join me and the rest of Beam PPMC in welcoming the

[ANNOUNCEMENT] New committers!

2016-10-21 Thread Davor Bonaci
Hi everyone, Please join me and the rest of Beam PPMC in welcoming the following contributors as our newest committers. They have significantly contributed to the project in different ways, and we look forward to many more contributions in the future. * Thomas Weise Thomas authored the Apache

Re: Jenkins build is unstable: beam_PostCommit_RunnableOnService_GoogleCloudDataflow #1388

2016-10-21 Thread Jason Kuster
Filed https://issues.apache.org/jira/browse/BEAM-795 to track On Fri, Oct 21, 2016 at 6:08 AM, Apache Jenkins Server < jenk...@builds.apache.org> wrote: > See GoogleCloudDataflow/1388/> > > -- --- Jason Kuster Apache Beam

[DISCUSS] Deferring (pre) combine for merging windows.

2016-10-21 Thread Amit Sela
I'd like to raise an issue that was discussed in BEAM-696 . I won't recap here because it would be extensive (and probably exhaustive), and I'd also like to restart the discussion here rather then summarize it. *The problem* In the case of (main)

Re: Start of release 0.3.0-incubating

2016-10-21 Thread Maximilian Michels
+1 for the release. We have plenty of fixes in and users have already asked for a new release. -Max On Fri, Oct 21, 2016 at 10:22 AM, Jean-Baptiste Onofré wrote: > Hi Aljoscha, > > OK for me, you can go ahead ;) > > Thanks again to tackle this release ! > > Regards > JB > >

Re: Tracking backward-incompatible changes for Beam

2016-10-21 Thread Jean-Baptiste Onofré
Hi Dan, +1, good idea. Regards JB On 10/21/2016 02:21 AM, Dan Halperin wrote: Hey everyone, In the Beam codebase, we’ve improved, rewritten, or deleted many APIs. While this has improved the model and gives us great freedom to experiment, we are also causing churn on users authoring Beam

Re: Start of release 0.3.0-incubating

2016-10-21 Thread Jean-Baptiste Onofré
Hi Aljoscha, OK for me, you can go ahead ;) Thanks again to tackle this release ! Regards JB On 10/21/2016 08:51 AM, Aljoscha Krettek wrote: +1 @JB We should definitely keep that in mind for the next releases. I think this one is now sufficiently announced so I'll get started on the

Re: Start of release 0.3.0-incubating

2016-10-21 Thread Aljoscha Krettek
+1 @JB We should definitely keep that in mind for the next releases. I think this one is now sufficiently announced so I'll get started on the process. (Which will take me a while since I have to do all the initial setup.) On Fri, 21 Oct 2016 at 06:32 Jean-Baptiste Onofré