xiangfu0 opened a new pull request, #19238: URL: https://github.com/apache/pinot/pull/19238
## Summary - build on the existing default-off `ingestFromURI` local-filesystem policy by resolving and validating the exact filesystem before ingestion work starts - reject local path, subclass, alias, and delegate forms while keeping configured and request-provided remote filesystems working - keep request filesystem lifecycle ownership explicit, return generic API errors, and document the compatibility opt-in - ensure independently created Hadoop filesystem adapters own the clients they close ## Existing behavior on master Master already contains `dd6520c7267` / #18660, which introduced the default-off setting and the initial direct URI and class checks. This change builds on that implementation to cover remaining delegate, lifecycle, ordering, and error-handling cases. ## Behavior before this change Filesystem selection could happen during copying, and endpoint-provided implementations were registered in shared factory state. Rejected requests could also create Controller staging directories before source validation. ## Testing - `PinotFSFactoryTest` - `ControllerConfTest` - `FileIngestionHelperTest` - `PinotIngestionRestletResourceStatelessTest` - `HadoopPinotFSTest` - Spotless, Checkstyle, and license checks for `pinot-spi`, `pinot-controller`, and `pinot-hdfs` - affected-module test compilation with compiler warnings enabled -- 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]
