Github user satishd commented on a diff in the pull request:

    https://github.com/apache/storm/pull/900#discussion_r47595521
  
    --- Diff: storm-core/src/jvm/backtype/storm/windowing/EvictionPolicy.java 
---
    @@ -39,4 +63,12 @@
          * @param event the input event to be tracked
          */
         void track(Event<T> event);
    +
    +    /**
    +     * Sets a context in the eviction policy that can be used while 
evicting the events.
    +     * E.g. For TimeEvictionPolicy, this could be used to set the 
reference timestamp.
    +     *
    +     * @param context
    +     */
    +    void setContext(Object context);
    --- End diff --
    
    May want to introduce class/interface EvictionPolicyContext which may have 
opaque structure for now. This can be done later as this PR is waiting for long.



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to