[
https://issues.apache.org/jira/browse/SAMZA-326?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14056794#comment-14056794
]
Chris Riccomini commented on SAMZA-326:
---------------------------------------
bq. Also, I noticed 0.7.1 release was breaking due to the new commit
(SAMZA-321) - the reason being missing import and build.gradle changes.
That's what I get for being optimistic. Caught me only testing on master. :P
Testing right now.
> LevelDbKeyValueStore range implements 'from' -> 'to' instead of 'from' ->
> 'until'
> ---------------------------------------------------------------------------------
>
> Key: SAMZA-326
> URL: https://issues.apache.org/jira/browse/SAMZA-326
> Project: Samza
> Issue Type: Bug
> Components: kv
> Affects Versions: 0.7.0
> Reporter: Chinmay Soman
> Assignee: Chinmay Soman
> Fix For: 0.7.1
>
> Attachments: samza_326.patch
>
>
> Most built-in ordered collection data structures in Java (SortedMap, List,
> ...) define a range (submap / sublist) as follows:
> 'from' (inclusive) -> 'until' (exclusive)
> However, the current LevelDbKeyValueStore implementation defines range as:
> 'from' (inclusive) -> 'to' (inclusive)
> While this is not technically incorrect, I think we should change the
> semantics to follow the norm.
> P.S. This can easily be done by changing the '<=' comparison in
> LevelDbRangeIterator to '<'
--
This message was sent by Atlassian JIRA
(v6.2#6252)
