[ 
https://issues.apache.org/jira/browse/HDFS-10742?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Chris Douglas updated HDFS-10742:
---------------------------------
    Attachment: HDFS-10742.015.patch

Modified v014 of the patch to show the alternative, since I could not convey 
the design in prose:
* Changed {{AutoClosableLock}} and {{InstrumentedLock}} to wrap a {{Lock}} 
instance, rather than hard-coding {{ReentrantLock}}
* Changed visibility of {{AutoCloseableLock#isLocked}} to package-private for 
unit tests, and annotated same
* Changed time arithmetic to match 
[guidance|https://docs.oracle.com/javase/8/docs/api/java/lang/System.html#nanoTime--]
 for {{System#nanotime}}
* Changed init to not suppress the first lock warning if the first acquisition 
is within the log delay period
* Changed logging check to avoid synchronizing on the instance
* Changed the unit test to use {{Timer}} instead of {{Thread#sleep}}

This retains most of the pattern as 
[requested|https://issues.apache.org/jira/browse/HDFS-10742?focusedCommentId=15447205&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15447205].

> Measurement of lock held time in FsDatasetImpl
> ----------------------------------------------
>
>                 Key: HDFS-10742
>                 URL: https://issues.apache.org/jira/browse/HDFS-10742
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>          Components: datanode
>    Affects Versions: 3.0.0-alpha2
>            Reporter: Chen Liang
>            Assignee: Chen Liang
>         Attachments: HDFS-10742.001.patch, HDFS-10742.002.patch, 
> HDFS-10742.003.patch, HDFS-10742.004.patch, HDFS-10742.005.patch, 
> HDFS-10742.006.patch, HDFS-10742.007.patch, HDFS-10742.008.patch, 
> HDFS-10742.009.patch, HDFS-10742.010.patch, HDFS-10742.011.patch, 
> HDFS-10742.012.patch, HDFS-10742.013.patch, HDFS-10742.014.patch, 
> HDFS-10742.015.patch
>
>
> This JIRA proposes to measure the time the of lock of {{FsDatasetImpl}} is 
> held by a thread. Doing so will allow us to measure lock statistics.
> This can be done by extending the {{AutoCloseableLock}} lock object in 
> {{FsDatasetImpl}}. In the future we can also consider replacing the lock with 
> a read-write lock.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
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