[jira] [Commented] (HBASE-4680) FSUtils.isInSafeMode() checks should operate on HBase root dir, where we have permissions

2011-10-31 Thread stack (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-4680?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13140386#comment-13140386
 ] 

stack commented on HBASE-4680:
--

Reverting because of this discussion: 
http://search-hadoop.com/m/7WOjpVyG5F/Hmaster+can%2527t+start+for+the+latest+trunk+versionsubj=Hmaster+can+t+start+for+the+latest+trunk+version



 FSUtils.isInSafeMode() checks should operate on HBase root dir, where we have 
 permissions
 -

 Key: HBASE-4680
 URL: https://issues.apache.org/jira/browse/HBASE-4680
 Project: HBase
  Issue Type: Bug
  Components: util
Affects Versions: 0.92.0, 0.94.0
Reporter: Gary Helmling
Assignee: Gary Helmling
Priority: Critical
 Fix For: 0.92.0

 Attachments: HBASE-4680.patch


 The HDFS safe mode check workaround introduced by HBASE-4510 performs a 
 {{FileSystem.setPermission()}} operation on the root directory (/) when 
 attempting to trigger a {{SafeModeException}}.  As a result, it requires 
 superuser privileges when running with DFS permission checking enabled.  
 Changing the operations to act on the HBase root directory should be safe, 
 since the master process must have write access to it.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HBASE-4680) FSUtils.isInSafeMode() checks should operate on HBase root dir, where we have permissions

2011-10-31 Thread stack (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-4680?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13140391#comment-13140391
 ] 

stack commented on HBASE-4680:
--

I reverted this change.

 FSUtils.isInSafeMode() checks should operate on HBase root dir, where we have 
 permissions
 -

 Key: HBASE-4680
 URL: https://issues.apache.org/jira/browse/HBASE-4680
 Project: HBase
  Issue Type: Bug
  Components: util
Affects Versions: 0.92.0, 0.94.0
Reporter: Gary Helmling
Assignee: Gary Helmling
Priority: Critical
 Fix For: 0.92.0

 Attachments: HBASE-4680.patch


 The HDFS safe mode check workaround introduced by HBASE-4510 performs a 
 {{FileSystem.setPermission()}} operation on the root directory (/) when 
 attempting to trigger a {{SafeModeException}}.  As a result, it requires 
 superuser privileges when running with DFS permission checking enabled.  
 Changing the operations to act on the HBase root directory should be safe, 
 since the master process must have write access to it.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HBASE-4680) FSUtils.isInSafeMode() checks should operate on HBase root dir, where we have permissions

2011-10-31 Thread Hudson (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-4680?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13140585#comment-13140585
 ] 

Hudson commented on HBASE-4680:
---

Integrated in HBase-TRUNK #2392 (See 
[https://builds.apache.org/job/HBase-TRUNK/2392/])
HBASE-4680 FSUtils.isInSafeMode() checks should operate on HBase root dir, 
where we have permissions; REVERT

stack : 
Files : 
* /hbase/trunk/CHANGES.txt
* /hbase/trunk/src/main/java/org/apache/hadoop/hbase/util/FSUtils.java


 FSUtils.isInSafeMode() checks should operate on HBase root dir, where we have 
 permissions
 -

 Key: HBASE-4680
 URL: https://issues.apache.org/jira/browse/HBASE-4680
 Project: HBase
  Issue Type: Bug
  Components: util
Affects Versions: 0.92.0, 0.94.0
Reporter: Gary Helmling
Assignee: Gary Helmling
 Fix For: 0.92.0

 Attachments: HBASE-4680.patch


 The HDFS safe mode check workaround introduced by HBASE-4510 performs a 
 {{FileSystem.setPermission()}} operation on the root directory (/) when 
 attempting to trigger a {{SafeModeException}}.  As a result, it requires 
 superuser privileges when running with DFS permission checking enabled.  
 Changing the operations to act on the HBase root directory should be safe, 
 since the master process must have write access to it.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HBASE-4680) FSUtils.isInSafeMode() checks should operate on HBase root dir, where we have permissions

2011-10-31 Thread Hudson (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-4680?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13140830#comment-13140830
 ] 

Hudson commented on HBASE-4680:
---

Integrated in HBase-0.92 #90 (See 
[https://builds.apache.org/job/HBase-0.92/90/])
HBASE-4680 FSUtils.isInSafeMode() checks should operate on HBase root dir, 
where we have permissions; REVERT
HBASE-4680  FSUtils.isInSafeMode() checks should operate on HBase root dir, 
where we have permissions

stack : 
Files : 
* /hbase/branches/0.92/CHANGES.txt
* /hbase/branches/0.92/src/main/java/org/apache/hadoop/hbase/util/FSUtils.java

garyh : 
Files : 
* /hbase/branches/0.92/CHANGES.txt
* /hbase/branches/0.92/src/main/java/org/apache/hadoop/hbase/util/FSUtils.java


 FSUtils.isInSafeMode() checks should operate on HBase root dir, where we have 
 permissions
 -

 Key: HBASE-4680
 URL: https://issues.apache.org/jira/browse/HBASE-4680
 Project: HBase
  Issue Type: Bug
  Components: util
Affects Versions: 0.92.0, 0.94.0
Reporter: Gary Helmling
Assignee: Gary Helmling
 Fix For: 0.92.0

 Attachments: HBASE-4680.patch


 The HDFS safe mode check workaround introduced by HBASE-4510 performs a 
 {{FileSystem.setPermission()}} operation on the root directory (/) when 
 attempting to trigger a {{SafeModeException}}.  As a result, it requires 
 superuser privileges when running with DFS permission checking enabled.  
 Changing the operations to act on the HBase root directory should be safe, 
 since the master process must have write access to it.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HBASE-4680) FSUtils.isInSafeMode() checks should operate on HBase root dir, where we have permissions

2011-10-30 Thread Harsh J (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-4680?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13139915#comment-13139915
 ] 

Harsh J commented on HBASE-4680:


I apologize for missing the whole activity on this (didn't see a comment on 
HBASE-4510, oops).

I think a better fix would have been to just catch other exceptions (which is 
something I missed despite commenting).

This change is fine too, but has lead to HBASE-4705 scenarios. Could we revert 
this and instead add a:

{code}
catch (Safemode) {
 return true;
} catch (Others) {
 ignore;
}
{code}

Kinda block, while still operating on the only dir guaranteed to exist (/ - 
root)?

 FSUtils.isInSafeMode() checks should operate on HBase root dir, where we have 
 permissions
 -

 Key: HBASE-4680
 URL: https://issues.apache.org/jira/browse/HBASE-4680
 Project: HBase
  Issue Type: Bug
  Components: util
Affects Versions: 0.92.0, 0.94.0
Reporter: Gary Helmling
Assignee: Gary Helmling
Priority: Critical
 Fix For: 0.92.0

 Attachments: HBASE-4680.patch


 The HDFS safe mode check workaround introduced by HBASE-4510 performs a 
 {{FileSystem.setPermission()}} operation on the root directory (/) when 
 attempting to trigger a {{SafeModeException}}.  As a result, it requires 
 superuser privileges when running with DFS permission checking enabled.  
 Changing the operations to act on the HBase root directory should be safe, 
 since the master process must have write access to it.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HBASE-4680) FSUtils.isInSafeMode() checks should operate on HBase root dir, where we have permissions

2011-10-28 Thread Hudson (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-4680?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13138743#comment-13138743
 ] 

Hudson commented on HBASE-4680:
---

Integrated in HBase-TRUNK #2379 (See 
[https://builds.apache.org/job/HBase-TRUNK/2379/])
HBASE-4680  FSUtils.isInSafeMode() checks should operate on HBase root dir, 
where we have permissions

garyh : 
Files : 
* /hbase/trunk/CHANGES.txt
* /hbase/trunk/src/main/java/org/apache/hadoop/hbase/util/FSUtils.java


 FSUtils.isInSafeMode() checks should operate on HBase root dir, where we have 
 permissions
 -

 Key: HBASE-4680
 URL: https://issues.apache.org/jira/browse/HBASE-4680
 Project: HBase
  Issue Type: Bug
  Components: util
Affects Versions: 0.92.0, 0.94.0
Reporter: Gary Helmling
Assignee: Gary Helmling
Priority: Critical
 Fix For: 0.92.0

 Attachments: HBASE-4680.patch


 The HDFS safe mode check workaround introduced by HBASE-4510 performs a 
 {{FileSystem.setPermission()}} operation on the root directory (/) when 
 attempting to trigger a {{SafeModeException}}.  As a result, it requires 
 superuser privileges when running with DFS permission checking enabled.  
 Changing the operations to act on the HBase root directory should be safe, 
 since the master process must have write access to it.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HBASE-4680) FSUtils.isInSafeMode() checks should operate on HBase root dir, where we have permissions

2011-10-27 Thread Hadoop QA (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-4680?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13138049#comment-13138049
 ] 

Hadoop QA commented on HBASE-4680:
--

-1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12500825/HBASE-4680.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/83//console

This message is automatically generated.

 FSUtils.isInSafeMode() checks should operate on HBase root dir, where we have 
 permissions
 -

 Key: HBASE-4680
 URL: https://issues.apache.org/jira/browse/HBASE-4680
 Project: HBase
  Issue Type: Bug
  Components: util
Affects Versions: 0.92.0
Reporter: Gary Helmling
Priority: Critical
 Fix For: 0.92.0

 Attachments: HBASE-4680.patch


 The HDFS safe mode check workaround introduced by HBASE-4510 performs a 
 {{FileSystem.setPermission()}} operation on the root directory (/) when 
 attempting to trigger a {{SafeModeException}}.  As a result, it requires 
 superuser privileges when running with DFS permission checking enabled.  
 Changing the operations to act on the HBase root directory should be safe, 
 since the master process must have write access to it.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HBASE-4680) FSUtils.isInSafeMode() checks should operate on HBase root dir, where we have permissions

2011-10-27 Thread stack (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-4680?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13138057#comment-13138057
 ] 

stack commented on HBASE-4680:
--

Failed because patch is git.

 FSUtils.isInSafeMode() checks should operate on HBase root dir, where we have 
 permissions
 -

 Key: HBASE-4680
 URL: https://issues.apache.org/jira/browse/HBASE-4680
 Project: HBase
  Issue Type: Bug
  Components: util
Affects Versions: 0.92.0
Reporter: Gary Helmling
Priority: Critical
 Fix For: 0.92.0

 Attachments: HBASE-4680.patch


 The HDFS safe mode check workaround introduced by HBASE-4510 performs a 
 {{FileSystem.setPermission()}} operation on the root directory (/) when 
 attempting to trigger a {{SafeModeException}}.  As a result, it requires 
 superuser privileges when running with DFS permission checking enabled.  
 Changing the operations to act on the HBase root directory should be safe, 
 since the master process must have write access to it.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira