steveloughran commented on a change in pull request #1711: HADOOP-16455. ABFS: Implement FileSystem.access() method. URL: https://github.com/apache/hadoop/pull/1711#discussion_r347151616
########## File path: hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/ITestAzureBlobFileSystemCheckAccess.java ########## @@ -32,9 +28,14 @@ import org.apache.hadoop.fs.permission.AclEntryType; import org.apache.hadoop.fs.permission.FsAction; import org.apache.hadoop.security.AccessControlException; + import org.junit.Assume; import org.junit.Test; +import java.io.FileNotFoundException; Review comment: please restore to where they were. java.* should be the first import. Please: import conflict is one of the key source of false-positive backport problems. Patches MUST NOT move imports around without good reason. This is not one of them. * If your IDE is set up to rearrange things: disable. * if your IDE has an "optimize imports" button -don't press it. * consider reviewing submitted PRs to look for spurious IDE-initiated changes ---------------------------------------------------------------- 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]
