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

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

GitHub user davidyan74 opened a pull request:

    https://github.com/apache/apex-malhar/pull/377

    *Review Only* APEXMALHAR-2192 Added some spillable method implementations

    This is not ready for merge yet. Please do not comment on missing unit 
tests, code style, debug statements, etc.
    
    @brightchen @ilooner I added removeAll method to 
SpillableByteArrayListMultimapImpl but the testRemoveAll test method fails. Can 
you please take a look at what I did wrong?

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/davidyan74/apex-malhar APEXMALHAR-2192

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/apex-malhar/pull/377.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #377
    
----
commit f8168445e1f78ca48924a90a2ee0140eb85bde23
Author: David Yan <[email protected]>
Date:   2016-08-22T20:15:36Z

    added some spillable method implementations

----


> Implement SpillableByteArrayListMultimapImpl.removeAll(Object key)
> ------------------------------------------------------------------
>
>                 Key: APEXMALHAR-2192
>                 URL: https://issues.apache.org/jira/browse/APEXMALHAR-2192
>             Project: Apache Apex Malhar
>          Issue Type: Sub-task
>            Reporter: David Yan
>            Assignee: David Yan
>
> This is needed by the spillable implementation of 
> SpillableWindowedKeyedStorage. In the implementation of WindowedOperator, 
> when purging a window, we need to purge all keys associated with a given 
> window.



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

Reply via email to