Re: Description of Flink event time processing

2018-08-02 Thread Elias Levy
Fabian, https://github.com/apache/flink/pull/6481 I added a page, but did not remove or edit any existing page. Let me know what you'd like to see trimmed. On Thu, Aug 2, 2018 at 8:44 AM Fabian Hueske wrote: > Hi Elias, > > Thanks for the update! > I think the document can be added to the

Re: Description of Flink event time processing

2018-08-02 Thread Fabian Hueske
Hi Elias, Thanks for the update! I think the document can be added to the docs now. It has some overlap with the Event Time Overview page. IMO, it should not replace the overview page but rather be a new page. Maybe, we can make the overview a bit slimmer and point to the more detailed

Re: Description of Flink event time processing

2018-07-31 Thread Fabian Hueske
Hi Elias, Sorry for the delay. I just made a pass over the document. I think it is very good. Let's have a look where this fits best into the docs and check if there is duplicate content on other pages that should be removed / reorganized. Best, Fabian 2018-07-31 3:17 GMT+02:00 Elias Levy : >

Re: Description of Flink event time processing

2018-07-30 Thread Elias Levy
Fabian, You have any time to review the changes? On Thu, Jul 19, 2018 at 2:19 AM Fabian Hueske wrote: > Hi Elias, > > Thanks for the update! > I'll try to have another look soon. > > Best, Fabian > > 2018-07-11 1:30 GMT+02:00 Elias Levy : > >> Thanks for all the comments. I've updated the

Re: Description of Flink event time processing

2018-07-19 Thread Fabian Hueske
Hi Elias, Thanks for the update! I'll try to have another look soon. Best, Fabian 2018-07-11 1:30 GMT+02:00 Elias Levy : > Thanks for all the comments. I've updated the document to account for the > feedback. Please take a look. > > On Fri, Jul 6, 2018 at 2:33 PM Elias Levy > wrote: > >>

Re: Description of Flink event time processing

2018-07-10 Thread Elias Levy
Thanks for all the comments. I've updated the document to account for the feedback. Please take a look. On Fri, Jul 6, 2018 at 2:33 PM Elias Levy wrote: > Apologies. Comments are now enabled. > > On Thu, Jul 5, 2018 at 6:09 PM Rong Rong wrote: > >> Hi Elias, >> >> Thanks for putting

Re: Description of Flink event time processing

2018-07-06 Thread Elias Levy
Apologies. Comments are now enabled. On Thu, Jul 5, 2018 at 6:09 PM Rong Rong wrote: > Hi Elias, > > Thanks for putting together the document. This is actually a very good, > well-rounded document. > I think you did not to enable access for comments for the link. Would you > mind enabling

Re: Description of Flink event time processing

2018-07-05 Thread Rong Rong
Hi Elias, Thanks for putting together the document. This is actually a very good, well-rounded document. I think you did not to enable access for comments for the link. Would you mind enabling comments for the google doc? Thanks, Rong On Thu, Jul 5, 2018 at 8:39 AM Fabian Hueske wrote: > Hi

Re: Description of Flink event time processing

2018-07-05 Thread Fabian Hueske
Hi Elias, Thanks for the great document! I made a pass over it and left a few comments. I think we should definitely add this to the documentation. Thanks, Fabian 2018-07-04 10:30 GMT+02:00 Fabian Hueske : > Hi Elias, > > I agree, the docs lack a coherent discussion of event time features. >

Re: Description of Flink event time processing

2018-07-04 Thread Fabian Hueske
Hi Elias, I agree, the docs lack a coherent discussion of event time features. Thank you for this write up! I just skimmed your document and will provide more detailed feedback later. It would be great to add such a page to the documentation. Best, Fabian 2018-07-03 3:07 GMT+02:00 Elias Levy :

Description of Flink event time processing

2018-07-02 Thread Elias Levy
The documentation of how Flink handles event time and watermarks is spread across several places. I've been wanting a single location that summarizes the subject, and as none was available, I wrote one up. You can find it here: