You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@hbase.apache.org by "Jean-Marc Spaggiari (JIRA)" <ji...@apache.org> on 2014/02/24 13:43:20 UTC

[jira] [Commented] (HBASE-10599) Replace System.currentMillis() with EnvironmentEdge.currentTimeMillis in memstore flusher and related places

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

Jean-Marc Spaggiari commented on HBASE-10599:
---------------------------------------------

Hi Ramkrishna,

For my knowledge, why should EnvironmentEdge.currentMillis() be prefered to System.currentMillis()?

> Replace System.currentMillis() with EnvironmentEdge.currentTimeMillis in memstore flusher and related places
> ------------------------------------------------------------------------------------------------------------
>
>                 Key: HBASE-10599
>                 URL: https://issues.apache.org/jira/browse/HBASE-10599
>             Project: HBase
>          Issue Type: Improvement
>    Affects Versions: 0.98.0, 0.99.0, 0.96.1.1
>            Reporter: ramkrishna.s.vasudevan
>            Assignee: ramkrishna.s.vasudevan
>             Fix For: 0.96.2, 0.98.1, 0.99.0
>
>
> Memstoreflusher still uses System.currentMillis.  Better to replace it with EnvironmentEdge.currentMillis(),



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)