Akanksha-kedia opened a new pull request, #18778: URL: https://github.com/apache/pinot/pull/18778
## Description Removes three `@Deprecated` methods from `PinotHelixResourceManager` that have no callers outside the class. This continues the pattern of cleaning up dead deprecated code (similar to #18699, #18472, #17072). ### Methods removed | Method | Replaced by | Callers | |--------|-------------|---------| | `deleteSchema(Schema)` | `deleteSchema(String)` | None — the only non-test caller already uses the `String` overload | | `getSchemaForTableConfig(TableConfig)` | `ZKMetadataProvider.getTableSchema()` directly | None | | `getServersForSegment(String, String)` | `getServers(String, String)` | None | All three methods were marked `@Deprecated` with their successors already in use. Removing them reduces the API surface and eliminates dead code. ## Tests No test changes needed — no test callers exist for these methods. ## Checklist - [x] No new public APIs without documentation - [x] Backward compatible change (methods were already deprecated; no callers remain) - [x] Code follows existing patterns in the codebase -- 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]
