[GitHub] [incubator-pinot] chenboat opened a new pull request #4733: Integration tests for LLC with PinotFS on non-local FS.

[GitBox <gi...@apache.org>]

chenboat opened a new pull request #4733: Integration tests for LLC with PinotFS on non-local FS.
URL: https://github.com/apache/incubator-pinot/pull/4733
 
 
   1. Add a new integration test on Low Level Consumer on non-local Pinot FS. A mini hdfs cluster is used in test instead.
   
   2. Fix a bug in move() of PinotFS.java: it does not work with a non-local FS (i.e., uri starting with hdfs://)because  Paths.get() needs an external configured FilesystemProvider. A simpler and more reliable solution is to use URI.resolve(): it does not need any external dependency.   

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org