HarperFast / HarperFast/rocksdb-js

Add support for `lower` and `upper` as range options

Open
#332 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
C++
Stars
21
Forks
2
Avg merge
2d 9h
Merged PRs (30d)
36

Description

The use of `start` and `end` as the the range options can be confusing when combined with `reverse` since it swaps the meaning of `start` to being the upper limit and the `end` to being the lower limit. Offering `lower` and `upper` options more explicitly map to the lower and upper limits of range regardless of the `reverse` flag.
We would probably also need corresponding `lowerExclusive` and `upperInclusive` flags for changing default boundary behavior.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.