Add tests for StoredTabletFile range serialization/deserialization
- Dominant language
- Java
- Stars
- 1.2k
- Forks
- 487
- Avg merge
- 4d 5h
- Merged PRs (30d)
- 13
Description
This is a follow on to #3621. As pointed out here https://github.com/apache/accumulo/pull/3621#discussion_r1265688298 the current implementation of TabletFileCq object will always have a valid range when construction the object. We should add some tests to verify and make sure this doesn't change and then we can get rid of the runtime validation check for verifying a range is a row range.
Contributor guide
Research direction
Read issue #3621 and the linked pull-request discussion first, then inspect StoredTabletFile and TabletFileCq to understand their range serialization and deserialization. Add tests showing that construction preserves a valid row range and that serialization round-trips it correctly; the tests should guard the behavior described in this issue.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- databases, testing
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100