steveloughran commented on issue #568: HADOOP-15691 Add PathCapabilities to FS and FC to complement StreamCapabilities URL: https://github.com/apache/hadoop/pull/568#issuecomment-470977176 javac are about a deprecated method, cannot be fixed. Checkstyles can be ``` ./hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/CommonPathCapabilities.java:28:public final class CommonPathCapabilities {:1: Utility classes should not have a public or default constructor. [HideUtilityClassConstructor] ./hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/impl/FsLinkResolution.java:76: T apply(final AbstractFileSystem fs, final Path path):13: Redundant 'final' modifier. [RedundantModifier] ./hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/impl/FsLinkResolution.java:76: T apply(final AbstractFileSystem fs, final Path path):42: Redundant 'final' modifier. [RedundantModifier] ./hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/impl/PathCapabilitiesSupport.java:27:@InterfaceAudience.Private: Missing a Javadoc comment. [JavadocType] ./hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/impl/PathCapabilitiesSupport.java:27:@InterfaceAudience.Private:1: Utility classes should not have a public or default constructor. [HideUtilityClassConstructor] ./hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/viewfs/ViewFileSystem.java:56:import org.apache.hadoop.fs.PathCapabilities;:8: Unused import - org.apache.hadoop.fs.PathCapabilities. [UnusedImports] ./hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/DistributedFileSystem.java:52:import org.apache.hadoop.fs.PathCapabilities;:8: Unused import - org.apache.hadoop.fs.PathCapabilities. [UnusedImports] ./hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/WebHdfsFileSystem.java:80:import org.apache.hadoop.fs.PathCapabilities;:8: Unused import - org.apache.hadoop.fs.PathCapabilities. [UnusedImports] ./hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/commit/ITestCommitOperations.java:27:import org.junit.Assume;:8: Unused import - org.junit.Assume. [UnusedImports] ```
---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
