From ed0f4db0dd8769d188e28750e5b78aa0d6f50677 Mon Sep 17 00:00:00 2001 From: Allen Wittenauer Date: Fri, 8 May 2015 17:01:08 -0700 Subject: [PATCH] HADOOP-11775. Fix Javadoc typos in hadoop-openstack module (Yanjun Wang via aw) --- hadoop-common-project/hadoop-common/CHANGES.txt | 3 +++ .../org/apache/hadoop/fs/swift/http/ExceptionDiags.java | 2 +- .../org/apache/hadoop/fs/swift/http/HttpBodyContent.java | 2 +- .../hadoop/fs/swift/http/SwiftProtocolConstants.java | 4 ++-- .../org/apache/hadoop/fs/swift/http/SwiftRestClient.java | 6 +++--- .../src/main/java/org/apache/hadoop/fs/swift/package.html | 2 +- .../hadoop/fs/swift/snative/SwiftNativeFileSystem.java | 4 ++-- .../fs/swift/snative/SwiftNativeFileSystemStore.java | 4 ++-- .../hadoop/fs/swift/snative/SwiftNativeOutputStream.java | 2 +- .../org/apache/hadoop/fs/swift/util/DurationStats.java | 2 +- .../org/apache/hadoop/fs/swift/util/SwiftTestUtils.java | 6 +++--- .../java/org/apache/hadoop/fs/swift/util/SwiftUtils.java | 8 ++++---- .../apache/hadoop/fs/swift/SwiftFileSystemBaseTest.java | 2 +- .../hadoop/fs/swift/TestSwiftFileSystemDirectories.java | 2 +- .../fs/swift/TestSwiftFileSystemExtendedContract.java | 2 +- 15 files changed, 27 insertions(+), 24 deletions(-) diff --git a/hadoop-common-project/hadoop-common/CHANGES.txt b/hadoop-common-project/hadoop-common/CHANGES.txt index 6e23efd64f..ff7991026c 100644 --- a/hadoop-common-project/hadoop-common/CHANGES.txt +++ b/hadoop-common-project/hadoop-common/CHANGES.txt @@ -457,6 +457,9 @@ Trunk (Unreleased) HADOOP-11865. Incorrect path mentioned in document for accessing script files (J.Andreina via aw) + HADOOP-11775. Fix Javadoc typos in hadoop-openstack module (Yanjun Wang + via aw) + OPTIMIZATIONS HADOOP-7761. Improve the performance of raw comparisons. (todd) diff --git a/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/http/ExceptionDiags.java b/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/http/ExceptionDiags.java index 0d540ed026..6baf5c1fac 100644 --- a/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/http/ExceptionDiags.java +++ b/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/http/ExceptionDiags.java @@ -29,7 +29,7 @@ import java.net.UnknownHostException; /** - * Variant of Hadoop Netutils exception wrapping with URI awareness and + * Variant of Hadoop NetUtils exception wrapping with URI awareness and * available in branch-1 too. */ public class ExceptionDiags { diff --git a/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/http/HttpBodyContent.java b/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/http/HttpBodyContent.java index 2b0f9234fc..b471f218e5 100644 --- a/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/http/HttpBodyContent.java +++ b/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/http/HttpBodyContent.java @@ -26,7 +26,7 @@ public class HttpBodyContent { /** * build a body response - * @param inputStream input stream from the operatin + * @param inputStream input stream from the operation * @param contentLength length of content; may be -1 for "don't know" */ public HttpBodyContent(HttpInputStreamWithRelease inputStream, diff --git a/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/http/SwiftProtocolConstants.java b/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/http/SwiftProtocolConstants.java index 6d12504b60..a01f32c18b 100644 --- a/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/http/SwiftProtocolConstants.java +++ b/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/http/SwiftProtocolConstants.java @@ -66,7 +66,7 @@ public class SwiftProtocolConstants { */ public static final String SERVICE_CATALOG_SWIFT = "swift"; /** - * section in the JSON catalog provided after auth listing the cloudfiles; + * section in the JSON catalog provided after auth listing the cloud files; * this is an alternate catalog entry name * {@value} */ @@ -185,7 +185,7 @@ public class SwiftProtocolConstants { FS_SWIFT + ".requestsize"; /** - * The default reqeuest size for reads: {@value} + * The default request size for reads: {@value} */ public static final int DEFAULT_SWIFT_REQUEST_SIZE = 64; diff --git a/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/http/SwiftRestClient.java b/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/http/SwiftRestClient.java index 55dad11124..a2f5a2ac1a 100644 --- a/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/http/SwiftRestClient.java +++ b/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/http/SwiftRestClient.java @@ -83,7 +83,7 @@ * This implements the client-side of the Swift REST API * * The core actions put, get and query data in the Swift object store, - * after authenticationg the client. + * after authenticating the client. * * Logging: * @@ -1572,7 +1572,7 @@ private static StringRequestEntity toJsonEntity(String data) throws * This is public for unit testing * * @param path path to object - * @param endpointURI damain url e.g. http://domain.com + * @param endpointURI domain url e.g. http://domain.com * @return valid URI for object * @throws SwiftException */ @@ -1653,7 +1653,7 @@ private void setAuthToken(HttpMethodBase method, AccessToken accessToken) * Execute a method in a new HttpClient instance. * If the auth failed, authenticate then retry the method. * - * @param method methot to exec + * @param method method to exec * @param Method type * @return the status code * @throws IOException on any failure diff --git a/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/package.html b/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/package.html index 0ccca32b4f..ad900f90d0 100644 --- a/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/package.html +++ b/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/package.html @@ -64,7 +64,7 @@

Supporting multiple Swift Filesystems

the hostname of an S3 container is two-level, the name of the service provider being the second path: swift://bucket.service/ -The service portion of this domainame is used as a reference into +The service portion of this domain name is used as a reference into the client settings -and so identify the service provider of that container. diff --git a/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/snative/SwiftNativeFileSystem.java b/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/snative/SwiftNativeFileSystem.java index 27a572fd84..e9faaf2556 100644 --- a/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/snative/SwiftNativeFileSystem.java +++ b/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/snative/SwiftNativeFileSystem.java @@ -301,7 +301,7 @@ public BlockLocation[] getFileBlockLocations(FileStatus file, * creating directories until one that exists is found. * * This strategy means if a file is created in an existing directory, - * one quick poll sufficies. + * one quick poll suffices. * * There is a big assumption here: that all parent directories of an existing * directory also exists. @@ -540,7 +540,7 @@ public FSDataInputStream open(Path path, int bufferSize) throws IOException { * Low-level operation to also set the block size for this operation * @param path the file name to open * @param bufferSize the size of the buffer to be used. - * @param readBlockSize how big should the read blockk/buffer size be? + * @param readBlockSize how big should the read block/buffer size be? * @return the input stream * @throws FileNotFoundException if the file is not found * @throws IOException any IO problem diff --git a/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/snative/SwiftNativeFileSystemStore.java b/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/snative/SwiftNativeFileSystemStore.java index 6d812a0e6e..cc3e3d2f72 100644 --- a/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/snative/SwiftNativeFileSystemStore.java +++ b/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/snative/SwiftNativeFileSystemStore.java @@ -590,7 +590,7 @@ public void rename(Path src, Path dst) //outcome #2 -move to subdir of dest destPath = toObjectPath(new Path(dst, src.getName())); } else { - //outcome #1 dest it's a file: fail if differeent + //outcome #1 dest it's a file: fail if different if (!renamingOnToSelf) { throw new FileAlreadyExistsException( "cannot rename a file over one that already exists"); @@ -953,7 +953,7 @@ public boolean delete(Path absolutePath, boolean recursive) throws IOException { statuses, "; ")); } - //delete the entries. including ourself. + //delete the entries. including ourselves. for (FileStatus entryStatus : statuses) { Path entryPath = entryStatus.getPath(); try { diff --git a/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/snative/SwiftNativeOutputStream.java b/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/snative/SwiftNativeOutputStream.java index 74710dee84..127fbb8d46 100644 --- a/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/snative/SwiftNativeOutputStream.java +++ b/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/snative/SwiftNativeOutputStream.java @@ -261,7 +261,7 @@ private void writeToBackupStream(byte[] buffer, int offset, int len) throws assert len >= 0 : "remainder to write is negative"; SwiftUtils.debug(LOG," writeToBackupStream(offset=%d, len=%d)", offset, len); if (len == 0) { - //no remainder -downgrade to noop + //no remainder -downgrade to no-op return; } diff --git a/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/util/DurationStats.java b/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/util/DurationStats.java index b6d54f7139..734cf8b6dc 100644 --- a/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/util/DurationStats.java +++ b/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/util/DurationStats.java @@ -40,7 +40,7 @@ public DurationStats(String operation) { } /** - * construct from anothr stats entry; + * construct from another stats entry; * all value are copied. * @param that the source statistics */ diff --git a/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/util/SwiftTestUtils.java b/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/util/SwiftTestUtils.java index 01ec739eda..b4dc5eb5a2 100644 --- a/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/util/SwiftTestUtils.java +++ b/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/util/SwiftTestUtils.java @@ -165,7 +165,7 @@ public static void writeDataset(FileSystem fs, } /** - * Read the file and convert to a byte dataaset + * Read the file and convert to a byte dataset * @param fs filesystem * @param path path to read from * @param len length of data to read @@ -185,7 +185,7 @@ public static byte[] readDataset(FileSystem fs, Path path, int len) } /** - * Assert that tthe array src[0..len] and dest[] are equal + * Assert that the array src[0..len] and dest[] are equal * @param src source data * @param dest actual * @param len length of bytes to compare @@ -522,7 +522,7 @@ public static void assertPathDoesNotExist(FileSystem fileSystem, * @param fs filesystem * @param dir directory to scan * @param subdir full path to look for - * @throws IOException IO probles + * @throws IOException IO problems */ public static void assertListStatusFinds(FileSystem fs, Path dir, diff --git a/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/util/SwiftUtils.java b/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/util/SwiftUtils.java index e86a307f9d..10e49c2473 100644 --- a/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/util/SwiftUtils.java +++ b/hadoop-tools/hadoop-openstack/src/main/java/org/apache/hadoop/fs/swift/util/SwiftUtils.java @@ -82,7 +82,7 @@ public static boolean isFilePretendingToBeDirectory(FileStatus fileStatus) { } /** - * Predicate: Is a swift object referring to the root direcory? + * Predicate: Is a swift object referring to the root directory? * @param swiftObject object to probe * @return true iff the object refers to the root */ @@ -147,7 +147,7 @@ public static String partitionFilenameFromNumber(int partNumber) { * @param fileSystem filesystem * @param path directory * @return a listing of the filestatuses of elements in the directory, one - * to a line, precedeed by the full path of the directory + * to a line, preceded by the full path of the directory * @throws IOException connectivity problems */ public static String ls(FileSystem fileSystem, Path path) throws @@ -169,7 +169,7 @@ public static String ls(FileSystem fileSystem, Path path) throws } /** - * Take an array of filestats and convert to a string (prefixed w/ a [01] counter + * Take an array of filestatus and convert to a string (prefixed w/ a [01] counter * @param stats array of stats * @param separator separator after every entry * @return a stringified set @@ -184,7 +184,7 @@ public static String fileStatsToString(FileStatus[] stats, String separator) { /** * Verify that the basic args to a read operation are valid; - * throws an exception if not -with meaningful text includeing + * throws an exception if not -with meaningful text including * @param buffer destination buffer * @param off offset * @param len number of bytes to read diff --git a/hadoop-tools/hadoop-openstack/src/test/java/org/apache/hadoop/fs/swift/SwiftFileSystemBaseTest.java b/hadoop-tools/hadoop-openstack/src/test/java/org/apache/hadoop/fs/swift/SwiftFileSystemBaseTest.java index 852796c8a8..12f58e6433 100644 --- a/hadoop-tools/hadoop-openstack/src/test/java/org/apache/hadoop/fs/swift/SwiftFileSystemBaseTest.java +++ b/hadoop-tools/hadoop-openstack/src/test/java/org/apache/hadoop/fs/swift/SwiftFileSystemBaseTest.java @@ -233,7 +233,7 @@ protected void rename(Path src, Path dst, boolean renameMustSucceed, /** * Get a string describing the outcome of a rename, by listing the dest * path and its parent along with some covering text - * @param src source patj + * @param src source path * @param dst dest path * @return a string for logs and exceptions * @throws IOException IO problems diff --git a/hadoop-tools/hadoop-openstack/src/test/java/org/apache/hadoop/fs/swift/TestSwiftFileSystemDirectories.java b/hadoop-tools/hadoop-openstack/src/test/java/org/apache/hadoop/fs/swift/TestSwiftFileSystemDirectories.java index ae598201c3..21fe918e7f 100644 --- a/hadoop-tools/hadoop-openstack/src/test/java/org/apache/hadoop/fs/swift/TestSwiftFileSystemDirectories.java +++ b/hadoop-tools/hadoop-openstack/src/test/java/org/apache/hadoop/fs/swift/TestSwiftFileSystemDirectories.java @@ -28,7 +28,7 @@ /** * Test swift-specific directory logic. - * This class is HDFS-1 compatible; its designed to be subclases by something + * This class is HDFS-1 compatible; its designed to be subclasses by something * with HDFS2 extensions */ public class TestSwiftFileSystemDirectories extends SwiftFileSystemBaseTest { diff --git a/hadoop-tools/hadoop-openstack/src/test/java/org/apache/hadoop/fs/swift/TestSwiftFileSystemExtendedContract.java b/hadoop-tools/hadoop-openstack/src/test/java/org/apache/hadoop/fs/swift/TestSwiftFileSystemExtendedContract.java index 967929b42c..844463db6d 100644 --- a/hadoop-tools/hadoop-openstack/src/test/java/org/apache/hadoop/fs/swift/TestSwiftFileSystemExtendedContract.java +++ b/hadoop-tools/hadoop-openstack/src/test/java/org/apache/hadoop/fs/swift/TestSwiftFileSystemExtendedContract.java @@ -131,7 +131,7 @@ public void testFilesystemIsCaseSensitive() throws Exception { out.writeUTF("l"); out.close(); assertExists("lower case file", lower); - //verifEy the length of the upper file hasn't changed + //verify the length of the upper file hasn't changed assertExists("Original upper case file " + upper, upper); FileStatus newStatus = fs.getFileStatus(upper); assertEquals("Expected status:" + upperStatus