HBase
  1. HBase
  2. HBASE-4384

Hard to tell what causes failure in CloseRegionHandler#getCurrentVersion

    Details

    • Type: Task Task
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 0.90.0
    • Fix Version/s: 0.90.5
    • Component/s: Zookeeper
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      The current code goes like:

      172    * Get the node's current version
      173    * @return The expectedVersion.  If -1, we failed getting the node
      174    */
      175   private int getCurrentVersion() {
      176     int expectedVersion = FAILED;
      177     try {
      178       if ((expectedVersion = ZKAssign.getVersion(
      179           server.getZooKeeper(), regionInfo)) == FAILED) {
      180         LOG.warn("Error getting node's version in CLOSING state," +
      181           " aborting close of " + regionInfo.getRegionNameAsString());
      182       }
      183     } catch (KeeperException e) {
      184       LOG.warn("Error creating node in CLOSING state, aborting close of " +
      185         regionInfo.getRegionNameAsString());
      186     }
      187     return expectedVersion;
      188   }
      189 }
      

      Both WARN cases would be identical this way. In case of an exception, I think an exception ought to be logged as well.

        Activity

        Hide
        Hudson added a comment -

        Integrated in HBase-TRUNK #2206 (See https://builds.apache.org/job/HBase-TRUNK/2206/)
        HBASE-4384 Hard to tell what causes failure in CloseRegionHandler#getCurrentVersion

        stack :
        Files :

        • /hbase/trunk/CHANGES.txt
        • /hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/handler/CloseRegionHandler.java
        Show
        Hudson added a comment - Integrated in HBase-TRUNK #2206 (See https://builds.apache.org/job/HBase-TRUNK/2206/ ) HBASE-4384 Hard to tell what causes failure in CloseRegionHandler#getCurrentVersion stack : Files : /hbase/trunk/CHANGES.txt /hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/handler/CloseRegionHandler.java
        Hide
        stack added a comment -

        Committed branch and trunk. Thanks for the patch Harsh.

        Show
        stack added a comment - Committed branch and trunk. Thanks for the patch Harsh.
        Hide
        Harsh J added a comment -

        stack, No, patch is for all branches 0.90 to trunk. Please disregard my first comment, it was made when I was under a great deal of workspace switching and I thought I was looking at one snippet of trunk source, while I was looking at something else instead.

        This patch is targeted for trunk, but can also be backported atop other branches (0.92 if branched already, and 0.90).

        Show
        Harsh J added a comment - stack, No, patch is for all branches 0.90 to trunk. Please disregard my first comment, it was made when I was under a great deal of workspace switching and I thought I was looking at one snippet of trunk source, while I was looking at something else instead. This patch is targeted for trunk, but can also be backported atop other branches (0.92 if branched already, and 0.90).
        Hide
        stack added a comment -

        @Harsh So the patch is for 0.90 branch?

        Show
        stack added a comment - @Harsh So the patch is for 0.90 branch?
        Hide
        Harsh J added a comment -

        (Updated topic comment/desc.)

        Show
        Harsh J added a comment - (Updated topic comment/desc.)
        Hide
        Harsh J added a comment -

        Mehh... Confusions.

        Show
        Harsh J added a comment - Mehh... Confusions.
        Hide
        Harsh J added a comment -

        Ah, this isn't a problem on trunk. The method appears to have been refactored.

        Show
        Harsh J added a comment - Ah, this isn't a problem on trunk. The method appears to have been refactored.

          People

          • Assignee:
            Harsh J
            Reporter:
            Harsh J
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development