[
https://issues.apache.org/jira/browse/HDFS-13767?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16568674#comment-16568674
]
Plamen Jeliazkov commented on HDFS-13767:
-----------------------------------------
Hey Chen,
Change looks pretty good so far; two things.
(1) I like the idea of splitting AlignmentContext into client and server
interfaces. I think originally Konstantin wanted just a single interface but I
think its clear there is a difference. I would not mind picking up that JIRA if
we want to go ahead with it. This would let us allow the server-side version of
AlignmentContext to better handle defering / re-queue'ing calls.
(2) A concern about the unit test -- I tried to add a 10 second sleep between
the thread start and the assert:
{code:java}
assertFalse(readSucceed.get());
{code}
However I found that the test would fail if I did. Which makes me question the
unit test.
I would expect the 'getFileStatus' to basically hang until the
{{{rollEditLogAndTail}}} call but seems it is updating anyway.
I tried also stopping the {{{EditLogTailer}}} on the ObserverNode but it still
updated anyway.
Please let me know if I am missing something about this test; I will look
further.
> Add msync server implementation.
> --------------------------------
>
> Key: HDFS-13767
> URL: https://issues.apache.org/jira/browse/HDFS-13767
> Project: Hadoop HDFS
> Issue Type: Sub-task
> Components: namenode
> Reporter: Chen Liang
> Assignee: Chen Liang
> Priority: Major
> Attachments: HDFS-13767-HDFS-12943.001.patch,
> HDFS-13767-HDFS-12943.002.patch, HDFS-13767.WIP.001.patch,
> HDFS-13767.WIP.002.patch, HDFS-13767.WIP.003.patch, HDFS-13767.WIP.004.patch
>
>
> This is a followup on HDFS-13688, where msync API is introduced to
> {{ClientProtocol}} but the server side implementation is missing. This is
> Jira is to implement the server side logic.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
