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

ASF GitHub Bot commented on APEXMALHAR-1897:
--------------------------------------------

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

    
https://github.com/apache/incubator-apex-malhar/pull/145#discussion_r56752821
  
    --- Diff: 
library/src/main/java/com/datatorrent/lib/state/BucketedState.java ---
    @@ -29,11 +31,17 @@
     public interface BucketedState
     {
       /**
    +   * An expired value. In some implementations where bucketId is time 
related then the event can be old and
    +   * the get methods- getSync & getAsync return this special slice 
instance.
    +   */
    +  Slice EXPIRED = new Slice(Ints.toByteArray(-1));
    --- End diff --
    
    I can use Slice(null, -1, -1). In the usages, comparisons should be made 
using ```==``` and not ```equals```.
    
    For every expired event I didn't want to instantiate a Slice object. So 
even if I extend Slice, I will re-use a special instance of slice so thought 
why not use just a special instance.
    
    There is a problem with all the approaches I can think of which is that 
Slice cannot have an immutable extension because all the members are publicly 
exposed without getters and setters.


> Create ManagedState
> -------------------
>
>                 Key: APEXMALHAR-1897
>                 URL: https://issues.apache.org/jira/browse/APEXMALHAR-1897
>             Project: Apache Apex Malhar
>          Issue Type: Sub-task
>            Reporter: Chandni Singh
>            Assignee: Chandni Singh
>             Fix For: 3.4.0
>
>
> ManagedState is described in the document below:
> https://docs.google.com/document/d/1gRWN9ufKSZSZD0N-pthlhpC9TZ8KwJ6hJlAX6nxl5f8/edit#heading=h.z87ti1fwyt0t



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

Reply via email to