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

Chris Riccomini commented on SAMZA-94:
--------------------------------------

This unit test appears to pass, and looks a lot like your example:

{noformat}
  @Test
  def testDeleteNotNull {
    val something = b(letters(0))
    store.put(something, something)

    val iterator = store.all
    assertTrue(iterator.hasNext)
    val entry = iterator.next
    assertTrue(Arrays.equals(something, entry.getKey))
    assertTrue(Arrays.equals(something, entry.getValue))
    assertFalse(iterator.hasNext)
    iterator.close()

    store.delete(something)

    val iterator2 = store.all()
    assertFalse(iterator2.hasNext)
    iterator2.close()
  }
{noformat}

> State management kv-store delete() does not remove key
> ------------------------------------------------------
>
>                 Key: SAMZA-94
>                 URL: https://issues.apache.org/jira/browse/SAMZA-94
>             Project: Samza
>          Issue Type: Bug
>          Components: kv
>    Affects Versions: 0.6.0
>            Reporter: Alan Li
>            Assignee: Chris Riccomini
>             Fix For: 0.7.0
>
>
> Sample code snippet: 
> {noformat}
>     _realtimeStateStore.put("foo", new HashMap());
>     
>     KeyValueIterator<String, Map<String, State>> realtimeIterator = 
> _realtimeStateStore.all();
>     while (realtimeIterator.hasNext())
>     {
>       Entry<String, Map<String, State>> entry = realtimeIterator.next();
>       System.out.println("read from leveldb: " + entry.getKey() + "=>" + 
> entry.getValue());
>     }
>     realtimeIterator.close();
>     
>     _realtimeStateStore.delete("foo");
>     
>     KeyValueIterator<String, Map<String, State>> iter = 
> _realtimeStateStore.all();
>     while (iter.hasNext())
>     {
>       Entry<String, Map<String, State>> entry = iter.next();
>       System.out.println("read from leveldb: " + entry.getKey() + "=>" + 
> entry.getValue());
>     }
>     iter.close();
> {noformat}
> Output:
>   read from leveldb: foo=>{}
>   read from leveldb: foo=>null
> I would expect that the kv would remove both the key & value on delete so 
> that subsequent iterators will not return keys that have been deleted.



--
This message was sent by Atlassian JIRA
(v6.1#6144)

Reply via email to