KKcorps opened a new pull request #5836: URL: https://github.com/apache/incubator-pinot/pull/5836
This PR fixes the issue #5835 of ingesting data from S3. The fix involves the following changes - Return full paths along with scheme in listFiles functions instead of path after the bucket. This change is needed because the output of the listFiles in passed to isDirectory function which throws Exception since the path doesn't contain the bucket - Take only the path instead of the full URI in segment generation job runner while creating a local file. The segment generator throws the error `inputFileURI doesn't start with file://` on creating the local file. This small change of taking `inputFileURI.getPath()` instead of `inputFileURI` fixes this issue. The changes have been verified by running a standalone job to ingest as well as upload data in S3. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
