presto
presto copied to clipboard
Fix an issue while accessing s3 when RaptorX is enabled
This is needed because RaptorX/Alluxio only works with URI schemes alluxio:// and ws://. This scheme is only used as a key for collecting stats.
Fixes #15951
== NO RELEASE NOTE ==
One minor comment, it is ok, if you want to defer it.
Shouldn't the right fix be removing the check here https://github.com/Alluxio/alluxio/blob/master/core/client/hdfs/src/main/java/alluxio/hadoop/LocalCacheFileSystem.java#L91
The supported scheme is not used beyond that.
Upgrading the dependency after the fix is released.
CC: @rongrong who is a contributor to the Alluxio Caching Filesystem code.
@rongrong Hi! Could you please comment the PR?
@arunthirupathi @jainxrohit Can you please provide an update on this? Were you able to test it internally?
@rongrong Can you please provide your inputs on this comment https://github.com/prestodb/presto/pull/18040#issuecomment-1189719660
Based on our discussion with Bin Fan (Alluxio), I will be closing this PR as the Alluxio team is in discussions(looking positive) to include S3 and HDFS(to begin with) in the supported filesystems for LocalCacheFileSystem (Alluxio) and it is estimated to be released in a month or two barring any setbacks. Once this change is released, we won't need the changes in this PR. Feel free to reach out in case of any questions.