Rename DataSegment to SegmentInfo
Open
Design Review
Evergreen
Refactoring
- Dominant language
- Java
- Stars
- 14.1k
- Forks
- 3.8k
- Avg merge
- 2d 58m
- Merged PRs (30d)
- 233
Description
If anything, the current class `Segment` should be called `DataSegment` and vice versa because `DataSegment` doesn't contain any data. We may either flip the names, or, I think better, leave `Segment` and rename `DataSegment` to `SegmentInfo`.
Contributor guide
Research direction
Search the Druid source for the Segment and DataSegment class definitions and all references, then compare their responsibilities with the proposed names. Apply the Segment and SegmentInfo naming consistently, and verify that the project builds and relevant tests pass.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- backend, databases
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100