apache / apache/pinot

Clean up SegmentLineage#toJsonObject and change listSegmentLineage to reversed chronological order.

Open
#9,053 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Java
Stars
6.1k
Forks
1.5k
Avg merge
1d 21h
Merged PRs (30d)
189

Description

As a follow up to https://github.com/apache/pinot/issues/9005, refactor `SegmentLineage#toJsonObject` by moving the sorting logic out to keep `toJsonObject` lightweight and in line with proper convention.
Also, change `PinotSegmentRestletResource#listSegmentLineage` to return entries in reversed chronological order instead.

Contributor guide

Open the contributing guide

Research direction

Start by reading SegmentLineage#toJsonObject and PinotSegmentRestletResource#listSegmentLineage, along with the follow-up issue 9005. Trace where sorting currently occurs and how lineage entries are assembled; done means toJsonObject no longer performs sorting and the REST resource returns entries in reversed chronological order.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
api
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.