Skip to content

Commit

Permalink
HADOOP-14390. Correct spelling of 'succeed' and variants. Contributed…
Browse files Browse the repository at this point in the history
… by Dongtao Zhang
  • Loading branch information
cdouglas committed May 5, 2017
1 parent a3954cc commit e4f34ec
Show file tree
Hide file tree
Showing 5 changed files with 11 additions and 11 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -251,7 +251,7 @@ private void testOperationsThroughMountLinksInternal(boolean located)
fsTarget.isFile(new Path(targetTestRoot,"user/foo")));

// Delete the created file
Assert.assertTrue("Delete should suceed",
Assert.assertTrue("Delete should succeed",
fsView.delete(new Path("/user/foo"), false));
Assert.assertFalse("File should not exist after delete",
fsView.exists(new Path("/user/foo")));
Expand All @@ -266,7 +266,7 @@ private void testOperationsThroughMountLinksInternal(boolean located)
fsTarget.isFile(new Path(targetTestRoot,"dir2/foo")));

// Delete the created file
Assert.assertTrue("Delete should suceed",
Assert.assertTrue("Delete should succeed",
fsView.delete(new Path("/internalDir/linkToDir2/foo"), false));
Assert.assertFalse("File should not exist after delete",
fsView.exists(new Path("/internalDir/linkToDir2/foo")));
Expand Down Expand Up @@ -370,7 +370,7 @@ private void testOperationsThroughMountLinksInternal(boolean located)
public void testRenameAcrossMounts1() throws IOException {
fileSystemTestHelper.createFile(fsView, "/user/foo");
fsView.rename(new Path("/user/foo"), new Path("/user2/fooBarBar"));
/* - code if we had wanted this to suceed
/* - code if we had wanted this to succeed
Assert.assertFalse(fSys.exists(new Path("/user/foo")));
Assert.assertFalse(fSysLocal.exists(new Path(targetTestRoot,"user/foo")));
Assert.assertTrue(fSys.isFile(FileSystemTestHelper.getTestRootPath(fSys,"/user2/fooBarBar")));
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -232,7 +232,7 @@ public void testOperationsThroughMountLinks() throws IOException {
isFile(fcTarget, new Path(targetTestRoot,"dir2/foo")));

// Delete the created file
Assert.assertTrue("Delete should suceed",
Assert.assertTrue("Delete should succeed",
fcView.delete(new Path("/internalDir/linkToDir2/foo"),false));
Assert.assertFalse("File should not exist after deletion",
exists(fcView, new Path("/internalDir/linkToDir2/foo")));
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -509,8 +509,8 @@ public void testSecondaryNamenodeError3() throws IOException {
Mockito.reset(faultInjector);
secondary.shutdown(); // secondary namenode crash!

// start new instance of secondary and verify that
// a new rollEditLog suceedes inspite of the fact that
// start new instance of secondary and verify that
// a new rollEditLog succeeds inspite of the fact that
// edits.new already exists.
//
secondary = startSecondaryNameNode(conf);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,7 @@ static private void handleException(String operation, Throwable e,

/**
* Create and write to a given number of files. Repeat each remote
* operation until is suceeds (does not throw an exception).
* operation until is succeeds (does not throw an exception).
*
* @return the number of exceptions caught
*/
Expand Down Expand Up @@ -178,7 +178,7 @@ static int openRead() {

/**
* Rename a given number of files. Repeat each remote
* operation until is suceeds (does not throw an exception).
* operation until is succeeds (does not throw an exception).
*
* @return the number of exceptions caught
*/
Expand Down Expand Up @@ -208,7 +208,7 @@ static int rename() {

/**
* Delete a given number of files. Repeat each remote
* operation until is suceeds (does not throw an exception).
* operation until is succeeds (does not throw an exception).
*
* @return the number of exceptions caught
*/
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -866,7 +866,7 @@ public void testValidateOrderNoGap() {
curAlloc.clear();
curAlloc.put(new ReservationInterval(9 * step, 10 * step),
Resource.newInstance(1024, 1));
assertTrue("validateOrderNoFap() should have suceeded",
assertTrue("validateOrderNoFap() should have succeeded",
IterativePlanner.validateOrderNoGap(allocation, curAlloc, false));

// 2. allocateLeft = false, fail when curAlloc has a gap
Expand All @@ -888,7 +888,7 @@ public void testValidateOrderNoGap() {
curAlloc.clear();
curAlloc.put(new ReservationInterval(13 * step, 14 * step),
Resource.newInstance(1024, 1));
assertTrue("validateOrderNoFap() should have suceeded",
assertTrue("validateOrderNoFap() should have succeeded",
IterativePlanner.validateOrderNoGap(allocation, curAlloc, true));

// 5. allocateLeft = true, fail when there is a gap between curAlloc and
Expand Down

0 comments on commit e4f34ec

Please sign in to comment.