HBase
  1. HBase
  2. HBASE-5083

Backup HMaster should have http infoport open with link to the active master

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.92.0
    • Fix Version/s: 0.94.9
    • Component/s: master
    • Labels:
      None
    • Hadoop Flags:
      Reviewed
    • Release Note:
      This patch includes adding a table for the backup masters to the UI of the master server. It also has a link from the backup master server's UI to point to the primary.

      Description

      Without ssh'ing and jps/ps'ing, it is difficult to see if a backup hmaster is up. It seems like it would be good for a backup hmaster to have a basic web page up on the info port so that users could see that it is up. Also it should probably either provide a link to the active master or automatically forward to the active master.

      1. 5083-0.94-v2.txt
        18 kB
        Lars Hofhansl
      2. 5083-trunk-v2.txt
        19 kB
        Lars Hofhansl
      3. backup_master.png
        144 kB
        Cody Marcel
      4. HBASE-5083_trunk.patch
        20 kB
        Lars Hofhansl
      5. HBASE-5083_trunk.patch
        20 kB
        Lars Hofhansl
      6. HBASE-5083_trunk.patch
        20 kB
        Cody Marcel
      7. HBASE-5083.patch
        19 kB
        Cody Marcel
      8. HBASE-5083.patch
        19 kB
        Cody Marcel
      9. HBASE-5083.patch
        19 kB
        Cody Marcel
      10. HBASE-5083.patch
        16 kB
        Cody Marcel
      11. HBASE-5083.patch
        17 kB
        Cody Marcel
      12. master.png
        164 kB
        Cody Marcel
      13. Trunk_Backup_Master.png
        159 kB
        Cody Marcel
      14. Trunk_Master.png
        129 kB
        Cody Marcel

        Issue Links

          Activity

          Jonathan Hsieh created issue -
          Hide
          stack added a comment -

          Related, we should make sure master shows a webpage even if meta is not deployed. Currently meta has to be deployed for master page to come up.

          I'd think backup master webpage would also point to the actual master if one.

          Show
          stack added a comment - Related, we should make sure master shows a webpage even if meta is not deployed. Currently meta has to be deployed for master page to come up. I'd think backup master webpage would also point to the actual master if one.
          Hide
          Jonathan Hsieh added a comment -

          Agreed and can probably be handled in the same patch. (updated description to add the missing other option).

          Show
          Jonathan Hsieh added a comment - Agreed and can probably be handled in the same patch. (updated description to add the missing other option).
          Jonathan Hsieh made changes -
          Field Original Value New Value
          Description Without ssh'ing and jps/ps'ing, it is difficult to see if a backup hmaster is up. It seems like it would be good for a backup hmaster to have a basic web page up on the info port so that users could see that it is up. Also it should probably either provide a link to the active master. Without ssh'ing and jps/ps'ing, it is difficult to see if a backup hmaster is up. It seems like it would be good for a backup hmaster to have a basic web page up on the info port so that users could see that it is up. Also it should probably either provide a link to the active master or automatically forward to the active master.
          Jonathan Hsieh made changes -
          Assignee Jonathan Hsieh [ jmhsieh ]
          Jonathan Hsieh made changes -
          Link This issue is related to HBASE-6367 [ HBASE-6367 ]
          Hide
          Lars Hofhansl added a comment - - edited

          This is somewhat annoying for us. I'll put up a simple patch next week. For 0.94 I will also backport HBASE-6367 as part of this.

          Show
          Lars Hofhansl added a comment - - edited This is somewhat annoying for us. I'll put up a simple patch next week. For 0.94 I will also backport HBASE-6367 as part of this.
          Hide
          Jonathan Hsieh added a comment -

          removing myself from this for the time being.

          Show
          Jonathan Hsieh added a comment - removing myself from this for the time being.
          Jonathan Hsieh made changes -
          Assignee Jonathan Hsieh [ jmhsieh ]
          Cody Marcel made changes -
          Assignee Cody Marcel [ cody.marcel@gmail.com ]
          Hide
          Cody Marcel added a comment -

          I tried setting up a review for this, but it failed: The file 'src/main/jamon/org/apache/hadoop/hbase/tmpl/master/BackupMasterStatusTmpl.jamon' (r238d2ad) could not be found in the repository

          Show
          Cody Marcel added a comment - I tried setting up a review for this, but it failed: The file 'src/main/jamon/org/apache/hadoop/hbase/tmpl/master/BackupMasterStatusTmpl.jamon' (r238d2ad) could not be found in the repository
          Cody Marcel made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Release Note This patch includes adding a table for the backup masters to the UI of the master server. It also has a link from the backup master server's UI to point to the primary.
          Fix Version/s 0.94.8 [ 12324145 ]
          Hide
          Cody Marcel added a comment -

          Screen shots for the UI

          Show
          Cody Marcel added a comment - Screen shots for the UI
          Cody Marcel made changes -
          Attachment backup_master.png [ 12585100 ]
          Attachment master.png [ 12585101 ]
          Cody Marcel made changes -
          Attachment HBASE-5083.patch [ 12585102 ]
          Hide
          Jesse Yates added a comment -

          hmmm, something is wonky with this patch... Looks like its diffing itself. Wanna try again?

          Show
          Jesse Yates added a comment - hmmm, something is wonky with this patch... Looks like its diffing itself. Wanna try again?
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12585102/HBASE-5083.patch
          against trunk revision .

          +1 @author. The patch does not contain any @author tags.

          -1 tests included. The patch doesn't appear to include any new or modified tests.
          Please justify why no new tests are needed for this patch.
          Also please list what manual steps were performed to verify this patch.

          -1 patch. The patch command could not apply the patch.

          Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/5853//console

          This message is automatically generated.

          Show
          Hadoop QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12585102/HBASE-5083.patch against trunk revision . +1 @author . The patch does not contain any @author tags. -1 tests included . The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. -1 patch . The patch command could not apply the patch. Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/5853//console This message is automatically generated.
          Hide
          Jonathan Hsieh added a comment -

          looks very 0.94. there will be a version for trunk as well?

          Show
          Jonathan Hsieh added a comment - looks very 0.94. there will be a version for trunk as well?
          Hide
          Cody Marcel added a comment -

          Merged local commits to correct the diff.

          Show
          Cody Marcel added a comment - Merged local commits to correct the diff.
          Cody Marcel made changes -
          Attachment HBASE-5083.patch [ 12585108 ]
          Hide
          Cody Marcel added a comment -

          Jonathan Hsieh THis already exists in trunk, mostly. I am backporting to 0.94.

          Show
          Cody Marcel added a comment - Jonathan Hsieh THis already exists in trunk, mostly. I am backporting to 0.94.
          Hide
          Jonathan Hsieh added a comment -

          I didn't realize that (I've been testing against at 0.95.0 and it doesn't seem to be there). Can you find the jira, add a link to it and also rename this to be a backport? If we port to 0.94 we should have it in 0.95 as well!

          Show
          Jonathan Hsieh added a comment - I didn't realize that (I've been testing against at 0.95.0 and it doesn't seem to be there). Can you find the jira, add a link to it and also rename this to be a backport? If we port to 0.94 we should have it in 0.95 as well!
          Hide
          Cody Marcel added a comment -

          Review board is up now with the fixed diff.

          Show
          Cody Marcel added a comment - Review board is up now with the fixed diff.
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12585108/HBASE-5083.patch
          against trunk revision .

          +1 @author. The patch does not contain any @author tags.

          -1 tests included. The patch doesn't appear to include any new or modified tests.
          Please justify why no new tests are needed for this patch.
          Also please list what manual steps were performed to verify this patch.

          -1 patch. The patch command could not apply the patch.

          Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/5855//console

          This message is automatically generated.

          Show
          Hadoop QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12585108/HBASE-5083.patch against trunk revision . +1 @author . The patch does not contain any @author tags. -1 tests included . The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. -1 patch . The patch command could not apply the patch. Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/5855//console This message is automatically generated.
          Cody Marcel made changes -
          Summary Backup HMaster should have http infoport open with link to the active master Backport from trunk: Backup HMaster should have http infoport open with link to the active master
          Lars Hofhansl made changes -
          Fix Version/s 0.94.9 [ 12324431 ]
          Fix Version/s 0.94.8 [ 12324145 ]
          Hide
          Cody Marcel added a comment -

          Added a test fix to the patch.

          Show
          Cody Marcel added a comment - Added a test fix to the patch.
          Cody Marcel made changes -
          Attachment HBASE-5083.patch [ 12585259 ]
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12585259/HBASE-5083.patch
          against trunk revision .

          +1 @author. The patch does not contain any @author tags.

          +1 tests included. The patch appears to include 3 new or modified tests.

          -1 patch. The patch command could not apply the patch.

          Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/5868//console

          This message is automatically generated.

          Show
          Hadoop QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12585259/HBASE-5083.patch against trunk revision . +1 @author . The patch does not contain any @author tags. +1 tests included . The patch appears to include 3 new or modified tests. -1 patch . The patch command could not apply the patch. Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/5868//console This message is automatically generated.
          Hide
          Lars Hofhansl added a comment -

          So to be clear only the portion that lists the backup master(s) on the active HMaster's UI is backported. The link to the active HMaster on all the backup HMaster (and hence the early initialization of the Info Servlet is new). Calling this a backport will be confusing later.

          Show
          Lars Hofhansl added a comment - So to be clear only the portion that lists the backup master(s) on the active HMaster's UI is backported. The link to the active HMaster on all the backup HMaster (and hence the early initialization of the Info Servlet is new). Calling this a backport will be confusing later.
          Hide
          Lars Hofhansl added a comment -

          LGTM +1

          Show
          Lars Hofhansl added a comment - LGTM +1
          Hide
          Lars Hofhansl added a comment -

          Any further comments?
          I would like to commit this unless I hear objections.

          Show
          Lars Hofhansl added a comment - Any further comments? I would like to commit this unless I hear objections.
          Hide
          Jonathan Hsieh added a comment -

          In the review board I posted a concern about a hardcoded info port. Can we at least make it the configuration's value.

          Show
          Jonathan Hsieh added a comment - In the review board I posted a concern about a hardcoded info port. Can we at least make it the configuration's value.
          Hide
          Lars Hofhansl added a comment -

          Yep... Missed that. Good catch. You sound like you have other concerns?

          Show
          Lars Hofhansl added a comment - Yep... Missed that. Good catch. You sound like you have other concerns?
          Hide
          Cody Marcel added a comment -

          Fixed hard coded port.

          Show
          Cody Marcel added a comment - Fixed hard coded port.
          Cody Marcel made changes -
          Attachment HBASE-5083.patch [ 12586233 ]
          Hide
          Lars Hofhansl added a comment -

          Jonathan Hsieh Are you good with this?

          Show
          Lars Hofhansl added a comment - Jonathan Hsieh Are you good with this?
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12586233/HBASE-5083.patch
          against trunk revision .

          +1 @author. The patch does not contain any @author tags.

          +1 tests included. The patch appears to include 3 new or modified tests.

          -1 patch. The patch command could not apply the patch.

          Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/5941//console

          This message is automatically generated.

          Show
          Hadoop QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12586233/HBASE-5083.patch against trunk revision . +1 @author . The patch does not contain any @author tags. +1 tests included . The patch appears to include 3 new or modified tests. -1 patch . The patch command could not apply the patch. Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/5941//console This message is automatically generated.
          Hide
          Lars Hofhansl added a comment -

          There's still a hardcoded port in the jamon file.

          Show
          Lars Hofhansl added a comment - There's still a hardcoded port in the jamon file.
          Cody Marcel made changes -
          Attachment HBASE-5083.patch [ 12586343 ]
          Hide
          Cody Marcel added a comment -

          Sorry, I accidentally left the last commit off the diff. This one should no longer have any hard coded.

          Show
          Cody Marcel added a comment - Sorry, I accidentally left the last commit off the diff. This one should no longer have any hard coded.
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12586343/HBASE-5083.patch
          against trunk revision .

          +1 @author. The patch does not contain any @author tags.

          +1 tests included. The patch appears to include 3 new or modified tests.

          -1 patch. The patch command could not apply the patch.

          Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/5948//console

          This message is automatically generated.

          Show
          Hadoop QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12586343/HBASE-5083.patch against trunk revision . +1 @author . The patch does not contain any @author tags. +1 tests included . The patch appears to include 3 new or modified tests. -1 patch . The patch command could not apply the patch. Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/5948//console This message is automatically generated.
          Hide
          Lars Hofhansl added a comment -

          Cool.
          HadoopQA can't apply the patch (it uses patch -p0). You need to generate the patch differently with git.

          Show
          Lars Hofhansl added a comment - Cool. HadoopQA can't apply the patch (it uses patch -p0). You need to generate the patch differently with git.
          Hide
          Cody Marcel added a comment -

          I believe Jesse Yates said that hadoopqa goes against trunk so it will fail. Maybe I have it wrong though. Is there a flag I need to add when generating the diff?

          Show
          Cody Marcel added a comment - I believe Jesse Yates said that hadoopqa goes against trunk so it will fail. Maybe I have it wrong though. Is there a flag I need to add when generating the diff?
          Hide
          Jesse Yates added a comment -

          Yeah, it applies against trunk, not 0.94. Also, it tries p0 to p2 to apply the patch, so just using the regular 'git diff' command is fine.

          Show
          Jesse Yates added a comment - Yeah, it applies against trunk, not 0.94. Also, it tries p0 to p2 to apply the patch, so just using the regular 'git diff' command is fine.
          Lars Hofhansl made changes -
          Summary Backport from trunk: Backup HMaster should have http infoport open with link to the active master Backup HMaster should have http infoport open with link to the active master
          Hide
          Lars Hofhansl added a comment -

          This patch has two parts:

          1. Backup HMasters have infoport open with UI to link to the active HMaster (this part is new)
          2. The active HMaster lists all backup master (this is in trunk, and was backported to 0.94).

          Cody Marcel Can you make a patch with just new part against trunk? That patch would then get committed against 0.95 (assuming Stack want's it) and trunk.

          For the 0.94 patch (combined backport and new feature) we should run the test suite locally.

          Show
          Lars Hofhansl added a comment - This patch has two parts: Backup HMasters have infoport open with UI to link to the active HMaster (this part is new) The active HMaster lists all backup master (this is in trunk, and was backported to 0.94). Cody Marcel Can you make a patch with just new part against trunk? That patch would then get committed against 0.95 (assuming Stack want's it) and trunk. For the 0.94 patch (combined backport and new feature) we should run the test suite locally.
          Hide
          Lars Hofhansl added a comment -

          Looks like the trunk patch will need some rearranging.

          Show
          Lars Hofhansl added a comment - Looks like the trunk patch will need some rearranging.
          Cody Marcel made changes -
          Attachment Trunk_Master.png [ 12588397 ]
          Cody Marcel made changes -
          Attachment Trunk_Backup_Master.png [ 12588398 ]
          Cody Marcel made changes -
          Attachment Trunk_Master.png [ 12588397 ]
          Cody Marcel made changes -
          Attachment Trunk_Backup_Master.png [ 12588398 ]
          Cody Marcel made changes -
          Attachment Trunk_Backup_Master.png [ 12588401 ]
          Attachment Trunk_Master.png [ 12588402 ]
          Hide
          Cody Marcel added a comment -

          I added some screen shots for how trunk will look with the changes. I am getting the patch ready now. Let me know if there is something you don't like about the layout.

          Show
          Cody Marcel added a comment - I added some screen shots for how trunk will look with the changes. I am getting the patch ready now. Let me know if there is something you don't like about the layout.
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12588402/Trunk_Master.png
          against trunk revision .

          +1 @author. The patch does not contain any @author tags.

          -1 tests included. The patch doesn't appear to include any new or modified tests.
          Please justify why no new tests are needed for this patch.
          Also please list what manual steps were performed to verify this patch.

          -1 patch. The patch command could not apply the patch.

          Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/6057//console

          This message is automatically generated.

          Show
          Hadoop QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12588402/Trunk_Master.png against trunk revision . +1 @author . The patch does not contain any @author tags. -1 tests included . The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. -1 patch . The patch command could not apply the patch. Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/6057//console This message is automatically generated.
          Hide
          stack added a comment -

          This looks good Cody Marcel Could you do two patches? One to add the info port to backup master and another for the backport rather than combine the two in the one patch?

          Show
          stack added a comment - This looks good Cody Marcel Could you do two patches? One to add the info port to backup master and another for the backport rather than combine the two in the one patch?
          Hide
          Lars Hofhansl added a comment -

          That's my fault. I had advised Cody to do a single patch.
          Want a different jira for the backport, stack?

          Show
          Lars Hofhansl added a comment - That's my fault. I had advised Cody to do a single patch. Want a different jira for the backport, stack ?
          Hide
          Cody Marcel added a comment -

          Trunk patch added.

          Show
          Cody Marcel added a comment - Trunk patch added.
          Cody Marcel made changes -
          Attachment HBASE-5083_trunk.patch [ 12588430 ]
          Hide
          Cody Marcel added a comment -

          Stack I can split it up if you, but I didn't see a clean dividing line since this changes the UI to display different things based on whether it is active or not. If you feel strongly about it, I am happy to oblige. Otherwise, I thinks it's good to go.

          Show
          Cody Marcel added a comment - Stack I can split it up if you, but I didn't see a clean dividing line since this changes the UI to display different things based on whether it is active or not. If you feel strongly about it, I am happy to oblige. Otherwise, I thinks it's good to go.
          Hide
          stack added a comment -

          Cody Marcel No worries. We can commit this to 0.94. How though does trunk get what it is missing added? Thanks.

          Show
          stack added a comment - Cody Marcel No worries. We can commit this to 0.94. How though does trunk get what it is missing added? Thanks.
          Hide
          Cody Marcel added a comment -

          The latest patch has the trunk version of the files. I am not sure why Hadoop QA didn't run on it. The trunk version should pass.

          Show
          Cody Marcel added a comment - The latest patch has the trunk version of the files. I am not sure why Hadoop QA didn't run on it. The trunk version should pass.
          Lars Hofhansl made changes -
          Status Patch Available [ 10002 ] Open [ 1 ]
          Hide
          Lars Hofhansl added a comment -

          Reattaching trunk patch for Hadoop QA

          Show
          Lars Hofhansl added a comment - Reattaching trunk patch for Hadoop QA
          Lars Hofhansl made changes -
          Attachment HBASE-5083_trunk.patch [ 12589124 ]
          Lars Hofhansl made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Hide
          Lars Hofhansl added a comment -

          Looks like HadoopQA has problems:

          Linux asf002.sp2.ygridcore.net 2.6.32-33-server #71-Ubuntu SMP Wed Jul 20 17:42:25 UTC 2011 x86_64 GNU/Linux
          /tmp/hudson5785289026968215041.sh: line 5: java: command not found

          I think we can commit this with visual inspection.
          Cody Marcel Could you run the full trunk and 0.94 test suite locally (feel free to use my machine )

          Show
          Lars Hofhansl added a comment - Looks like HadoopQA has problems: Linux asf002.sp2.ygridcore.net 2.6.32-33-server #71-Ubuntu SMP Wed Jul 20 17:42:25 UTC 2011 x86_64 GNU/Linux /tmp/hudson5785289026968215041.sh: line 5: java: command not found I think we can commit this with visual inspection. Cody Marcel Could you run the full trunk and 0.94 test suite locally (feel free to use my machine )
          Hide
          Lars Hofhansl added a comment -

          And again

          Show
          Lars Hofhansl added a comment - And again
          Lars Hofhansl made changes -
          Attachment HBASE-5083_trunk.patch [ 12589380 ]
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12589380/HBASE-5083_trunk.patch
          against trunk revision .

          +1 @author. The patch does not contain any @author tags.

          +1 tests included. The patch appears to include 3 new or modified tests.

          +1 hadoop1.0. The patch compiles against the hadoop 1.0 profile.

          +1 hadoop2.0. The patch compiles against the hadoop 2.0 profile.

          +1 javadoc. The javadoc tool did not generate any warning messages.

          +1 javac. The applied patch does not increase the total number of javac compiler warnings.

          +1 findbugs. The patch does not introduce any new Findbugs (version 1.3.9) warnings.

          +1 release audit. The applied patch does not increase the total number of release audit warnings.

          -1 lineLengths. The patch introduces lines longer than 100

          +1 site. The mvn site goal succeeds with this patch.

          +1 core tests. The patch passed unit tests in .

          Test results: https://builds.apache.org/job/PreCommit-HBASE-Build/6117//testReport/
          Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/6117//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-prefix-tree.html
          Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/6117//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-client.html
          Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/6117//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-common.html
          Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/6117//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-protocol.html
          Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/6117//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-server.html
          Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/6117//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-hadoop1-compat.html
          Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/6117//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-examples.html
          Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/6117//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-hadoop-compat.html
          Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/6117//console

          This message is automatically generated.

          Show
          Hadoop QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12589380/HBASE-5083_trunk.patch against trunk revision . +1 @author . The patch does not contain any @author tags. +1 tests included . The patch appears to include 3 new or modified tests. +1 hadoop1.0 . The patch compiles against the hadoop 1.0 profile. +1 hadoop2.0 . The patch compiles against the hadoop 2.0 profile. +1 javadoc . The javadoc tool did not generate any warning messages. +1 javac . The applied patch does not increase the total number of javac compiler warnings. +1 findbugs . The patch does not introduce any new Findbugs (version 1.3.9) warnings. +1 release audit . The applied patch does not increase the total number of release audit warnings. -1 lineLengths . The patch introduces lines longer than 100 +1 site . The mvn site goal succeeds with this patch. +1 core tests . The patch passed unit tests in . Test results: https://builds.apache.org/job/PreCommit-HBASE-Build/6117//testReport/ Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/6117//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-prefix-tree.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/6117//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-client.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/6117//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-common.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/6117//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-protocol.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/6117//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-server.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/6117//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-hadoop1-compat.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/6117//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-examples.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/6117//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-hadoop-compat.html Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/6117//console This message is automatically generated.
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12589380/HBASE-5083_trunk.patch
          against trunk revision .

          +1 @author. The patch does not contain any @author tags.

          +1 tests included. The patch appears to include 3 new or modified tests.

          +1 hadoop1.0. The patch compiles against the hadoop 1.0 profile.

          +1 hadoop2.0. The patch compiles against the hadoop 2.0 profile.

          +1 javadoc. The javadoc tool did not generate any warning messages.

          +1 javac. The applied patch does not increase the total number of javac compiler warnings.

          +1 findbugs. The patch does not introduce any new Findbugs (version 1.3.9) warnings.

          +1 release audit. The applied patch does not increase the total number of release audit warnings.

          -1 lineLengths. The patch introduces lines longer than 100

          +1 site. The mvn site goal succeeds with this patch.

          -1 core tests. The patch failed these unit tests:
          org.apache.hadoop.hbase.mapreduce.TestHFileOutputFormat

          Test results: https://builds.apache.org/job/PreCommit-HBASE-Build/6118//testReport/
          Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/6118//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-protocol.html
          Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/6118//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-client.html
          Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/6118//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-examples.html
          Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/6118//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-hadoop1-compat.html
          Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/6118//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-prefix-tree.html
          Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/6118//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-common.html
          Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/6118//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-server.html
          Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/6118//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-hadoop-compat.html
          Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/6118//console

          This message is automatically generated.

          Show
          Hadoop QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12589380/HBASE-5083_trunk.patch against trunk revision . +1 @author . The patch does not contain any @author tags. +1 tests included . The patch appears to include 3 new or modified tests. +1 hadoop1.0 . The patch compiles against the hadoop 1.0 profile. +1 hadoop2.0 . The patch compiles against the hadoop 2.0 profile. +1 javadoc . The javadoc tool did not generate any warning messages. +1 javac . The applied patch does not increase the total number of javac compiler warnings. +1 findbugs . The patch does not introduce any new Findbugs (version 1.3.9) warnings. +1 release audit . The applied patch does not increase the total number of release audit warnings. -1 lineLengths . The patch introduces lines longer than 100 +1 site . The mvn site goal succeeds with this patch. -1 core tests . The patch failed these unit tests: org.apache.hadoop.hbase.mapreduce.TestHFileOutputFormat Test results: https://builds.apache.org/job/PreCommit-HBASE-Build/6118//testReport/ Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/6118//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-protocol.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/6118//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-client.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/6118//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-examples.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/6118//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-hadoop1-compat.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/6118//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-prefix-tree.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/6118//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-common.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/6118//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-server.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/6118//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-hadoop-compat.html Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/6118//console This message is automatically generated.
          Hide
          Lars Hofhansl added a comment -

          I triggered HadoopQA manually. The test is unrelated.
          I'll break the > 100 character line upon commmit.

          Show
          Lars Hofhansl added a comment - I triggered HadoopQA manually. The test is unrelated. I'll break the > 100 character line upon commmit.
          Hide
          Lars Hofhansl added a comment -

          The long lines are in the template. I won't break those, as it is not clear whether all browsers know how to deal with whitespace in Urls.

          Show
          Lars Hofhansl added a comment - The long lines are in the template. I won't break those, as it is not clear whether all browsers know how to deal with whitespace in Urls.
          Hide
          Lars Hofhansl added a comment -

          +1

          Show
          Lars Hofhansl added a comment - +1
          Hide
          Lars Hofhansl added a comment -

          Any objections to a commit today?

          Show
          Lars Hofhansl added a comment - Any objections to a commit today?
          Hide
          Lars Hofhansl added a comment -

          Anybody?

          Show
          Lars Hofhansl added a comment - Anybody?
          Hide
          stack added a comment -

          I skimmed. LGTM.

          Show
          stack added a comment - I skimmed. LGTM.
          Hide
          Jesse Yates added a comment - - edited

          Looking at the trunk patch, a couple of nits:
          in hbase-server/src/main/jamon/org/apache/hadoop/hbase/tmpl/master/BackupMasterStatusTmpl.jamon

          +Copyright 2011 The Apache Software Foundation

          isn't needed.

          + </tr>
          + <%java>
          + Arrays.sort(serverNames);
          + for (ServerName serverName: serverNames) {
          + </%java>

          Spacing looks a little bit off.

          In HMaster,

          + if(master.isActiveMaster())

          Unknown macro: { + metaLocation = getMetaLocationOrNull(master); + //ServerName metaLocation = master.getCatalogTracker().getMetaLocation(); + servers = master.getServerManager().getOnlineServersList(); + deadServers = master.getServerManager().getDeadServers().copyServerNames(); + }

          Spacing is off (everything else looks to be 2 spaces, not 4 (or is that a tab? can't tell just reading the text diff)).

          + return (master.getCatalogTracker() == null) ? null : master.getCatalogTracker().getMetaLocation();

          Wish there wasn't a need for the null here and instead a special ServerName that we could use when its null (increases potential for NullPointerExceptions, makes code a little more brittle, requires more null checks other places, etc.), but just complaining - this is fine and fits in with everything else.

          otherwise, it looks fine.

          +1, if you wouldn't mind fixing the nits on commit.

          Show
          Jesse Yates added a comment - - edited Looking at the trunk patch, a couple of nits: in hbase-server/src/main/jamon/org/apache/hadoop/hbase/tmpl/master/BackupMasterStatusTmpl.jamon +Copyright 2011 The Apache Software Foundation isn't needed. + </tr> + <%java> + Arrays.sort(serverNames); + for (ServerName serverName: serverNames) { + </%java> Spacing looks a little bit off. In HMaster, + if(master.isActiveMaster()) Unknown macro: { + metaLocation = getMetaLocationOrNull(master); + //ServerName metaLocation = master.getCatalogTracker().getMetaLocation(); + servers = master.getServerManager().getOnlineServersList(); + deadServers = master.getServerManager().getDeadServers().copyServerNames(); + } Spacing is off (everything else looks to be 2 spaces, not 4 (or is that a tab? can't tell just reading the text diff)). + return (master.getCatalogTracker() == null) ? null : master.getCatalogTracker().getMetaLocation(); Wish there wasn't a need for the null here and instead a special ServerName that we could use when its null (increases potential for NullPointerExceptions, makes code a little more brittle, requires more null checks other places, etc.), but just complaining - this is fine and fits in with everything else. otherwise, it looks fine. +1, if you wouldn't mind fixing the nits on commit.
          Hide
          Jesse Yates added a comment -

          skimmed 0.94 patch, LGTM.

          Show
          Jesse Yates added a comment - skimmed 0.94 patch, LGTM.
          Hide
          Lars Hofhansl added a comment -

          I fixed up the whitespace issues (there were some more).

          Show
          Lars Hofhansl added a comment - I fixed up the whitespace issues (there were some more).
          Hide
          Lars Hofhansl added a comment -

          What I am about to commit.

          Show
          Lars Hofhansl added a comment - What I am about to commit.
          Lars Hofhansl made changes -
          Attachment 5083-trunk-v2.txt [ 12589475 ]
          Lars Hofhansl made changes -
          Status Patch Available [ 10002 ] Open [ 1 ]
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12589475/5083-trunk-v2.txt
          against trunk revision .

          +1 @author. The patch does not contain any @author tags.

          +1 tests included. The patch appears to include 3 new or modified tests.

          -1 patch. The patch command could not apply the patch.

          Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/6127//console

          This message is automatically generated.

          Show
          Hadoop QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12589475/5083-trunk-v2.txt against trunk revision . +1 @author . The patch does not contain any @author tags. +1 tests included . The patch appears to include 3 new or modified tests. -1 patch . The patch command could not apply the patch. Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/6127//console This message is automatically generated.
          Hide
          Lars Hofhansl added a comment -

          And the 0.94 version

          Show
          Lars Hofhansl added a comment - And the 0.94 version
          Lars Hofhansl made changes -
          Attachment 5083-0.94-v2.txt [ 12589478 ]
          Lars Hofhansl made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Hadoop Flags Reviewed [ 10343 ]
          Resolution Fixed [ 1 ]
          Hide
          Cody Marcel added a comment -

          Thanks for fixing up the spaces.

          Show
          Cody Marcel added a comment - Thanks for fixing up the spaces.
          Hide
          Hudson added a comment -

          Integrated in HBase-0.94 #1023 (See https://builds.apache.org/job/HBase-0.94/1023/)
          HBASE-5083 Backup HMaster should have http infoport open with link to the active master (Cody Marcel) (Revision 1496209)

          Result = ABORTED
          larsh :
          Files :

          • /hbase/branches/0.94/src/main/jamon/org/apache/hadoop/hbase/tmpl/master/BackupMasterStatusTmpl.jamon
          • /hbase/branches/0.94/src/main/jamon/org/apache/hadoop/hbase/tmpl/master/MasterStatusTmpl.jamon
          • /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/master/ActiveMasterManager.java
          • /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/master/HMaster.java
          • /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/master/MasterStatusServlet.java
          • /hbase/branches/0.94/src/test/java/org/apache/hadoop/hbase/master/TestMasterStatusServlet.java
          Show
          Hudson added a comment - Integrated in HBase-0.94 #1023 (See https://builds.apache.org/job/HBase-0.94/1023/ ) HBASE-5083 Backup HMaster should have http infoport open with link to the active master (Cody Marcel) (Revision 1496209) Result = ABORTED larsh : Files : /hbase/branches/0.94/src/main/jamon/org/apache/hadoop/hbase/tmpl/master/BackupMasterStatusTmpl.jamon /hbase/branches/0.94/src/main/jamon/org/apache/hadoop/hbase/tmpl/master/MasterStatusTmpl.jamon /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/master/ActiveMasterManager.java /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/master/HMaster.java /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/master/MasterStatusServlet.java /hbase/branches/0.94/src/test/java/org/apache/hadoop/hbase/master/TestMasterStatusServlet.java
          Hide
          Hudson added a comment -

          Integrated in hbase-0.95 #265 (See https://builds.apache.org/job/hbase-0.95/265/)
          HBASE-5083 Delete empty file (Revision 1496204)
          HBASE-5083 Add missing file. (Revision 1496203)
          HBASE-5083 Backup HMaster should have http infoport open with link to the active master (Cody Marcel) (Revision 1496201)

          Result = SUCCESS
          larsh :
          Files :

          • /hbase/branches/0.95/hbase-server/src/main/jamon/org/apache/hadoop/hbase/tmpl/master/BackupMasterListTmpl.jamon

          larsh :
          Files :

          • /hbase/branches/0.95/hbase-server/src/main/jamon/org/apache/hadoop/hbase/tmpl/master/BackupMasterStatusTmpl.jamon

          larsh :
          Files :

          • /hbase/branches/0.95/hbase-server/src/main/jamon/org/apache/hadoop/hbase/tmpl/master/BackupMasterListTmpl.jamon
          • /hbase/branches/0.95/hbase-server/src/main/jamon/org/apache/hadoop/hbase/tmpl/master/MasterStatusTmpl.jamon
          • /hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/master/ActiveMasterManager.java
          • /hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java
          • /hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/master/MasterStatusServlet.java
          • /hbase/branches/0.95/hbase-server/src/test/java/org/apache/hadoop/hbase/master/TestMasterStatusServlet.java
          Show
          Hudson added a comment - Integrated in hbase-0.95 #265 (See https://builds.apache.org/job/hbase-0.95/265/ ) HBASE-5083 Delete empty file (Revision 1496204) HBASE-5083 Add missing file. (Revision 1496203) HBASE-5083 Backup HMaster should have http infoport open with link to the active master (Cody Marcel) (Revision 1496201) Result = SUCCESS larsh : Files : /hbase/branches/0.95/hbase-server/src/main/jamon/org/apache/hadoop/hbase/tmpl/master/BackupMasterListTmpl.jamon larsh : Files : /hbase/branches/0.95/hbase-server/src/main/jamon/org/apache/hadoop/hbase/tmpl/master/BackupMasterStatusTmpl.jamon larsh : Files : /hbase/branches/0.95/hbase-server/src/main/jamon/org/apache/hadoop/hbase/tmpl/master/BackupMasterListTmpl.jamon /hbase/branches/0.95/hbase-server/src/main/jamon/org/apache/hadoop/hbase/tmpl/master/MasterStatusTmpl.jamon /hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/master/ActiveMasterManager.java /hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java /hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/master/MasterStatusServlet.java /hbase/branches/0.95/hbase-server/src/test/java/org/apache/hadoop/hbase/master/TestMasterStatusServlet.java
          Hide
          Hudson added a comment -

          Integrated in hbase-0.95-on-hadoop2 #147 (See https://builds.apache.org/job/hbase-0.95-on-hadoop2/147/)
          HBASE-5083 Delete empty file (Revision 1496204)
          HBASE-5083 Add missing file. (Revision 1496203)
          HBASE-5083 Backup HMaster should have http infoport open with link to the active master (Cody Marcel) (Revision 1496201)

          Result = FAILURE
          larsh :
          Files :

          • /hbase/branches/0.95/hbase-server/src/main/jamon/org/apache/hadoop/hbase/tmpl/master/BackupMasterListTmpl.jamon

          larsh :
          Files :

          • /hbase/branches/0.95/hbase-server/src/main/jamon/org/apache/hadoop/hbase/tmpl/master/BackupMasterStatusTmpl.jamon

          larsh :
          Files :

          • /hbase/branches/0.95/hbase-server/src/main/jamon/org/apache/hadoop/hbase/tmpl/master/BackupMasterListTmpl.jamon
          • /hbase/branches/0.95/hbase-server/src/main/jamon/org/apache/hadoop/hbase/tmpl/master/MasterStatusTmpl.jamon
          • /hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/master/ActiveMasterManager.java
          • /hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java
          • /hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/master/MasterStatusServlet.java
          • /hbase/branches/0.95/hbase-server/src/test/java/org/apache/hadoop/hbase/master/TestMasterStatusServlet.java
          Show
          Hudson added a comment - Integrated in hbase-0.95-on-hadoop2 #147 (See https://builds.apache.org/job/hbase-0.95-on-hadoop2/147/ ) HBASE-5083 Delete empty file (Revision 1496204) HBASE-5083 Add missing file. (Revision 1496203) HBASE-5083 Backup HMaster should have http infoport open with link to the active master (Cody Marcel) (Revision 1496201) Result = FAILURE larsh : Files : /hbase/branches/0.95/hbase-server/src/main/jamon/org/apache/hadoop/hbase/tmpl/master/BackupMasterListTmpl.jamon larsh : Files : /hbase/branches/0.95/hbase-server/src/main/jamon/org/apache/hadoop/hbase/tmpl/master/BackupMasterStatusTmpl.jamon larsh : Files : /hbase/branches/0.95/hbase-server/src/main/jamon/org/apache/hadoop/hbase/tmpl/master/BackupMasterListTmpl.jamon /hbase/branches/0.95/hbase-server/src/main/jamon/org/apache/hadoop/hbase/tmpl/master/MasterStatusTmpl.jamon /hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/master/ActiveMasterManager.java /hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java /hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/master/MasterStatusServlet.java /hbase/branches/0.95/hbase-server/src/test/java/org/apache/hadoop/hbase/master/TestMasterStatusServlet.java
          Hide
          Hudson added a comment -

          Integrated in HBase-TRUNK-on-Hadoop-2.0.0 #582 (See https://builds.apache.org/job/HBase-TRUNK-on-Hadoop-2.0.0/582/)
          HBASE-5083 Backup HMaster should have http infoport open with link to the active master (Cody Marcel) (Revision 1496198)

          Result = FAILURE
          larsh :
          Files :

          • /hbase/trunk/hbase-server/src/main/jamon/org/apache/hadoop/hbase/tmpl/master/BackupMasterListTmpl.jamon
          • /hbase/trunk/hbase-server/src/main/jamon/org/apache/hadoop/hbase/tmpl/master/BackupMasterStatusTmpl.jamon
          • /hbase/trunk/hbase-server/src/main/jamon/org/apache/hadoop/hbase/tmpl/master/MasterStatusTmpl.jamon
          • /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/master/ActiveMasterManager.java
          • /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java
          • /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/master/MasterStatusServlet.java
          • /hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/master/TestMasterStatusServlet.java
          Show
          Hudson added a comment - Integrated in HBase-TRUNK-on-Hadoop-2.0.0 #582 (See https://builds.apache.org/job/HBase-TRUNK-on-Hadoop-2.0.0/582/ ) HBASE-5083 Backup HMaster should have http infoport open with link to the active master (Cody Marcel) (Revision 1496198) Result = FAILURE larsh : Files : /hbase/trunk/hbase-server/src/main/jamon/org/apache/hadoop/hbase/tmpl/master/BackupMasterListTmpl.jamon /hbase/trunk/hbase-server/src/main/jamon/org/apache/hadoop/hbase/tmpl/master/BackupMasterStatusTmpl.jamon /hbase/trunk/hbase-server/src/main/jamon/org/apache/hadoop/hbase/tmpl/master/MasterStatusTmpl.jamon /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/master/ActiveMasterManager.java /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/master/MasterStatusServlet.java /hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/master/TestMasterStatusServlet.java
          Lars Hofhansl made changes -
          Status Resolved [ 5 ] Closed [ 6 ]

            People

            • Assignee:
              Cody Marcel
              Reporter:
              Jonathan Hsieh
            • Votes:
              0 Vote for this issue
              Watchers:
              10 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development