[jira] [Commented] (HDFS-1321) If service port and main port are the same, there is no clear log message explaining the issue.

2011-06-26 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/HDFS-1321?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13055066#comment-13055066
 ] 

Hudson commented on HDFS-1321:
--

Integrated in Hadoop-Hdfs-trunk #707 (See 
[https://builds.apache.org/job/Hadoop-Hdfs-trunk/707/])


 If service port and main port are the same, there is no clear log message 
 explaining the issue.
 ---

 Key: HDFS-1321
 URL: https://issues.apache.org/jira/browse/HDFS-1321
 Project: Hadoop HDFS
  Issue Type: Bug
  Components: name-node
Affects Versions: 0.23.0
Reporter: gary murry
Assignee: Jim Plush
Priority: Minor
  Labels: newbie
 Fix For: 0.23.0

 Attachments: HDFS-1321-take2.txt, HDFS-1321-take3.txt, 
 HDFS-1321-take4.txt, HDFS-1321.patch


 With the introduction of a service port to the namenode, there is now a 
 chance for user error to set the two port equal.  This will cause the 
 namenode to fail to start up.  It would be nice if there was a log message 
 explaining the port clash.  Or just treat things as if the service port was 
 not specified. 

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HDFS-1381) HDFS javadocs hard-code references to dfs.namenode.name.dir and dfs.datanode.data.dir parameters

2011-06-26 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/HDFS-1381?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13055067#comment-13055067
 ] 

Hudson commented on HDFS-1381:
--

Integrated in Hadoop-Hdfs-trunk #707 (See 
[https://builds.apache.org/job/Hadoop-Hdfs-trunk/707/])
HDFS-1381. HDFS javadocs hard-code references to dfs.namenode.name.dir and 
dfs.datanode.data.dir parameters (Jim Plush via atm)

atm : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVNview=revrev=1139715
Files : 
* 
/hadoop/common/trunk/hdfs/src/test/hdfs/org/apache/hadoop/hdfs/TestDFSStorageStateRecovery.java
* /hadoop/common/trunk/hdfs/CHANGES.txt
* 
/hadoop/common/trunk/hdfs/src/test/hdfs/org/apache/hadoop/hdfs/MiniDFSCluster.java
* 
/hadoop/common/trunk/hdfs/src/test/hdfs/org/apache/hadoop/hdfs/UpgradeUtilities.java


 HDFS javadocs hard-code references to dfs.namenode.name.dir and 
 dfs.datanode.data.dir parameters
 

 Key: HDFS-1381
 URL: https://issues.apache.org/jira/browse/HDFS-1381
 Project: Hadoop HDFS
  Issue Type: Bug
  Components: test
Affects Versions: 0.20.1
Reporter: Jakob Homan
Assignee: Jim Plush
  Labels: newbie
 Fix For: 0.23.0

 Attachments: HDFS-1381-take1.txt, HDFS-1381-take2.txt


 The javadoc for MiniDFSCluster makes repeated references to setting 
 dfs.name.dir and dfs.data.dir.  These should be replaced with references to 
 DFSConfigKeys' DFS_NAMENODE_NAME_DIR_KEY and DFS_DATANODE_DATA_DIR_KEY, 
 respectively.  The old values are deprecated in DFSConfigKeys, but we should 
 switch to the new values where ever we can.
 Also, a quick search the code shows that TestDFSStorageStateRecovery.java and 
 UpgradeUtilities.java should be updated as well.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HDFS-1723) quota errors messages should use the same scale

2011-06-26 Thread Jim Plush (JIRA)

[ 
https://issues.apache.org/jira/browse/HDFS-1723?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13055129#comment-13055129
 ] 

Jim Plush commented on HDFS-1723:
-

good catch, I'll update the patch and re-submit

 quota errors messages should use the same scale
 ---

 Key: HDFS-1723
 URL: https://issues.apache.org/jira/browse/HDFS-1723
 Project: Hadoop HDFS
  Issue Type: Improvement
Affects Versions: 0.21.0
Reporter: Allen Wittenauer
Assignee: Jim Plush
Priority: Minor
  Labels: newbie
 Fix For: 0.23.0

 Attachments: HDFS-1723-take1.txt, HDFS-1723-take2.txt


 A typical error message looks like this:
 org.apache.hadoop.hdfs.protocol.DSQuotaExceededException: 
 org.apache.hadoop.hdfs.protocol.DSQuotaExceededException: The DiskSpace quota 
 of /dir is exceeded: quota=3298534883328 diskspace consumed=5246.0g
 Since the two values are in difference scales and one is replicated vs. not 
 replicated (I think), this isn't very easy for the user to understand.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HDFS-929) DFSClient#getBlockSize is unused

2011-06-26 Thread Jim Plush (JIRA)

[ 
https://issues.apache.org/jira/browse/HDFS-929?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13055130#comment-13055130
 ] 

Jim Plush commented on HDFS-929:


apologies, I read your comment Maybe you can mark it as deprecated? as you 
wanted it marked deprecated. 

 DFSClient#getBlockSize is unused
 

 Key: HDFS-929
 URL: https://issues.apache.org/jira/browse/HDFS-929
 Project: Hadoop HDFS
  Issue Type: Improvement
Affects Versions: 0.21.0
Reporter: Eli Collins
Assignee: Jim Plush
Priority: Minor
 Fix For: 0.23.0

 Attachments: HDFS-929-take1.txt


 DFSClient#getBlockSize is unused. Since it's a public class internal to HDFS 
 we just remove it? If not then we should add a unit test.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Updated] (HDFS-1723) quota errors messages should use the same scale

2011-06-26 Thread Jim Plush (JIRA)

 [ 
https://issues.apache.org/jira/browse/HDFS-1723?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jim Plush updated HDFS-1723:


Attachment: HDFS-1723-take3.txt

refactoring based on Aaron's comments regarding removing the 
NSQuotaExceededException from the patch as it's not required for the fix.

 quota errors messages should use the same scale
 ---

 Key: HDFS-1723
 URL: https://issues.apache.org/jira/browse/HDFS-1723
 Project: Hadoop HDFS
  Issue Type: Improvement
Affects Versions: 0.21.0
Reporter: Allen Wittenauer
Assignee: Jim Plush
Priority: Minor
  Labels: newbie
 Fix For: 0.23.0

 Attachments: HDFS-1723-take1.txt, HDFS-1723-take2.txt, 
 HDFS-1723-take3.txt


 A typical error message looks like this:
 org.apache.hadoop.hdfs.protocol.DSQuotaExceededException: 
 org.apache.hadoop.hdfs.protocol.DSQuotaExceededException: The DiskSpace quota 
 of /dir is exceeded: quota=3298534883328 diskspace consumed=5246.0g
 Since the two values are in difference scales and one is replicated vs. not 
 replicated (I think), this isn't very easy for the user to understand.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HDFS-979) FSImage should specify which dirs are missing when refusing to come up

2011-06-26 Thread Jim Plush (JIRA)

[ 
https://issues.apache.org/jira/browse/HDFS-979?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13055154#comment-13055154
 ] 

Jim Plush commented on HDFS-979:


There seem to be several other places where a generic message like this is 
displayed...


➺ Ack --java All specified directories are not accessible
hdfs/src/java/org/apache/hadoop/hdfs/server/datanode/BlockPoolSliceStorage.java
134:  All specified directories are not accessible or do not exist.);

hdfs/src/java/org/apache/hadoop/hdfs/server/datanode/DataStorage.java
188:  All specified directories are not accessible or do not exist.);

hdfs/src/java/org/apache/hadoop/hdfs/server/namenode/FSImage.java
201:  All specified directories are not accessible or do not exist.);


 FSImage should specify which dirs are missing when refusing to come up
 --

 Key: HDFS-979
 URL: https://issues.apache.org/jira/browse/HDFS-979
 Project: Hadoop HDFS
  Issue Type: Improvement
  Components: name-node
Affects Versions: 0.22.0
Reporter: Steve Loughran
Priority: Minor

 When {{FSImage}} can't come up as either it has no data or edit dirs, it 
 tells me this
 {code}
 java.io.IOException: All specified directories are not accessible or do not 
 exist.
 {code}
 What it doesn't do is say which of the two attributes are missing. This would 
 be beneficial to anyone trying to track down the problem. Also, I don't think 
 the message is correct. It's bailing out because dataDirs.size() == 0 || 
 editsDirs.size() == 0 , because a list is empty -not because the dirs aren't 
 there, as there hasn't been any validation yet.
 More useful would be
 # Explicit mention of which attributes are null
 # Declare that this is because they are not in the config

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HDFS-1723) quota errors messages should use the same scale

2011-06-26 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/HDFS-1723?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13055157#comment-13055157
 ] 

Hadoop QA commented on HDFS-1723:
-

+1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12483868/HDFS-1723-take3.txt
  against trunk revision 1139715.

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

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

+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 core tests.  The patch passed core unit tests.

+1 contrib tests.  The patch passed contrib unit tests.

+1 system test framework.  The patch passed system test framework compile.

Test results: 
https://builds.apache.org/job/PreCommit-HDFS-Build/842//testReport/
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HDFS-Build/842//artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/842//console

This message is automatically generated.

 quota errors messages should use the same scale
 ---

 Key: HDFS-1723
 URL: https://issues.apache.org/jira/browse/HDFS-1723
 Project: Hadoop HDFS
  Issue Type: Improvement
Affects Versions: 0.21.0
Reporter: Allen Wittenauer
Assignee: Jim Plush
Priority: Minor
  Labels: newbie
 Fix For: 0.23.0

 Attachments: HDFS-1723-take1.txt, HDFS-1723-take2.txt, 
 HDFS-1723-take3.txt


 A typical error message looks like this:
 org.apache.hadoop.hdfs.protocol.DSQuotaExceededException: 
 org.apache.hadoop.hdfs.protocol.DSQuotaExceededException: The DiskSpace quota 
 of /dir is exceeded: quota=3298534883328 diskspace consumed=5246.0g
 Since the two values are in difference scales and one is replicated vs. not 
 replicated (I think), this isn't very easy for the user to understand.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Updated] (HDFS-2107) Move block management code to a package

2011-06-26 Thread Tsz Wo (Nicholas), SZE (JIRA)

 [ 
https://issues.apache.org/jira/browse/HDFS-2107?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Tsz Wo (Nicholas), SZE updated HDFS-2107:
-

Attachment: h2107_20110626.patch

All the findbugs and javadoc warnings are existing problems.

h2107_20110626.patch: fixed the warnings.

 Move block management code to a package
 ---

 Key: HDFS-2107
 URL: https://issues.apache.org/jira/browse/HDFS-2107
 Project: Hadoop HDFS
  Issue Type: Sub-task
  Components: name-node
Affects Versions: 0.23.0
Reporter: Tsz Wo (Nicholas), SZE
Assignee: Tsz Wo (Nicholas), SZE
 Attachments: h2107_20110624.patch, h2107_20110626.patch, svn_mv.sh


 In namenode, everything (except metrics) is in 
 {{org.apache.hadoop.hdfs.server.namenode}}.  Let's create a package for block 
 management.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HDFS-2107) Move block management code to a package

2011-06-26 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/HDFS-2107?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13055220#comment-13055220
 ] 

Hadoop QA commented on HDFS-2107:
-

+1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12483875/h2107_20110626.patch
  against trunk revision 1139715.

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

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

+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 core tests.  The patch passed core unit tests.

+1 contrib tests.  The patch passed contrib unit tests.

+1 system test framework.  The patch passed system test framework compile.

Test results: 
https://builds.apache.org/job/PreCommit-HDFS-Build/843//testReport/
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HDFS-Build/843//artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/843//console

This message is automatically generated.

 Move block management code to a package
 ---

 Key: HDFS-2107
 URL: https://issues.apache.org/jira/browse/HDFS-2107
 Project: Hadoop HDFS
  Issue Type: Sub-task
  Components: name-node
Affects Versions: 0.23.0
Reporter: Tsz Wo (Nicholas), SZE
Assignee: Tsz Wo (Nicholas), SZE
 Attachments: h2107_20110624.patch, h2107_20110626.patch, svn_mv.sh


 In namenode, everything (except metrics) is in 
 {{org.apache.hadoop.hdfs.server.namenode}}.  Let's create a package for block 
 management.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Updated] (HDFS-979) FSImage should specify which dirs are missing when refusing to come up

2011-06-26 Thread Jim Plush (JIRA)

 [ 
https://issues.apache.org/jira/browse/HDFS-979?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jim Plush updated HDFS-979:
---

Attachment: HDFS-979-take1.txt

Adding a verbose (the more verbose the more helpful?) Exception message should 
either the dataDirs or the editDirs show a size of 0 in the 
recoverTransitionRead method. It will also loop through the directories and if 
they are a file scheme, check to make sure those directories actually do exist 
on disk. 

For example if the Edit dirs was showing an empty list and the directory did 
not exist locally the message would be: 

All specified directories are not accessible or do not exist. Specifically:  
FSImage reports 0 NameNode edit dirs. The config value of 
'dfs.namenode.edits.dir' is: file:///opt/hadoop-doesnotexist/dfs/name.  
Directory: /opt/hadoop-doesnotexist/dfs/name DOES NOT EXIST.

 FSImage should specify which dirs are missing when refusing to come up
 --

 Key: HDFS-979
 URL: https://issues.apache.org/jira/browse/HDFS-979
 Project: Hadoop HDFS
  Issue Type: Improvement
  Components: name-node
Affects Versions: 0.22.0
Reporter: Steve Loughran
Priority: Minor
 Attachments: HDFS-979-take1.txt


 When {{FSImage}} can't come up as either it has no data or edit dirs, it 
 tells me this
 {code}
 java.io.IOException: All specified directories are not accessible or do not 
 exist.
 {code}
 What it doesn't do is say which of the two attributes are missing. This would 
 be beneficial to anyone trying to track down the problem. Also, I don't think 
 the message is correct. It's bailing out because dataDirs.size() == 0 || 
 editsDirs.size() == 0 , because a list is empty -not because the dirs aren't 
 there, as there hasn't been any validation yet.
 More useful would be
 # Explicit mention of which attributes are null
 # Declare that this is because they are not in the config

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Updated] (HDFS-979) FSImage should specify which dirs are missing when refusing to come up

2011-06-26 Thread Jim Plush (JIRA)

 [ 
https://issues.apache.org/jira/browse/HDFS-979?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jim Plush updated HDFS-979:
---

Fix Version/s: 0.23.0
 Assignee: Jim Plush
 Release Note: Added more verbose error logging in FSImage should there be 
an issue getting the size of the NameNode data or edits directories.
   Status: Patch Available  (was: Open)

 FSImage should specify which dirs are missing when refusing to come up
 --

 Key: HDFS-979
 URL: https://issues.apache.org/jira/browse/HDFS-979
 Project: Hadoop HDFS
  Issue Type: Improvement
  Components: name-node
Affects Versions: 0.22.0
Reporter: Steve Loughran
Assignee: Jim Plush
Priority: Minor
 Fix For: 0.23.0

 Attachments: HDFS-979-take1.txt


 When {{FSImage}} can't come up as either it has no data or edit dirs, it 
 tells me this
 {code}
 java.io.IOException: All specified directories are not accessible or do not 
 exist.
 {code}
 What it doesn't do is say which of the two attributes are missing. This would 
 be beneficial to anyone trying to track down the problem. Also, I don't think 
 the message is correct. It's bailing out because dataDirs.size() == 0 || 
 editsDirs.size() == 0 , because a list is empty -not because the dirs aren't 
 there, as there hasn't been any validation yet.
 More useful would be
 # Explicit mention of which attributes are null
 # Declare that this is because they are not in the config

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HDFS-979) FSImage should specify which dirs are missing when refusing to come up

2011-06-26 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/HDFS-979?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13055240#comment-13055240
 ] 

Hadoop QA commented on HDFS-979:


-1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12483878/HDFS-979-take1.txt
  against trunk revision 1139715.

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

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

+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 appears to introduce 1 new Findbugs (version 1.3.9) 
warnings.

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

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

+1 contrib tests.  The patch passed contrib unit tests.

+1 system test framework.  The patch passed system test framework compile.

Test results: 
https://builds.apache.org/job/PreCommit-HDFS-Build/844//testReport/
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HDFS-Build/844//artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/844//console

This message is automatically generated.

 FSImage should specify which dirs are missing when refusing to come up
 --

 Key: HDFS-979
 URL: https://issues.apache.org/jira/browse/HDFS-979
 Project: Hadoop HDFS
  Issue Type: Improvement
  Components: name-node
Affects Versions: 0.22.0
Reporter: Steve Loughran
Assignee: Jim Plush
Priority: Minor
 Fix For: 0.23.0

 Attachments: HDFS-979-take1.txt


 When {{FSImage}} can't come up as either it has no data or edit dirs, it 
 tells me this
 {code}
 java.io.IOException: All specified directories are not accessible or do not 
 exist.
 {code}
 What it doesn't do is say which of the two attributes are missing. This would 
 be beneficial to anyone trying to track down the problem. Also, I don't think 
 the message is correct. It's bailing out because dataDirs.size() == 0 || 
 editsDirs.size() == 0 , because a list is empty -not because the dirs aren't 
 there, as there hasn't been any validation yet.
 More useful would be
 # Explicit mention of which attributes are null
 # Declare that this is because they are not in the config

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Updated] (HDFS-979) FSImage should specify which dirs are missing when refusing to come up

2011-06-26 Thread Jim Plush (JIRA)

 [ 
https://issues.apache.org/jira/browse/HDFS-979?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jim Plush updated HDFS-979:
---

Attachment: HDFS-979-take2.txt

Updated to use a StringBuilder in the loop after complaints from Findbugs 
report.

 FSImage should specify which dirs are missing when refusing to come up
 --

 Key: HDFS-979
 URL: https://issues.apache.org/jira/browse/HDFS-979
 Project: Hadoop HDFS
  Issue Type: Improvement
  Components: name-node
Affects Versions: 0.22.0
Reporter: Steve Loughran
Assignee: Jim Plush
Priority: Minor
 Fix For: 0.23.0

 Attachments: HDFS-979-take1.txt, HDFS-979-take2.txt


 When {{FSImage}} can't come up as either it has no data or edit dirs, it 
 tells me this
 {code}
 java.io.IOException: All specified directories are not accessible or do not 
 exist.
 {code}
 What it doesn't do is say which of the two attributes are missing. This would 
 be beneficial to anyone trying to track down the problem. Also, I don't think 
 the message is correct. It's bailing out because dataDirs.size() == 0 || 
 editsDirs.size() == 0 , because a list is empty -not because the dirs aren't 
 there, as there hasn't been any validation yet.
 More useful would be
 # Explicit mention of which attributes are null
 # Declare that this is because they are not in the config

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HDFS-979) FSImage should specify which dirs are missing when refusing to come up

2011-06-26 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/HDFS-979?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13055276#comment-13055276
 ] 

Hadoop QA commented on HDFS-979:


+1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12483882/HDFS-979-take2.txt
  against trunk revision 1139715.

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

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

+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 core tests.  The patch passed core unit tests.

+1 contrib tests.  The patch passed contrib unit tests.

+1 system test framework.  The patch passed system test framework compile.

Test results: 
https://builds.apache.org/job/PreCommit-HDFS-Build/845//testReport/
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HDFS-Build/845//artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/845//console

This message is automatically generated.

 FSImage should specify which dirs are missing when refusing to come up
 --

 Key: HDFS-979
 URL: https://issues.apache.org/jira/browse/HDFS-979
 Project: Hadoop HDFS
  Issue Type: Improvement
  Components: name-node
Affects Versions: 0.22.0
Reporter: Steve Loughran
Assignee: Jim Plush
Priority: Minor
 Fix For: 0.23.0

 Attachments: HDFS-979-take1.txt, HDFS-979-take2.txt


 When {{FSImage}} can't come up as either it has no data or edit dirs, it 
 tells me this
 {code}
 java.io.IOException: All specified directories are not accessible or do not 
 exist.
 {code}
 What it doesn't do is say which of the two attributes are missing. This would 
 be beneficial to anyone trying to track down the problem. Also, I don't think 
 the message is correct. It's bailing out because dataDirs.size() == 0 || 
 editsDirs.size() == 0 , because a list is empty -not because the dirs aren't 
 there, as there hasn't been any validation yet.
 More useful would be
 # Explicit mention of which attributes are null
 # Declare that this is because they are not in the config

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Created] (HDFS-2109) Store uMask as member variable to DFSClient.Conf

2011-06-26 Thread Bharath Mundlapudi (JIRA)
Store uMask as member variable to DFSClient.Conf


 Key: HDFS-2109
 URL: https://issues.apache.org/jira/browse/HDFS-2109
 Project: Hadoop HDFS
  Issue Type: Bug
  Components: hdfs client
Affects Versions: 0.23.0
Reporter: Bharath Mundlapudi
Assignee: Bharath Mundlapudi
 Fix For: 0.23.0


As a part of removing reference to conf in DFSClient, I am proposing replacing 
FsPermission.getUMask(conf) everywhere in DFSClient class with
dfsClientConf.uMask. 

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Updated] (HDFS-2109) Store uMask as member variable to DFSClient.Conf

2011-06-26 Thread Bharath Mundlapudi (JIRA)

 [ 
https://issues.apache.org/jira/browse/HDFS-2109?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Bharath Mundlapudi updated HDFS-2109:
-

Description: 
As a part of removing reference to conf in DFSClient, I am proposing replacing 
FsPermission.getUMask(conf) everywhere in DFSClient class with
dfsClientConf.uMask by storing uMask as a member variable to DFSClient.Conf. 

  was:
As a part of removing reference to conf in DFSClient, I am proposing replacing 
FsPermission.getUMask(conf) everywhere in DFSClient class with
dfsClientConf.uMask. 


 Store uMask as member variable to DFSClient.Conf
 

 Key: HDFS-2109
 URL: https://issues.apache.org/jira/browse/HDFS-2109
 Project: Hadoop HDFS
  Issue Type: Bug
  Components: hdfs client
Affects Versions: 0.23.0
Reporter: Bharath Mundlapudi
Assignee: Bharath Mundlapudi
 Fix For: 0.23.0


 As a part of removing reference to conf in DFSClient, I am proposing 
 replacing FsPermission.getUMask(conf) everywhere in DFSClient class with
 dfsClientConf.uMask by storing uMask as a member variable to DFSClient.Conf. 

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Updated] (HDFS-2109) Store uMask as member variable to DFSClient.Conf

2011-06-26 Thread Bharath Mundlapudi (JIRA)

 [ 
https://issues.apache.org/jira/browse/HDFS-2109?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Bharath Mundlapudi updated HDFS-2109:
-

Attachment: HDFS-2109-1.patch

Attaching the patch.

 Store uMask as member variable to DFSClient.Conf
 

 Key: HDFS-2109
 URL: https://issues.apache.org/jira/browse/HDFS-2109
 Project: Hadoop HDFS
  Issue Type: Bug
  Components: hdfs client
Affects Versions: 0.23.0
Reporter: Bharath Mundlapudi
Assignee: Bharath Mundlapudi
 Fix For: 0.23.0

 Attachments: HDFS-2109-1.patch


 As a part of removing reference to conf in DFSClient, I am proposing 
 replacing FsPermission.getUMask(conf) everywhere in DFSClient class with
 dfsClientConf.uMask by storing uMask as a member variable to DFSClient.Conf. 

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HDFS-1723) quota errors messages should use the same scale

2011-06-26 Thread Aaron T. Myers (JIRA)

[ 
https://issues.apache.org/jira/browse/HDFS-1723?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13055292#comment-13055292
 ] 

Aaron T. Myers commented on HDFS-1723:
--

Thanks for addressing my comments, Jim. I should have noticed this earlier, but 
I have one final comment and then I'll give it a +1 / commit it:

Earlier in the test we're explicitly setting the quota value to 1k, so we might 
as well explicitly test that the output is 1.0k, rather than have a regex which 
will match it. We wouldn't want this test to pass if it output, for example, 
quota=9.5k

 quota errors messages should use the same scale
 ---

 Key: HDFS-1723
 URL: https://issues.apache.org/jira/browse/HDFS-1723
 Project: Hadoop HDFS
  Issue Type: Improvement
Affects Versions: 0.21.0
Reporter: Allen Wittenauer
Assignee: Jim Plush
Priority: Minor
  Labels: newbie
 Fix For: 0.23.0

 Attachments: HDFS-1723-take1.txt, HDFS-1723-take2.txt, 
 HDFS-1723-take3.txt


 A typical error message looks like this:
 org.apache.hadoop.hdfs.protocol.DSQuotaExceededException: 
 org.apache.hadoop.hdfs.protocol.DSQuotaExceededException: The DiskSpace quota 
 of /dir is exceeded: quota=3298534883328 diskspace consumed=5246.0g
 Since the two values are in difference scales and one is replicated vs. not 
 replicated (I think), this isn't very easy for the user to understand.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HDFS-1026) Quota checks fail for small files and quotas

2011-06-26 Thread Eli Collins (JIRA)

[ 
https://issues.apache.org/jira/browse/HDFS-1026?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13055293#comment-13055293
 ] 

Eli Collins commented on HDFS-1026:
---

Yes, add a comment in FSDirectory#addBlock and in the code that generates a 
quota violation exception LOG.warn that the quota check failed because the 
quota is smaller than block size * # replicas.

 Quota checks fail for small files and quotas
 

 Key: HDFS-1026
 URL: https://issues.apache.org/jira/browse/HDFS-1026
 Project: Hadoop HDFS
  Issue Type: Bug
  Components: documentation, name-node
Affects Versions: 0.20.1, 0.20.2, 0.20.3, 0.21.0, 0.22.0
Reporter: Eli Collins
Priority: Blocker

 If a directory has a quota less than blockSize * numReplicas then you can't 
 add a file to it, even if the file size is less than the quota. This is 
 because FSDirectory#addBlock updates the count assuming at least one block is 
 written in full. We don't know how much of the block will be written when 
 addBlock is called and supporting such small quotas is not important so 
 perhaps we should document this and log an error message instead of making 
 small (blockSize * numReplicas) quotas work.
 {code}
 // check quota limits and updated space consumed
 updateCount(inodes, inodes.length-1, 0, 
 fileINode.getPreferredBlockSize()*fileINode.getReplication(), true);
 {code}
 You can reproduce with the following commands:
 {code}
 $ dd if=/dev/zero of=temp bs=1000 count=64
 $ hadoop fs -mkdir /user/eli/dir
 $ hdfs dfsadmin -setSpaceQuota 191M /user/eli/dir
 $ hadoop fs -put temp /user/eli/dir  # Causes DSQuotaExceededException
 {code}

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HDFS-1723) quota errors messages should use the same scale

2011-06-26 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/HDFS-1723?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13055323#comment-13055323
 ] 

Hadoop QA commented on HDFS-1723:
-

+1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12483898/HDFS-1723-take4.txt
  against trunk revision 1139715.

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

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

+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 core tests.  The patch passed core unit tests.

+1 contrib tests.  The patch passed contrib unit tests.

+1 system test framework.  The patch passed system test framework compile.

Test results: 
https://builds.apache.org/job/PreCommit-HDFS-Build/846//testReport/
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HDFS-Build/846//artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/846//console

This message is automatically generated.

 quota errors messages should use the same scale
 ---

 Key: HDFS-1723
 URL: https://issues.apache.org/jira/browse/HDFS-1723
 Project: Hadoop HDFS
  Issue Type: Improvement
Affects Versions: 0.21.0
Reporter: Allen Wittenauer
Assignee: Jim Plush
Priority: Minor
  Labels: newbie
 Fix For: 0.23.0

 Attachments: HDFS-1723-take1.txt, HDFS-1723-take2.txt, 
 HDFS-1723-take3.txt, HDFS-1723-take4.txt


 A typical error message looks like this:
 org.apache.hadoop.hdfs.protocol.DSQuotaExceededException: 
 org.apache.hadoop.hdfs.protocol.DSQuotaExceededException: The DiskSpace quota 
 of /dir is exceeded: quota=3298534883328 diskspace consumed=5246.0g
 Since the two values are in difference scales and one is replicated vs. not 
 replicated (I think), this isn't very easy for the user to understand.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira