HDFS-3789. JournalManager#format() should be able to throw IOException. Contributed by Ivan Kelly.

git-svn-id: https://svn.apache.org/repos/asf/hadoop/common/branches/branch-2@1403603 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
Uma Maheswara Rao G 2012-10-30 01:55:39 +00:00
parent 394d07c6a2
commit 59df5eb19f
6 changed files with 8 additions and 5 deletions

View File

@ -95,6 +95,9 @@ Release 2.0.3-alpha - Unreleased
HDFS-3695. Genericize format() to non-file JournalManagers.
(todd via umamahesh)
HDFS-3789. JournalManager#format() should be able to throw IOException.
(Ivan Kelly via umamahesh)
OPTIMIZATIONS
BUG FIXES

View File

@ -272,7 +272,7 @@ public class BookKeeperJournalManager implements JournalManager {
}
@Override
public void format(NamespaceInfo ns) {
public void format(NamespaceInfo ns) throws IOException {
// Currently, BKJM automatically formats itself when first accessed.
// TODO: change over to explicit formatting so that the admin can
// clear out the BK storage when reformatting a cluster.

View File

@ -79,7 +79,7 @@ class FileJournalManager implements JournalManager {
public void close() throws IOException {}
@Override
public void format(NamespaceInfo ns) {
public void format(NamespaceInfo ns) throws IOException {
// Formatting file journals is done by the StorageDirectory
// format code, since they may share their directory with
// checkpoints, etc.

View File

@ -41,7 +41,7 @@ public interface JournalManager extends Closeable, LogsPurgeable,
* Format the underlying storage, removing any previously
* stored data.
*/
void format(NamespaceInfo ns);
void format(NamespaceInfo ns) throws IOException;
/**
* Begin writing to a new segment of the log stream, which starts at

View File

@ -172,7 +172,7 @@ public class JournalSet implements JournalManager {
}
@Override
public void format(NamespaceInfo nsInfo) {
public void format(NamespaceInfo nsInfo) throws IOException {
// The iteration is done by FSEditLog itself
throw new UnsupportedOperationException();
}

View File

@ -157,7 +157,7 @@ public class TestGenericJournalConf {
}
@Override
public void format(NamespaceInfo nsInfo) {
public void format(NamespaceInfo nsInfo) throws IOException {
formatCalled = true;
}