cbevard1 opened a new issue #2306: URL: https://github.com/apache/accumulo/issues/2306
https://github.com/apache/accumulo/blob/8d714bb19bcf3fc4f29818fd16b04901fdeb3a99/server/base/src/main/java/org/apache/accumulo/server/manager/recovery/HadoopLogCloser.java#L72-L75 The underlying FileSystem implementation is abstracted in Accumulo, but custom FileSystem implementations are restricted by this section of code in the HadoopLogCloser. I'm working on an S3 classpath extension that would allow S3 to be used as the underlying file system, but this thrown exception makes that impossible. Instead of throwing an exception here couldn't the log closer simply log a warning? It's very unlikely someone would mistakenly make all the configuration changes necessary in order for this like of code to be executed. I considered making the custom FileSystem implementation extend DistributedFileSystem instead, but that class is too tightly coupled to HDFS for that route to make sense. -- 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]
