Skip to content

Commit 1440f07

Browse files
slfan1989slfan1989
committed
HADOOP-19415. Fix CheckStyle Issue.
1 parent 20b615e commit 1440f07

File tree

4 files changed

+23
-23
lines changed

4 files changed

+23
-23
lines changed

‎hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/FSMainOperationsBaseTest.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -350,7 +350,7 @@ public void testListStatusFilterWithNoMatches() throws Exception {
350350
// listStatus with filters returns empty correctly
351351
FileStatus[] filteredPaths = fSys.listStatus(
352352
getTestRootPath(fSys, "test"), TEST_X_FILTER);
353-
assertEquals(0,filteredPaths.length);
353+
assertEquals(0, filteredPaths.length);
354354

355355
}
356356

@@ -371,7 +371,7 @@ public void testListStatusFilterWithSomeMatches() throws Exception {
371371
// should return 2 paths ("/test/hadoop/axa" and "/test/hadoop/axx")
372372
FileStatus[] filteredPaths = fSys.listStatus(
373373
getTestRootPath(fSys, "test/hadoop"), TEST_X_FILTER);
374-
assertEquals(2,filteredPaths.length);
374+
assertEquals(2, filteredPaths.length);
375375
assertTrue(containsTestRootPath(getTestRootPath(fSys,
376376
TEST_DIR_AXA), filteredPaths));
377377
assertTrue(containsTestRootPath(getTestRootPath(fSys,
@@ -503,7 +503,7 @@ public void testGlobStatusFilterWithEmptyPathResults() throws Exception {
503503
FileStatus[] filteredPaths = fSys.globStatus(
504504
getTestRootPath(fSys, "test/hadoop/?"),
505505
DEFAULT_FILTER);
506-
assertEquals(0,filteredPaths.length);
506+
assertEquals(0, filteredPaths.length);
507507
}
508508

509509
@Test
@@ -607,7 +607,7 @@ public void testGlobStatusFilterWithNoMatchingPathsAndNonTrivialFilter()
607607
FileStatus[] filteredPaths = fSys.globStatus(
608608
getTestRootPath(fSys, "test/hadoop/?"),
609609
TEST_X_FILTER);
610-
assertEquals(0,filteredPaths.length);
610+
assertEquals(0, filteredPaths.length);
611611
}
612612

613613
@Test

‎hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/FileContextMainOperationsBaseTest.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -443,7 +443,7 @@ public void testListStatusFilterWithNoMatches() throws Exception {
443443
// listStatus with filters returns empty correctly
444444
FileStatus[] filteredPaths = fc.util().listStatus(
445445
getTestRootPath(fc, "test"), TEST_X_FILTER);
446-
assertEquals(0,filteredPaths.length);
446+
assertEquals(0, filteredPaths.length);
447447

448448
}
449449

@@ -465,7 +465,7 @@ public void testListStatusFilterWithSomeMatches() throws Exception {
465465
FileStatus[] filteredPaths = fc.util()
466466
.listStatus(getTestRootPath(fc, "test/hadoop"),
467467
TEST_X_FILTER);
468-
assertEquals(2,filteredPaths.length);
468+
assertEquals(2, filteredPaths.length);
469469
assertTrue(containsPath(getTestRootPath(fc,
470470
TEST_DIR_AXA), filteredPaths));
471471
assertTrue(containsPath(getTestRootPath(fc,
@@ -597,7 +597,7 @@ public void testGlobStatusFilterWithEmptyPathResults() throws Exception {
597597
FileStatus[] filteredPaths = fc.util().globStatus(
598598
getTestRootPath(fc, "test/hadoop/?"),
599599
DEFAULT_FILTER);
600-
assertEquals(0,filteredPaths.length);
600+
assertEquals(0, filteredPaths.length);
601601
}
602602

603603
@Test
@@ -701,7 +701,7 @@ public void testGlobStatusFilterWithNoMatchingPathsAndNonTrivialFilter()
701701
FileStatus[] filteredPaths = fc.util().globStatus(
702702
getTestRootPath(fc, "test/hadoop/?"),
703703
TEST_X_FILTER);
704-
assertEquals(0,filteredPaths.length);
704+
assertEquals(0, filteredPaths.length);
705705
}
706706

707707
@Test

‎hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/viewfs/TestViewFsWithAuthorityLocalFs.java

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -65,13 +65,13 @@ public void tearDown() throws Exception {
6565
public void testBasicPaths() {
6666
assertEquals(schemeWithAuthority, fcView.getDefaultFileSystem().getUri());
6767
assertEquals(fcView.makeQualified(
68-
new Path("/user/" + System.getProperty("user.name"))),
69-
fcView.getWorkingDirectory());
68+
new Path("/user/" + System.getProperty("user.name"))),
69+
fcView.getWorkingDirectory());
7070
assertEquals(fcView.makeQualified(
71-
new Path("/user/" + System.getProperty("user.name"))),
72-
fcView.getHomeDirectory());
71+
new Path("/user/" + System.getProperty("user.name"))),
72+
fcView.getHomeDirectory());
7373
assertEquals(
74-
new Path("/foo/bar").makeQualified(schemeWithAuthority, null),
75-
fcView.makeQualified(new Path("/foo/bar")));
74+
new Path("/foo/bar").makeQualified(schemeWithAuthority, null),
75+
fcView.makeQualified(new Path("/foo/bar")));
7676
}
7777
}

‎hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/viewfs/ViewFsBaseTest.java

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -674,7 +674,7 @@ public void testResolvePathMissingThroughMountPoints() throws IOException {
674674
public void testResolvePathMissingThroughMountPoints2() throws IOException {
675675
assertThrows(FileNotFoundException.class, () -> {
676676
fcView.mkdir(fileContextTestHelper.getTestRootPath(fcView, "/user/dirX"),
677-
FileContext.DEFAULT_PERM, false);
677+
FileContext.DEFAULT_PERM, false);
678678
fcView.resolvePath(new Path("/user/dirX/nonExisting"));
679679
});
680680
}
@@ -694,37 +694,37 @@ public void testResolvePathMissingThroughMountPoints2() throws IOException {
694694
public void testInternalMkdirSlash() throws IOException {
695695
assertThrows(AccessControlException.class, () -> {
696696
fcView.mkdir(fileContextTestHelper.getTestRootPath(fcView, "/"),
697-
FileContext.DEFAULT_PERM, false);
697+
FileContext.DEFAULT_PERM, false);
698698
});
699699
}
700700

701701
@Test
702702
public void testInternalMkdirExisting1() throws IOException {
703703
assertThrows(AccessControlException.class, () -> {
704704
fcView.mkdir(fileContextTestHelper.getTestRootPath(fcView, "/internalDir"),
705-
FileContext.DEFAULT_PERM, false);
705+
FileContext.DEFAULT_PERM, false);
706706
});
707707
}
708708
@Test
709709
public void testInternalMkdirExisting2() throws IOException {
710710
assertThrows(AccessControlException.class, () -> {
711711
fcView.mkdir(fileContextTestHelper.getTestRootPath(fcView,
712712
"/internalDir/linkToDir2"),
713-
FileContext.DEFAULT_PERM, false);
713+
FileContext.DEFAULT_PERM, false);
714714
});
715715
}
716716
@Test
717717
public void testInternalMkdirNew() throws IOException {
718718
assertThrows(AccessControlException.class, () -> {
719719
fcView.mkdir(fileContextTestHelper.getTestRootPath(fcView, "/dirNew"),
720-
FileContext.DEFAULT_PERM, false);
720+
FileContext.DEFAULT_PERM, false);
721721
});
722722
}
723723
@Test
724724
public void testInternalMkdirNew2() throws IOException {
725725
assertThrows(AccessControlException.class, () -> {
726726
fcView.mkdir(fileContextTestHelper.getTestRootPath(fcView, "/internalDir/dirNew"),
727-
FileContext.DEFAULT_PERM, false);
727+
FileContext.DEFAULT_PERM, false);
728728
});
729729
}
730730

@@ -790,7 +790,7 @@ public void testInternalDeleteExisting() throws IOException {
790790
public void testInternalDeleteExisting2() throws IOException {
791791
assertThrows(AccessControlException.class, () -> {
792792
assertTrue(fcView.getFileStatus(new Path("/internalDir/linkToDir2")).isDirectory(),
793-
"Delete of link to dir should succeed");
793+
"Delete of link to dir should succeed");
794794
fcView.delete(new Path("/internalDir/linkToDir2"), false);
795795
});
796796
}
@@ -808,7 +808,7 @@ public void testInternalRename1() throws IOException {
808808
public void testInternalRename2() throws IOException {
809809
assertThrows(AccessControlException.class, () -> {
810810
assertTrue(fcView.getFileStatus(new Path("/internalDir/linkToDir2")).isDirectory(),
811-
"linkTODir2 should be a dir");
811+
"linkTODir2 should be a dir");
812812
fcView.rename(new Path("/internalDir/linkToDir2"), new Path("/internalDir/dir1"));
813813
});
814814
}
@@ -950,7 +950,7 @@ public void testInternalCreateSnapshot2() throws IOException {
950950
public void testInternalRenameSnapshot() throws IOException {
951951
assertThrows(AccessControlException.class, () -> {
952952
fcView.renameSnapshot(new Path("/internalDir"), "snapOldName",
953-
"snapNewName");
953+
"snapNewName");
954954
});
955955
}
956956

0 commit comments

Comments
 (0)