[
https://issues.apache.org/jira/browse/HDFS-2100?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Aaron T. Myers updated HDFS-2100:
-
Attachment: hdfs-2100.1.patch
Updated patch addressing Todd's comments.
{quote}
Seems like you got distracted halfway through writing this comment:
// The created file should still exist in the
{quote}
Quite right. Changed to:
{noformat}
// The created file should still exist after the restart.
{noformat}
{quote}
You might consider using ImmutableSet.of(path1) and ImmutableSet.of(path2,
path3) from guava - it's a bit easier to read IMO but no biggy
{quote}
That's a great tip. I hated writing that code just to create a briefly-lived
set.
{quote}
Can you add a javadoc to explain what bug this is testing for and what the
simulation is? Seems funny that the test case is called
testMultipleSecondasryCheckpoint but it only instantiates one 2NN.
{quote}
Added the following:
{noformat}
/**
* Test to simulate interleaved checkpointing by 2 2NNs after a storage
* directory has been taken offline. The first will cause the directory to
* come back online, but it won't have any valid contents. The second 2NN will
* then try to perform a checkpoint. The NN should not serve up the image or
* edits from the restored (empty) dir.
*/
{noformat}
Improve TestStorageRestore
--
Key: HDFS-2100
URL: https://issues.apache.org/jira/browse/HDFS-2100
Project: Hadoop HDFS
Issue Type: Test
Components: test
Affects Versions: 0.23.0
Reporter: Aaron T. Myers
Assignee: Aaron T. Myers
Priority: Minor
Fix For: 0.23.0
Attachments: hdfs-2100.0.patch, hdfs-2100.1.patch
Though running multiple 2NNs isn't supported, accidentally doing so should
not result in HDFS metadata corruptions. We should add a test case to
exercise this possibility when name.dir.storage.restore is enabled, which is
a particularly delicate code path.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira