Hi, also, I think the proposed changes are very good. I really like the clear separation of concerns imposed by the updated design.
Speaking for the Flink runner, I think it should be very easy to migrate to the new scheme as long as something like the ReduceFnRunner is maintained for dealing with the low-level details of windows/triggers. Also, it's not mentioned in the doc, but transformation to a runner-specific job will still be handled by Pipeline.PipelineVisitor, right? Just wanted to make sure there. Cheers, Aljoscha On Sat, 16 Apr 2016 at 07:43 Aljoscha Krettek <aljos...@apache.org> wrote: > Hi, > is there already a design doc on moving the triggers to be > cross-language/a syntax tree. It is mentioned in one line in this doc and > in passing in some other Jira issues and pull requests/commits. I'm quite > interested in how this would go. It would also not allow user-written > triggers anymore, correct? > > Cheers, > Aljoscha > > On Thu, 24 Mar 2016 at 07:41 Jean-Baptiste Onofré <j...@nanthrax.net> wrote: > >> Hi Kenn, >> >> thanks for the update. I'm reading it now. >> >> Regards >> JB >> >> On 03/23/2016 10:17 PM, Kenneth Knowles wrote: >> > Hi everyone, >> > >> > Incorporating the feedback from the 1-pager I circulated a week ago, I >> have >> > put together a concrete design document for the new API(s). >> > >> > >> https://docs.google.com/document/d/1bao-5B6uBuf-kwH1meenAuXXS0c9cBQ1B2J59I3FiyI/edit?usp=sharing >> > >> > I appreciate any and all feedback on the design. >> > >> > Kenn >> > >> >> -- >> Jean-Baptiste Onofré >> jbono...@apache.org >> http://blog.nanthrax.net >> Talend - http://www.talend.com >> >