jihoonson commented on pull request #10830: URL: https://github.com/apache/druid/pull/10830#issuecomment-772206529
> Do the docs need to be updated as well? Right now it just says "URIs of the input files." https://druid.apache.org/docs/latest/ingestion/native-batch.html#http-input-source The linked section starts with `The HTTP input source is to support reading files directly from remote sites via HTTP`. I think this implies the URIs must be HTTP URIs, but guess it doesn't harm to make it clear. I updated the doc. I also fixed the HDFS inputSource to allow only `hdfs` paths. All other inputSources already have a scheme check if they accept URIs. ---------------------------------------------------------------- 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...@druid.apache.org For additional commands, e-mail: commits-h...@druid.apache.org
