[ https://issues.apache.org/jira/browse/HBASE-24446?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17126954#comment-17126954 ]
Andrew Kyle Purtell commented on HBASE-24446: --------------------------------------------- So this has only been committed to branch-1? With my PMC hat on, let me recommend we stick to established commit order from newest to oldest: master first (if applicable), then branch-2 (if applicable), then the branch-2.x es (if applicable), then branch-1. It's fine to leave this commit in place now assuming the others will follow quickly. > Use EnvironmentEdgeManager to compute clock skew in Master > ---------------------------------------------------------- > > Key: HBASE-24446 > URL: https://issues.apache.org/jira/browse/HBASE-24446 > Project: HBase > Issue Type: Bug > Affects Versions: 1.6.0 > Reporter: Sandeep Guggilam > Assignee: Sandeep Guggilam > Priority: Minor > Fix For: 3.0.0-alpha-1, 1.7.0, 2.4.0 > > > There are few cases where the Master is not able to complete the > initialization as it waiting for the region server to report to it. The > region server actually reported to the master but the master rejected the > request because of clock skew issue though both of them are on same JVM > The Region server uses EnvironmentEdgeManager.currentTime to report the > current time and HMaster uses System.currentTimeMillis() to get the current > time for computation against the reported time by RS. We should also just > use EnvironmentEdgeManager even in Master as we are expected not to use > System.currentTime directly and instead go through EnvironmentEdgeManager > -- This message was sent by Atlassian Jira (v8.3.4#803005)