HDFS-2886. CreateEditLogs should generate a realistic edit log. Contributed by Konstantin Shvachko.
git-svn-id: https://svn.apache.org/repos/asf/hadoop/common/branches/branch-0.23@1241298 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
parent
c1596a6c3b
commit
0c7f4cddce
|
@ -1565,6 +1565,8 @@ Release 0.22.1 - Unreleased
|
|||
|
||||
HDFS-2718. Optimize OP_ADD in edits loading. (shv)
|
||||
|
||||
HDFS-2886. CreateEditLogs should generate a realistic edit log. (shv)
|
||||
|
||||
BUG FIXES
|
||||
|
||||
HDFS-2877. If locking of a storage dir fails, it will remove the other
|
||||
|
|
|
@ -333,12 +333,9 @@ public class FSDirectory implements Closeable {
|
|||
*/
|
||||
void updateFile(INodeFile file,
|
||||
String path,
|
||||
PermissionStatus permissions,
|
||||
BlockInfo[] blocks,
|
||||
short replication,
|
||||
long mtime,
|
||||
long atime,
|
||||
long preferredBlockSize) throws IOException {
|
||||
long atime) throws IOException {
|
||||
|
||||
// Update the salient file attributes.
|
||||
file.setAccessTime(atime);
|
||||
|
|
|
@ -203,9 +203,8 @@ public class FSEditLogLoader {
|
|||
addCloseOp.mtime, addCloseOp.atime, blockSize,
|
||||
addCloseOp.clientName, addCloseOp.clientMachine);
|
||||
} else {
|
||||
fsDir.updateFile(oldFile,
|
||||
addCloseOp.path, permissions, blocks, replication,
|
||||
addCloseOp.mtime, addCloseOp.atime, blockSize);
|
||||
fsDir.updateFile(oldFile, addCloseOp.path, blocks,
|
||||
addCloseOp.mtime, addCloseOp.atime);
|
||||
if(addCloseOp.opCode == FSEditLogOpCodes.OP_CLOSE) { // OP_CLOSE
|
||||
assert oldFile.isUnderConstruction() :
|
||||
"File is not under construction: " + addCloseOp.path;
|
||||
|
|
|
@ -93,7 +93,9 @@ public class CreateEditsLog {
|
|||
dirInode = new INodeDirectory(p, 0L);
|
||||
editLog.logMkDir(currentDir, dirInode);
|
||||
}
|
||||
editLog.logOpenFile(filePath, inode);
|
||||
editLog.logOpenFile(filePath,
|
||||
new INodeFileUnderConstruction(
|
||||
p, replication, 0, blockSize, "", "", null));
|
||||
editLog.logCloseFile(filePath, inode);
|
||||
|
||||
if (currentBlockId - bidAtSync >= 2000) { // sync every 2K blocks
|
||||
|
|
|
@ -128,8 +128,8 @@ public class TestEditLog extends TestCase {
|
|||
for (int i = 0; i < numTransactions; i++) {
|
||||
INodeFileUnderConstruction inode = new INodeFileUnderConstruction(
|
||||
p, replication, blockSize, 0, "", "", null);
|
||||
editLog.logOpenFile("/filename" + startIndex + i, inode);
|
||||
editLog.logCloseFile("/filename" + startIndex + i, inode);
|
||||
editLog.logOpenFile("/filename" + (startIndex + i), inode);
|
||||
editLog.logCloseFile("/filename" + (startIndex + i), inode);
|
||||
editLog.logSync();
|
||||
}
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue