[jira] [Commented] (FLINK-12481) Make processing time timer trigger run via the mailbox

2019-09-04 Thread Alex (Jira)


[ 
https://issues.apache.org/jira/browse/FLINK-12481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16922372#comment-16922372
 ] 

Alex commented on FLINK-12481:
--

PR; https://github.com/apache/flink/pull/9564

> Make processing time timer trigger run via the mailbox
> --
>
> Key: FLINK-12481
> URL: https://issues.apache.org/jira/browse/FLINK-12481
> Project: Flink
>  Issue Type: Sub-task
>  Components: Runtime / Task
>Reporter: Stefan Richter
>Assignee: Alex
>Priority: Major
>  Labels: pull-request-available
>  Time Spent: 0.5h
>  Remaining Estimate: 0h
>
> This sub-task integrates the mailbox with processing time timer triggering. 
> Those triggers should now be enqueued as mailbox events and picked up by the 
> stream task's main thread for processing.



--
This message was sent by Atlassian Jira
(v8.3.2#803003)


[jira] [Commented] (FLINK-12481) Make processing time timer trigger run via the mailbox

2019-08-13 Thread Alex (JIRA)


[ 
https://issues.apache.org/jira/browse/FLINK-12481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16905930#comment-16905930
 ] 

Alex commented on FLINK-12481:
--

Blocked by FLINK-12958.

> Make processing time timer trigger run via the mailbox
> --
>
> Key: FLINK-12481
> URL: https://issues.apache.org/jira/browse/FLINK-12481
> Project: Flink
>  Issue Type: Sub-task
>  Components: Runtime / Task
>Reporter: Stefan Richter
>Assignee: Alex
>Priority: Major
>  Labels: pull-request-available
>  Time Spent: 20m
>  Remaining Estimate: 0h
>
> This sub-task integrates the mailbox with processing time timer triggering. 
> Those triggers should now be enqueued as mailbox events and picked up by the 
> stream task's main thread for processing.



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[jira] [Commented] (FLINK-12481) Make processing time timer trigger run via the mailbox

2019-08-08 Thread Biao Liu (JIRA)


[ 
https://issues.apache.org/jira/browse/FLINK-12481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16903548#comment-16903548
 ] 

Biao Liu commented on FLINK-12481:
--

Hi  [~1u0], thanks for feedback. :)

> Make processing time timer trigger run via the mailbox
> --
>
> Key: FLINK-12481
> URL: https://issues.apache.org/jira/browse/FLINK-12481
> Project: Flink
>  Issue Type: Sub-task
>  Components: Runtime / Task
>Reporter: Stefan Richter
>Assignee: Alex
>Priority: Major
>  Labels: pull-request-available
>  Time Spent: 20m
>  Remaining Estimate: 0h
>
> This sub-task integrates the mailbox with processing time timer triggering. 
> Those triggers should now be enqueued as mailbox events and picked up by the 
> stream task's main thread for processing.



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[jira] [Commented] (FLINK-12481) Make processing time timer trigger run via the mailbox

2019-08-08 Thread Alex (JIRA)


[ 
https://issues.apache.org/jira/browse/FLINK-12481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16903104#comment-16903104
 ] 

Alex commented on FLINK-12481:
--

Hi [~SleePy], I've temporally closed the original PR of moving timers to 
mailbox model.

There were some discussions and changes in the underlying mailbox/executor 
implementation (that this effort depends on). The work was also put aside for 
awhile due to common effort for release 1.9.

I'll rebase and adapt my PR to the current {{master}} branch codebase, but the 
migration approach would probably will be very similar to the one proposed in 
the first PR.

> Make processing time timer trigger run via the mailbox
> --
>
> Key: FLINK-12481
> URL: https://issues.apache.org/jira/browse/FLINK-12481
> Project: Flink
>  Issue Type: Sub-task
>  Components: Runtime / Task
>Reporter: Stefan Richter
>Assignee: Alex
>Priority: Major
>  Labels: pull-request-available
>  Time Spent: 20m
>  Remaining Estimate: 0h
>
> This sub-task integrates the mailbox with processing time timer triggering. 
> Those triggers should now be enqueued as mailbox events and picked up by the 
> stream task's main thread for processing.



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[jira] [Commented] (FLINK-12481) Make processing time timer trigger run via the mailbox

2019-08-05 Thread Biao Liu (JIRA)


[ 
https://issues.apache.org/jira/browse/FLINK-12481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16899901#comment-16899901
 ] 

Biao Liu commented on FLINK-12481:
--

Hi [~srichter], I'm just wondering what the latest state of this issue. The 
relevant PR seems to be abandoned. 

> Make processing time timer trigger run via the mailbox
> --
>
> Key: FLINK-12481
> URL: https://issues.apache.org/jira/browse/FLINK-12481
> Project: Flink
>  Issue Type: Sub-task
>  Components: Runtime / Task
>Reporter: Stefan Richter
>Assignee: Alex
>Priority: Major
>  Labels: pull-request-available
>  Time Spent: 20m
>  Remaining Estimate: 0h
>
> This sub-task integrates the mailbox with processing time timer triggering. 
> Those triggers should now be enqueued as mailbox events and picked up by the 
> stream task's main thread for processing.



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[jira] [Commented] (FLINK-12481) Make processing time timer trigger run via the mailbox

2019-05-13 Thread aitozi (JIRA)


[ 
https://issues.apache.org/jira/browse/FLINK-12481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16838457#comment-16838457
 ] 

aitozi commented on FLINK-12481:


OK,thanks for your reply. Looking forward it. :)

> Make processing time timer trigger run via the mailbox
> --
>
> Key: FLINK-12481
> URL: https://issues.apache.org/jira/browse/FLINK-12481
> Project: Flink
>  Issue Type: Sub-task
>  Components: Runtime / Operators
>Reporter: Stefan Richter
>Priority: Major
>
> This sub-task integrates the mailbox with processing time timer triggering. 
> Those triggers should now be enqueued as mailbox events and picked up by the 
> stream task's main thread for processing.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (FLINK-12481) Make processing time timer trigger run via the mailbox

2019-05-13 Thread Stefan Richter (JIRA)


[ 
https://issues.apache.org/jira/browse/FLINK-12481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16838391#comment-16838391
 ] 

Stefan Richter commented on FLINK-12481:


Hi [~aitozi] and thanks for offering some help. Can we maybe postpone the 
decision until the mailbox is already introduced because this sub-task is 
essentially blocked on that work? If I have not enough resources at that point, 
you can have it but otherwise I would work on it myself. Also the timer case is 
a bit more tricky than the checkpoint case, e.g. scheduled timers can also be 
cancelled and checkpoint triggers not. I will get back to you when I completed 
the blocker sub-task for this one.

> Make processing time timer trigger run via the mailbox
> --
>
> Key: FLINK-12481
> URL: https://issues.apache.org/jira/browse/FLINK-12481
> Project: Flink
>  Issue Type: Sub-task
>  Components: Runtime / Operators
>Reporter: Stefan Richter
>Priority: Major
>
> This sub-task integrates the mailbox with processing time timer triggering. 
> Those triggers should now be enqueued as mailbox events and picked up by the 
> stream task's main thread for processing.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (FLINK-12481) Make processing time timer trigger run via the mailbox

2019-05-12 Thread aitozi (JIRA)


[ 
https://issues.apache.org/jira/browse/FLINK-12481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16838280#comment-16838280
 ] 

aitozi commented on FLINK-12481:


Hi, [~srichter] After going through the doc of this feature, I think it is a 
good improvement to eliminate the usage of checkpoint lock. Before I was 
confused by the checkpoint lock used in processing time trigger. And i am 
interested on this issue, can i do a favour to work on this issue, thanks.

> Make processing time timer trigger run via the mailbox
> --
>
> Key: FLINK-12481
> URL: https://issues.apache.org/jira/browse/FLINK-12481
> Project: Flink
>  Issue Type: Sub-task
>  Components: Runtime / Operators
>Reporter: Stefan Richter
>Priority: Major
>
> This sub-task integrates the mailbox with processing time timer triggering. 
> Those triggers should now be enqueued as mailbox events and picked up by the 
> stream task's main thread for processing.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)