[ 
https://issues.apache.org/jira/browse/HDFS-13609?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16520734#comment-16520734
 ] 

Erik Krogen edited comment on HDFS-13609 at 6/22/18 7:57 PM:
-------------------------------------------------------------

Thanks for looking at {{BackupImage}} [~shv]! I had no idea what was happening 
there :) I have removed this extra parameter altogether; now on the QJM 
{{optimizeLatency == inProgressOK}}. This reduced the scope of changes 
significantly.

[~linyiqun], thanks for taking a look! I agree with you on the Precondition 
check; I have incorporated into v003. For your second comment, it should be {{< 
0}}. A return value of {{highestTxnCount == 0}} is expected behavior if you 
have read all available edits and continue to request more; this is not an 
error situation, while seeing a value {{< 0}} is an error. Let me know if you 
disagree.

Uploaded v003 patch incorporating all of [~shv] and [~linyiqun]'s comments.


was (Author: xkrogen):
Thanks for looking at {{BackupImage}} [~shv]! I had no idea what was happening 
there :) I have removed this extra parameter altogether; now on the QJM 
{{optimizeLatency == inProgressOK}}. This reduced the scope of changes 
significantly.

[~linyiqun], thanks for taking a look! I agree with you on the Precondition 
check; I have incorporated into v003. For your second comment, it should be {{ 
< 0 }}. A return value of {{highestTxnCount == 0}} is expected behavior if you 
have read all available edits and continue to request more; this is not an 
error situation, while seeing a value {{ < 0 }} is an error. Let me know if you 
disagree.

Uploaded v003 patch incorporating all of [~shv] and [~linyiqun]'s comments.

> [Edit Tail Fast Path Pt 3] NameNode-side changes to support tailing edits via 
> RPC
> ---------------------------------------------------------------------------------
>
>                 Key: HDFS-13609
>                 URL: https://issues.apache.org/jira/browse/HDFS-13609
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: ha, namenode
>            Reporter: Erik Krogen
>            Assignee: Erik Krogen
>            Priority: Major
>         Attachments: HDFS-13609-HDFS-12943.000.patch, 
> HDFS-13609-HDFS-12943.001.patch, HDFS-13609-HDFS-12943.002.patch, 
> HDFS-13609-HDFS-12943.003.patch
>
>
> See HDFS-13150 for the full design.
> This JIRA is targetted at the NameNode-side changes to enable tailing 
> in-progress edits via the RPC mechanism added in HDFS-13608. Most changes are 
> in the QuorumJournalManager.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org

Reply via email to