bugcleanupgood first issue
倉庫指標
- Star
- (4,937 star)
- PR 合併指標
- (平均合併 6天 7小時) (30 天內合併 186 個 PR)
描述
In the TableDataManager:
void addSegment(File indexDir, IndexLoadingConfig indexLoadingConfig)is never used in production code (the only non-testing calleraddOfflineSegment()is not wired up)void addSegment(String segmentName, TableConfig tableConfig, IndexLoadingConfig indexLoadingConfig)can be simplified becauseTableConfigis included in theIndexLoadingConfigvoid addOrReplaceSegment(String segmentName, IndexLoadingConfig indexLoadingConfig, SegmentZKMetadata zkMetadata, @Nullable SegmentMetadata localMetadata)should be broken into 2 methods:addSegment()andrefreshSegment()- Loading committed segment into real-time table should share the same logic as loading segment into offline table. Currently the logic is diverged, which causes moving tier not happening for real-time table
Ideally we want to keep the following methods:
addSegmentaddConsumingSegment(real-time only)refreshSegmentreloadSegmentremoveSegment