[jira] [Comment Edited] (FLINK-3758) Add possibility to register accumulators in custom triggers

["Aljoscha Krettek (JIRA)" <ji...@apache.org>]

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

Aljoscha Krettek edited comment on FLINK-3758 at 5/9/16 1:29 PM:
-----------------------------------------------------------------

Please open a PR. The changes look good, module some fixes in the comments, for example {{TriggerContext.getAccumulator(String, DefaultAccumulator)}} still has the JavaDoc from {{RuntimeContext.getAccumulator(String)}}.


was (Author: aljoscha):
Please open a PR. The changes look good, module some fixes in the comments, for example {{TriggerContext.getAccumulator(String, DefaultAccumulator)}} still has the comment from {{RuntimeContext.getAccumulator(String)}}.

> Add possibility to register accumulators in custom triggers
> -----------------------------------------------------------
>
>                 Key: FLINK-3758
>                 URL: https://issues.apache.org/jira/browse/FLINK-3758
>             Project: Flink
>          Issue Type: Improvement
>            Reporter: Konstantin Knauf
>            Assignee: Konstantin Knauf
>            Priority: Minor
>
> For monitoring purposes it would be nice to be able to to use accumulators in custom trigger functions. 
> Basically, the trigger context could just expose {{getAccumulator}} of {{RuntimeContext}} or does this create problems I am not aware of?
> Adding accumulators in a trigger function is more difficult, I think, but that's not really neccessary as the accummulator could just be added in some other upstream operator.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)