deepthi912 opened a new pull request, #19366:
URL: https://github.com/apache/pinot/pull/19366
## Summary
Adds `?verbose=true` to `GET /tables/{tableName}/segments/needReload`. When
set, each server response also lists the segments that need reload; the boolean
short-circuited path is unchanged and remains the default.
- `ServerSegmentsReloadCheckResponse` gains a nullable
`segmentsNeedingReload` (`@JsonInclude(NON_NULL)`, backward-compatible 2-arg
ctor preserved).
- `TableDataManager` gets a default `getSegmentNamesNeedingReload()`;
`BaseTableDataManager` consolidates both entry points into a shared
`findSegmentsNeedingReload(boolean shortCircuit)` helper so the boolean path
keeps its early-exit cost.
- `TablesResource` reads the query param and dispatches;
`ServerSegmentMetadataReader`, `TableMetadataReader`, and
`PinotTableReloadService` thread `verbose` down the call chain.
Existing callers that don't pass the flag see no behavior change.
## Test plan
- [ ] `mvn -pl
pinot-common,pinot-segment-local,pinot-core,pinot-server,pinot-controller test`
- [ ] Manual: `curl .../tables/foo_OFFLINE/segments/needReload` returns the
same shape as before
- [ ] Manual: `curl .../tables/foo_OFFLINE/segments/needReload?verbose=true`
lists segment names per server
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
