HBase
  1. HBase
  2. HBASE-6743

Move EnvironmentEdge classes to hbase-common

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.95.0
    • Component/s: None
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      Move the classes related to EnvironmentEdge to the hbase-common module. If we want to replace all occurrences of System.currentTimeMillis() with EnvironmentEdgeManager.currentTimeMillis(), we need to be able to reference these classes from multiple modules.

      1. 6743v1.txt
        6 kB
        Chris Trezzo
      2. 6743v2.txt
        12 kB
        Chris Trezzo
      3. 6743v3.txt
        13 kB
        stack

        Issue Links

          Activity

          Hide
          stack added a comment -

          Marking closed.

          Show
          stack added a comment - Marking closed.
          Hide
          Hudson added a comment -

          Integrated in HBase-TRUNK-on-Hadoop-2.0.0 #197 (See https://builds.apache.org/job/HBase-TRUNK-on-Hadoop-2.0.0/197/)
          HBASE-6743 Move EnvironmentEdge classes to hbase-common (Revision 1391321)

          Result = FAILURE
          stack :
          Files :

          • /hbase/trunk/hbase-common/src/main/java/org/apache/hadoop/hbase/util/DefaultEnvironmentEdge.java
          • /hbase/trunk/hbase-common/src/main/java/org/apache/hadoop/hbase/util/EnvironmentEdge.java
          • /hbase/trunk/hbase-common/src/main/java/org/apache/hadoop/hbase/util/EnvironmentEdgeManager.java
          • /hbase/trunk/hbase-common/src/main/java/org/apache/hadoop/hbase/util/IncrementingEnvironmentEdge.java
          • /hbase/trunk/hbase-common/src/test/java/org/apache/hadoop/hbase/util/EnvironmentEdgeManagerTestHelper.java
          • /hbase/trunk/hbase-common/src/test/java/org/apache/hadoop/hbase/util/TestEnvironmentEdgeManager.java
          • /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/util/DefaultEnvironmentEdge.java
          • /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/util/EnvironmentEdge.java
          • /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/util/EnvironmentEdgeManager.java
          • /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/util/IncrementingEnvironmentEdge.java
          • /hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/util/EnvironmentEdgeManagerTestHelper.java
          • /hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/util/TestEnvironmentEdgeManager.java
          Show
          Hudson added a comment - Integrated in HBase-TRUNK-on-Hadoop-2.0.0 #197 (See https://builds.apache.org/job/HBase-TRUNK-on-Hadoop-2.0.0/197/ ) HBASE-6743 Move EnvironmentEdge classes to hbase-common (Revision 1391321) Result = FAILURE stack : Files : /hbase/trunk/hbase-common/src/main/java/org/apache/hadoop/hbase/util/DefaultEnvironmentEdge.java /hbase/trunk/hbase-common/src/main/java/org/apache/hadoop/hbase/util/EnvironmentEdge.java /hbase/trunk/hbase-common/src/main/java/org/apache/hadoop/hbase/util/EnvironmentEdgeManager.java /hbase/trunk/hbase-common/src/main/java/org/apache/hadoop/hbase/util/IncrementingEnvironmentEdge.java /hbase/trunk/hbase-common/src/test/java/org/apache/hadoop/hbase/util/EnvironmentEdgeManagerTestHelper.java /hbase/trunk/hbase-common/src/test/java/org/apache/hadoop/hbase/util/TestEnvironmentEdgeManager.java /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/util/DefaultEnvironmentEdge.java /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/util/EnvironmentEdge.java /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/util/EnvironmentEdgeManager.java /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/util/IncrementingEnvironmentEdge.java /hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/util/EnvironmentEdgeManagerTestHelper.java /hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/util/TestEnvironmentEdgeManager.java
          Hide
          Hudson added a comment -

          Integrated in HBase-TRUNK #3388 (See https://builds.apache.org/job/HBase-TRUNK/3388/)
          HBASE-6743 Move EnvironmentEdge classes to hbase-common (Revision 1391321)

          Result = SUCCESS
          stack :
          Files :

          • /hbase/trunk/hbase-common/src/main/java/org/apache/hadoop/hbase/util/DefaultEnvironmentEdge.java
          • /hbase/trunk/hbase-common/src/main/java/org/apache/hadoop/hbase/util/EnvironmentEdge.java
          • /hbase/trunk/hbase-common/src/main/java/org/apache/hadoop/hbase/util/EnvironmentEdgeManager.java
          • /hbase/trunk/hbase-common/src/main/java/org/apache/hadoop/hbase/util/IncrementingEnvironmentEdge.java
          • /hbase/trunk/hbase-common/src/test/java/org/apache/hadoop/hbase/util/EnvironmentEdgeManagerTestHelper.java
          • /hbase/trunk/hbase-common/src/test/java/org/apache/hadoop/hbase/util/TestEnvironmentEdgeManager.java
          • /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/util/DefaultEnvironmentEdge.java
          • /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/util/EnvironmentEdge.java
          • /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/util/EnvironmentEdgeManager.java
          • /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/util/IncrementingEnvironmentEdge.java
          • /hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/util/EnvironmentEdgeManagerTestHelper.java
          • /hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/util/TestEnvironmentEdgeManager.java
          Show
          Hudson added a comment - Integrated in HBase-TRUNK #3388 (See https://builds.apache.org/job/HBase-TRUNK/3388/ ) HBASE-6743 Move EnvironmentEdge classes to hbase-common (Revision 1391321) Result = SUCCESS stack : Files : /hbase/trunk/hbase-common/src/main/java/org/apache/hadoop/hbase/util/DefaultEnvironmentEdge.java /hbase/trunk/hbase-common/src/main/java/org/apache/hadoop/hbase/util/EnvironmentEdge.java /hbase/trunk/hbase-common/src/main/java/org/apache/hadoop/hbase/util/EnvironmentEdgeManager.java /hbase/trunk/hbase-common/src/main/java/org/apache/hadoop/hbase/util/IncrementingEnvironmentEdge.java /hbase/trunk/hbase-common/src/test/java/org/apache/hadoop/hbase/util/EnvironmentEdgeManagerTestHelper.java /hbase/trunk/hbase-common/src/test/java/org/apache/hadoop/hbase/util/TestEnvironmentEdgeManager.java /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/util/DefaultEnvironmentEdge.java /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/util/EnvironmentEdge.java /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/util/EnvironmentEdgeManager.java /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/util/IncrementingEnvironmentEdge.java /hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/util/EnvironmentEdgeManagerTestHelper.java /hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/util/TestEnvironmentEdgeManager.java
          Hide
          stack added a comment -

          Committed to trunk. Thanks for the patch Chris.

          Show
          stack added a comment - Committed to trunk. Thanks for the patch Chris.
          Hide
          stack added a comment -

          Added the EnvironmentEdge tests and utils too.

          Show
          stack added a comment - Added the EnvironmentEdge tests and utils too.
          Hide
          Chris Trezzo added a comment -

          Here is an updated patch using git.

          Show
          Chris Trezzo added a comment - Here is an updated patch using git.
          Hide
          Chris Trezzo added a comment -

          Yes. Woops. Moving files with svn can be a pain. Will fix. Thanks!

          Show
          Chris Trezzo added a comment - Yes. Woops. Moving files with svn can be a pain. Will fix. Thanks!
          Hide
          stack added a comment -

          Did you forget to svn add the classes back in hbase-common Chris? They are not in the patch?

          Show
          stack added a comment - Did you forget to svn add the classes back in hbase-common Chris? They are not in the patch?
          Hide
          Chris Trezzo added a comment -

          Trying a HadoopQA run.

          Show
          Chris Trezzo added a comment - Trying a HadoopQA run.
          Hide
          Chris Trezzo added a comment -

          Attached a first patch.

          Show
          Chris Trezzo added a comment - Attached a first patch.

            People

            • Assignee:
              Chris Trezzo
              Reporter:
              Chris Trezzo
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development