Details
-
Type:
Sub-task
-
Status: Resolved
-
Priority:
Major
-
Resolution: Fixed
-
Affects Version/s: QuorumJournalManager (HDFS-3077)
-
Fix Version/s: QuorumJournalManager (HDFS-3077)
-
Component/s: namenode
-
Labels:None
-
Hadoop Flags:Reviewed
-
Target Version/s:
Description
This API is currently marked TODO. We need it to maintain bounded disk space usage on the JournalNode storage.
Activity
+1, the patch looks good to me.
Show
Aaron T. Myers
added a comment - +1, the patch looks good to me.
Attached patch implements purgeEditLogs() by RPCing to the JNs. It's asynchronous (i.e we don't wait for a quorum), but that's OK because this is just a cleanup method. If it fails, we'll purge those entries next time.
Show
Todd Lipcon
added a comment - Attached patch implements purgeEditLogs() by RPCing to the JNs. It's asynchronous (i.e we don't wait for a quorum), but that's OK because this is just a cleanup method. If it fails, we'll purge those entries next time.
Committed to branch, thanks for the review.