K2V issues #305

New Issue

Closed

opened 2022-05-17 08:52:01 +00:00 by lx · 0 comments

lx commented

2022-05-17 08:52:01 +00:00

Owner

K2V end not taken into account properly

This line in api/k2v/range.rs:

if entry.sort_key() == e {
    ...
}

is wrong. TODO:

test case where end actually fails
fix test case

Missing test case for 'tombstones' in ReadBatch

TODO:

add test case

**K2V `end` not taken into account properly** This line in `api/k2v/range.rs`: ``` if entry.sort_key() == e { ... } ``` is wrong. TODO: - test case where end actually fails - fix test case **Missing test case for 'tombstones' in ReadBatch** TODO: - add test case

lx added the

label 2022-05-17 08:52:01 +00:00

lx changed title from ~~K2V `end` not taken into account properly~~ to K2V issues

2022-05-17 09:03:44 +00:00

lx referenced this issue from a commit

2022-05-17 09:50:57 +00:00

K2V: Fix `end` parameter and add tests (fix #305)

lx referenced a pull request that will close this issue

2022-05-17 09:53:26 +00:00

K2V: Fix #305 #306

lx closed this issue

2022-05-17 11:10:40 +00:00

Sign in to join this conversation.

No Label

S3 Compatibility

No Milestone

No Assignees

1 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Deuxfleurs/garage#305

No description provided.