+1
Regards
JB
On 04/21/2017 07:24 PM, Kenneth Knowles wrote:
Hi all,
I propose that we remove KeyedCombineFn before the first stable release.
I don't think it adds enough value for the complexity it adds to e.g.
CombineWithContext [1] and state [2, 3], and it doesn't seem to me that
users
Good point, when you start talking about anything less than a full join,
triggers get involved to describe how one actually achieves the desired
semantics, and they may end up being tied to just one of the inputs (e.g.,
you may only care about the watermark for one side of the join). Am
expecting
All of your plans sound good. Private constructors, yes. Just one Trigger
subclass, yes. I don't have strong feelings about the naming here. It could
be just AfterWatermark or AfterWatermark.PastEndOfWindow.
The conversion from Trigger to TriggerStateMachine goes through
beam_runner_api.proto
There's something to be said about having different triggering depending on
which side of a join data comes from, perhaps?
(delightful doc, as usual)
Kenn
On Fri, Apr 21, 2017 at 1:33 PM, Tyler Akidau
wrote:
> Thanks for reading, Luke. The simple answer is that
Thanks for reading, Luke. The simple answer is that CoGBK is basically
flatten + GBK. Flatten is a non-grouping operation that merges the input
streams into a single output stream. GBK then groups the data within that
single union stream as you might otherwise expect, yielding a single table.
So I
The doc is a good read.
I think you do a great job of explaining table -> stream, stream -> stream,
and stream -> table when there is only one stream.
But when there are multiple streams reading/writing to a table, how does
that impact what occurs?
For example, with CoGBK you have multiple
+1, as I’m almost always in favour of simplification
> On 21. Apr 2017, at 19:59, Robert Bradshaw
> wrote:
>
> Strongly in favor of removing this. If it's actually needed one can
> incorporate the key into the value for inspection in the various
> phases of the
Strongly in favor of removing this. If it's actually needed one can
incorporate the key into the value for inspection in the various
phases of the CombineFn, so it's no loss of expressiveness. It's
perfectly reasonable to make this (rare) usecase more complicated to
greatly simplify the common
+1
On Fri, Apr 21, 2017 at 10:53 AM Thomas Groh
wrote:
> A happy +1. This simplifies the code base, and if we find a compelling use,
> it shouldn't be too bad to add it back in.
>
> On Fri, Apr 21, 2017 at 10:24 AM, Kenneth Knowles
> wrote:
>
A happy +1. This simplifies the code base, and if we find a compelling use,
it shouldn't be too bad to add it back in.
On Fri, Apr 21, 2017 at 10:24 AM, Kenneth Knowles
wrote:
> Hi all,
>
> I propose that we remove KeyedCombineFn before the first stable release.
>
> I
+1 -- this is a good simplification.
On Fri, Apr 21, 2017 at 10:24 AM, Kenneth Knowles
wrote:
> Hi all,
>
> I propose that we remove KeyedCombineFn before the first stable release.
>
> I don't think it adds enough value for the complexity it adds to e.g.
>
Hi all,
I propose that we remove KeyedCombineFn before the first stable release.
I don't think it adds enough value for the complexity it adds to e.g.
CombineWithContext [1] and state [2, 3], and it doesn't seem to me that
users really use it when we might expect. I am happy to be demonstrated
12 matches
Mail list logo