[jira] [Commented] (HDFS-7381) Decouple the management of block id and gen stamps from FSNamesystem

2014-11-08 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HDFS-7381:
-

{color:red}-1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12680304/HDFS-7381.000.patch
  against trunk revision 9a4e0d3.

{color:green}+1 @author{color}.  The patch does not contain any @author 
tags.

{color:green}+1 tests included{color}.  The patch appears to include 2 new 
or modified test files.

{color:green}+1 javac{color}.  The applied patch does not increase the 
total number of javac compiler warnings.

{color:red}-1 javadoc{color}.  The javadoc tool appears to have generated 6 
warning messages.
See 
https://builds.apache.org/job/PreCommit-HDFS-Build/8698//artifact/patchprocess/diffJavadocWarnings.txt
 for details.

{color:green}+1 eclipse:eclipse{color}.  The patch built with 
eclipse:eclipse.

{color:green}+1 findbugs{color}.  The patch does not introduce any new 
Findbugs (version 2.0.3) warnings.

{color:green}+1 release audit{color}.  The applied patch does not increase 
the total number of release audit warnings.

{color:red}-1 core tests{color}.  The patch failed these unit tests in 
hadoop-hdfs-project/hadoop-hdfs:

  org.apache.hadoop.hdfs.TestDFSUpgradeFromImage
  org.apache.hadoop.hdfs.server.balancer.TestBalancer
  
org.apache.hadoop.hdfs.qjournal.client.TestQuorumJournalManager

{color:green}+1 contrib tests{color}.  The patch passed contrib unit tests.

Test results: 
https://builds.apache.org/job/PreCommit-HDFS-Build/8698//testReport/
Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/8698//console

This message is automatically generated.

 Decouple the management of block id and gen stamps from FSNamesystem
 

 Key: HDFS-7381
 URL: https://issues.apache.org/jira/browse/HDFS-7381
 Project: Hadoop HDFS
  Issue Type: Improvement
Reporter: Haohui Mai
Assignee: Haohui Mai
 Attachments: HDFS-7381.000.patch


 The block layer should be responsible of managing block ids and generation 
 stamps. Currently the functionality is misplace into {{FSNamesystem}}.
 This jira proposes to decouple them from the {{FSNamesystem}} class.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HDFS-7379) TestBalancer#testBalancerWithRamDisk creates test files incorrectly

2014-11-08 Thread Hudson (JIRA)

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

Hudson commented on HDFS-7379:
--

FAILURE: Integrated in Hadoop-Yarn-trunk #737 (See 
[https://builds.apache.org/job/Hadoop-Yarn-trunk/737/])
HDFS-7379. TestBalancer#testBalancerWithRamDisk creates test files incorrectly. 
Contributed by Xiaoyu Yao. (wheat9: rev 
57760c0663288a7611c3609891ef92f1abf4bb53)
* hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
* 
hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/balancer/TestBalancer.java


 TestBalancer#testBalancerWithRamDisk creates test files incorrectly
 ---

 Key: HDFS-7379
 URL: https://issues.apache.org/jira/browse/HDFS-7379
 Project: Hadoop HDFS
  Issue Type: Bug
  Components: test
Affects Versions: 2.6.0
Reporter: Xiaoyu Yao
Assignee: Xiaoyu Yao
Priority: Minor
 Fix For: 2.6.0

 Attachments: HDFS-7379.01.patch


 There is a copy-paste error during for test file creation. The test is 
 supposed to create two test files named path1 and path2 on RAM_DISK. But The 
 error caused path1 to be created twice with the second creation overwrite 
 (delete) the first one on RAM_DISK. This caused verification failure for 
 path2 as it never gets created. The fix is to create test files with the 
 correct names. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HDFS-7379) TestBalancer#testBalancerWithRamDisk creates test files incorrectly

2014-11-08 Thread Hudson (JIRA)

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

Hudson commented on HDFS-7379:
--

FAILURE: Integrated in Hadoop-Hdfs-trunk #1927 (See 
[https://builds.apache.org/job/Hadoop-Hdfs-trunk/1927/])
HDFS-7379. TestBalancer#testBalancerWithRamDisk creates test files incorrectly. 
Contributed by Xiaoyu Yao. (wheat9: rev 
57760c0663288a7611c3609891ef92f1abf4bb53)
* 
hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/balancer/TestBalancer.java
* hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt


 TestBalancer#testBalancerWithRamDisk creates test files incorrectly
 ---

 Key: HDFS-7379
 URL: https://issues.apache.org/jira/browse/HDFS-7379
 Project: Hadoop HDFS
  Issue Type: Bug
  Components: test
Affects Versions: 2.6.0
Reporter: Xiaoyu Yao
Assignee: Xiaoyu Yao
Priority: Minor
 Fix For: 2.6.0

 Attachments: HDFS-7379.01.patch


 There is a copy-paste error during for test file creation. The test is 
 supposed to create two test files named path1 and path2 on RAM_DISK. But The 
 error caused path1 to be created twice with the second creation overwrite 
 (delete) the first one on RAM_DISK. This caused verification failure for 
 path2 as it never gets created. The fix is to create test files with the 
 correct names. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HDFS-7379) TestBalancer#testBalancerWithRamDisk creates test files incorrectly

2014-11-08 Thread Hudson (JIRA)

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

Hudson commented on HDFS-7379:
--

FAILURE: Integrated in Hadoop-Mapreduce-trunk #1951 (See 
[https://builds.apache.org/job/Hadoop-Mapreduce-trunk/1951/])
HDFS-7379. TestBalancer#testBalancerWithRamDisk creates test files incorrectly. 
Contributed by Xiaoyu Yao. (wheat9: rev 
57760c0663288a7611c3609891ef92f1abf4bb53)
* 
hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/balancer/TestBalancer.java
* hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt


 TestBalancer#testBalancerWithRamDisk creates test files incorrectly
 ---

 Key: HDFS-7379
 URL: https://issues.apache.org/jira/browse/HDFS-7379
 Project: Hadoop HDFS
  Issue Type: Bug
  Components: test
Affects Versions: 2.6.0
Reporter: Xiaoyu Yao
Assignee: Xiaoyu Yao
Priority: Minor
 Fix For: 2.6.0

 Attachments: HDFS-7379.01.patch


 There is a copy-paste error during for test file creation. The test is 
 supposed to create two test files named path1 and path2 on RAM_DISK. But The 
 error caused path1 to be created twice with the second creation overwrite 
 (delete) the first one on RAM_DISK. This caused verification failure for 
 path2 as it never gets created. The fix is to create test files with the 
 correct names. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HDFS-7358) Clients may get stuck waiting when using ByteArrayManager

2014-11-08 Thread Tsz Wo Nicholas Sze (JIRA)

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

Tsz Wo Nicholas Sze updated HDFS-7358:
--
Attachment: h7358_20141108.patch

h7358_20141108.patch: fixes TestHFlush failure.

 Clients may get stuck waiting when using ByteArrayManager
 -

 Key: HDFS-7358
 URL: https://issues.apache.org/jira/browse/HDFS-7358
 Project: Hadoop HDFS
  Issue Type: Bug
  Components: hdfs-client
Reporter: Tsz Wo Nicholas Sze
Assignee: Tsz Wo Nicholas Sze
 Attachments: h7358_20141104.patch, h7358_20141104_wait_timeout.patch, 
 h7358_20141105.patch, h7358_20141106.patch, h7358_20141107.patch, 
 h7358_20141108.patch


 [~stack] reported that clients might get stuck waiting when using 
 ByteArrayManager; see [his 
 comments|https://issues.apache.org/jira/browse/HDFS-7276?focusedCommentId=14197036page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-14197036].



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Created] (HDFS-7382) DataNode in secure mode may throw NullPointerException if client connects before DataNode registers itself with NameNode.

2014-11-08 Thread Chris Nauroth (JIRA)
Chris Nauroth created HDFS-7382:
---

 Summary: DataNode in secure mode may throw NullPointerException if 
client connects before DataNode registers itself with NameNode.
 Key: HDFS-7382
 URL: https://issues.apache.org/jira/browse/HDFS-7382
 Project: Hadoop HDFS
  Issue Type: Bug
  Components: datanode
Reporter: Chris Nauroth
Assignee: Chris Nauroth
Priority: Minor


{{SaslDataTransferServer#receive}} needs to check if the DataNode is listening 
on a privileged port.  It does this by checking the address from the 
{{DatanodeID}}.  However, there is a window of time when this will be {{null}}. 
 If a client is still holding a {{LocatedBlock}} that references that DataNode 
and chooses to connect, then there is a risk of getting a 
{{NullPointerException}}.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HDFS-7382) DataNode in secure mode may throw NullPointerException if client connects before DataNode registers itself with NameNode.

2014-11-08 Thread Chris Nauroth (JIRA)

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

Chris Nauroth updated HDFS-7382:

Component/s: security

 DataNode in secure mode may throw NullPointerException if client connects 
 before DataNode registers itself with NameNode.
 -

 Key: HDFS-7382
 URL: https://issues.apache.org/jira/browse/HDFS-7382
 Project: Hadoop HDFS
  Issue Type: Bug
  Components: datanode, security
Reporter: Chris Nauroth
Assignee: Chris Nauroth
Priority: Minor

 {{SaslDataTransferServer#receive}} needs to check if the DataNode is 
 listening on a privileged port.  It does this by checking the address from 
 the {{DatanodeID}}.  However, there is a window of time when this will be 
 {{null}}.  If a client is still holding a {{LocatedBlock}} that references 
 that DataNode and chooses to connect, then there is a risk of getting a 
 {{NullPointerException}}.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HDFS-7382) DataNode in secure mode may throw NullPointerException if client connects before DataNode registers itself with NameNode.

2014-11-08 Thread Chris Nauroth (JIRA)

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

Chris Nauroth updated HDFS-7382:

Attachment: HDFS-7382.1.patch

We don't really need to rely on NameNode registration.  A DataNode always knows 
which port it's listening on for DataTransferProtocol, based solely on its own 
local configuration.  The attached patch implements the fix.  I'm still passing 
through the {{DatanodeID}}, because that's still useful for debug logging.

 DataNode in secure mode may throw NullPointerException if client connects 
 before DataNode registers itself with NameNode.
 -

 Key: HDFS-7382
 URL: https://issues.apache.org/jira/browse/HDFS-7382
 Project: Hadoop HDFS
  Issue Type: Bug
  Components: datanode, security
Reporter: Chris Nauroth
Assignee: Chris Nauroth
Priority: Minor
 Attachments: HDFS-7382.1.patch


 {{SaslDataTransferServer#receive}} needs to check if the DataNode is 
 listening on a privileged port.  It does this by checking the address from 
 the {{DatanodeID}}.  However, there is a window of time when this will be 
 {{null}}.  If a client is still holding a {{LocatedBlock}} that references 
 that DataNode and chooses to connect, then there is a risk of getting a 
 {{NullPointerException}}.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HDFS-7382) DataNode in secure mode may throw NullPointerException if client connects before DataNode registers itself with NameNode.

2014-11-08 Thread Chris Nauroth (JIRA)

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

Chris Nauroth updated HDFS-7382:

Status: Patch Available  (was: Open)

 DataNode in secure mode may throw NullPointerException if client connects 
 before DataNode registers itself with NameNode.
 -

 Key: HDFS-7382
 URL: https://issues.apache.org/jira/browse/HDFS-7382
 Project: Hadoop HDFS
  Issue Type: Bug
  Components: datanode, security
Reporter: Chris Nauroth
Assignee: Chris Nauroth
Priority: Minor
 Attachments: HDFS-7382.1.patch


 {{SaslDataTransferServer#receive}} needs to check if the DataNode is 
 listening on a privileged port.  It does this by checking the address from 
 the {{DatanodeID}}.  However, there is a window of time when this will be 
 {{null}}.  If a client is still holding a {{LocatedBlock}} that references 
 that DataNode and chooses to connect, then there is a risk of getting a 
 {{NullPointerException}}.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HDFS-7382) DataNode in secure mode may throw NullPointerException if client connects before DataNode registers itself with NameNode.

2014-11-08 Thread Tsz Wo Nicholas Sze (JIRA)

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

Tsz Wo Nicholas Sze updated HDFS-7382:
--
Hadoop Flags: Reviewed

+1 patch looks good.

 DataNode in secure mode may throw NullPointerException if client connects 
 before DataNode registers itself with NameNode.
 -

 Key: HDFS-7382
 URL: https://issues.apache.org/jira/browse/HDFS-7382
 Project: Hadoop HDFS
  Issue Type: Bug
  Components: datanode, security
Reporter: Chris Nauroth
Assignee: Chris Nauroth
Priority: Minor
 Attachments: HDFS-7382.1.patch


 {{SaslDataTransferServer#receive}} needs to check if the DataNode is 
 listening on a privileged port.  It does this by checking the address from 
 the {{DatanodeID}}.  However, there is a window of time when this will be 
 {{null}}.  If a client is still holding a {{LocatedBlock}} that references 
 that DataNode and chooses to connect, then there is a risk of getting a 
 {{NullPointerException}}.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Created] (HDFS-7383) DataNode.requestShortCircuitFdsForRead may throw NullPointerException

2014-11-08 Thread Tsz Wo Nicholas Sze (JIRA)
Tsz Wo Nicholas Sze created HDFS-7383:
-

 Summary: DataNode.requestShortCircuitFdsForRead may throw 
NullPointerException
 Key: HDFS-7383
 URL: https://issues.apache.org/jira/browse/HDFS-7383
 Project: Hadoop HDFS
  Issue Type: Bug
  Components: datanode
Reporter: Tsz Wo Nicholas Sze
Assignee: Tsz Wo Nicholas Sze


{noformat}
java.lang.NullPointerException
at 
org.apache.hadoop.hdfs.server.datanode.DataNode.requestShortCircuitFdsForRead(DataNode.java:1525)
at 
org.apache.hadoop.hdfs.server.datanode.DataXceiver.requestShortCircuitFds(DataXceiver.java:286)
at 
org.apache.hadoop.hdfs.protocol.datatransfer.Receiver.opRequestShortCircuitFds(Receiver.java:185)
at 
org.apache.hadoop.hdfs.protocol.datatransfer.Receiver.processOp(Receiver.java:89)
at 
org.apache.hadoop.hdfs.server.datanode.DataXceiver.run(DataXceiver.java:234)
at java.lang.Thread.run(Thread.java:745)
{noformat}




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HDFS-7358) Clients may get stuck waiting when using ByteArrayManager

2014-11-08 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HDFS-7358:
-

{color:red}-1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12680428/h7358_20141108.patch
  against trunk revision 737d928.

{color:green}+1 @author{color}.  The patch does not contain any @author 
tags.

{color:green}+1 tests included{color}.  The patch appears to include 2 new 
or modified test files.

{color:green}+1 javac{color}.  The applied patch does not increase the 
total number of javac compiler warnings.

{color:green}+1 javadoc{color}.  There were no new javadoc warning messages.

{color:green}+1 eclipse:eclipse{color}.  The patch built with 
eclipse:eclipse.

{color:red}-1 findbugs{color}.  The patch appears to introduce 1 new 
Findbugs (version 2.0.3) warnings.

{color:green}+1 release audit{color}.  The applied patch does not increase 
the total number of release audit warnings.

{color:green}+1 core tests{color}.  The patch passed unit tests in 
hadoop-hdfs-project/hadoop-hdfs.

{color:green}+1 contrib tests{color}.  The patch passed contrib unit tests.

Test results: 
https://builds.apache.org/job/PreCommit-HDFS-Build/8699//testReport/
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HDFS-Build/8699//artifact/patchprocess/newPatchFindbugsWarningshadoop-hdfs.html
Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/8699//console

This message is automatically generated.

 Clients may get stuck waiting when using ByteArrayManager
 -

 Key: HDFS-7358
 URL: https://issues.apache.org/jira/browse/HDFS-7358
 Project: Hadoop HDFS
  Issue Type: Bug
  Components: hdfs-client
Reporter: Tsz Wo Nicholas Sze
Assignee: Tsz Wo Nicholas Sze
 Attachments: h7358_20141104.patch, h7358_20141104_wait_timeout.patch, 
 h7358_20141105.patch, h7358_20141106.patch, h7358_20141107.patch, 
 h7358_20141108.patch


 [~stack] reported that clients might get stuck waiting when using 
 ByteArrayManager; see [his 
 comments|https://issues.apache.org/jira/browse/HDFS-7276?focusedCommentId=14197036page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-14197036].



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HDFS-7382) DataNode in secure mode may throw NullPointerException if client connects before DataNode registers itself with NameNode.

2014-11-08 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HDFS-7382:
-

{color:red}-1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12680433/HDFS-7382.1.patch
  against trunk revision 737d928.

{color:green}+1 @author{color}.  The patch does not contain any @author 
tags.

{color:red}-1 tests included{color}.  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.

{color:green}+1 javac{color}.  The applied patch does not increase the 
total number of javac compiler warnings.

{color:green}+1 javadoc{color}.  There were no new javadoc warning messages.

{color:green}+1 eclipse:eclipse{color}.  The patch built with 
eclipse:eclipse.

{color:green}+1 findbugs{color}.  The patch does not introduce any new 
Findbugs (version 2.0.3) warnings.

{color:green}+1 release audit{color}.  The applied patch does not increase 
the total number of release audit warnings.

{color:red}-1 core tests{color}.  The patch failed these unit tests in 
hadoop-hdfs-project/hadoop-hdfs:

  org.apache.hadoop.hdfs.web.TestWebHDFSForHA

{color:green}+1 contrib tests{color}.  The patch passed contrib unit tests.

Test results: 
https://builds.apache.org/job/PreCommit-HDFS-Build/8700//testReport/
Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/8700//console

This message is automatically generated.

 DataNode in secure mode may throw NullPointerException if client connects 
 before DataNode registers itself with NameNode.
 -

 Key: HDFS-7382
 URL: https://issues.apache.org/jira/browse/HDFS-7382
 Project: Hadoop HDFS
  Issue Type: Bug
  Components: datanode, security
Reporter: Chris Nauroth
Assignee: Chris Nauroth
Priority: Minor
 Attachments: HDFS-7382.1.patch


 {{SaslDataTransferServer#receive}} needs to check if the DataNode is 
 listening on a privileged port.  It does this by checking the address from 
 the {{DatanodeID}}.  However, there is a window of time when this will be 
 {{null}}.  If a client is still holding a {{LocatedBlock}} that references 
 that DataNode and chooses to connect, then there is a risk of getting a 
 {{NullPointerException}}.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HDFS-7383) DataNode.requestShortCircuitFdsForRead may throw NullPointerException

2014-11-08 Thread Tsz Wo Nicholas Sze (JIRA)

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

Tsz Wo Nicholas Sze updated HDFS-7383:
--
Status: Patch Available  (was: Open)

 DataNode.requestShortCircuitFdsForRead may throw NullPointerException
 -

 Key: HDFS-7383
 URL: https://issues.apache.org/jira/browse/HDFS-7383
 Project: Hadoop HDFS
  Issue Type: Bug
  Components: datanode
Reporter: Tsz Wo Nicholas Sze
Assignee: Tsz Wo Nicholas Sze
 Attachments: h7383_20141108.patch


 {noformat}
 java.lang.NullPointerException
   at 
 org.apache.hadoop.hdfs.server.datanode.DataNode.requestShortCircuitFdsForRead(DataNode.java:1525)
   at 
 org.apache.hadoop.hdfs.server.datanode.DataXceiver.requestShortCircuitFds(DataXceiver.java:286)
   at 
 org.apache.hadoop.hdfs.protocol.datatransfer.Receiver.opRequestShortCircuitFds(Receiver.java:185)
   at 
 org.apache.hadoop.hdfs.protocol.datatransfer.Receiver.processOp(Receiver.java:89)
   at 
 org.apache.hadoop.hdfs.server.datanode.DataXceiver.run(DataXceiver.java:234)
   at java.lang.Thread.run(Thread.java:745)
 {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HDFS-7383) DataNode.requestShortCircuitFdsForRead may throw NullPointerException

2014-11-08 Thread Tsz Wo Nicholas Sze (JIRA)

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

Tsz Wo Nicholas Sze updated HDFS-7383:
--
Attachment: h7383_20141108.patch

h7383_20141108.patch: checks null.

 DataNode.requestShortCircuitFdsForRead may throw NullPointerException
 -

 Key: HDFS-7383
 URL: https://issues.apache.org/jira/browse/HDFS-7383
 Project: Hadoop HDFS
  Issue Type: Bug
  Components: datanode
Reporter: Tsz Wo Nicholas Sze
Assignee: Tsz Wo Nicholas Sze
 Attachments: h7383_20141108.patch


 {noformat}
 java.lang.NullPointerException
   at 
 org.apache.hadoop.hdfs.server.datanode.DataNode.requestShortCircuitFdsForRead(DataNode.java:1525)
   at 
 org.apache.hadoop.hdfs.server.datanode.DataXceiver.requestShortCircuitFds(DataXceiver.java:286)
   at 
 org.apache.hadoop.hdfs.protocol.datatransfer.Receiver.opRequestShortCircuitFds(Receiver.java:185)
   at 
 org.apache.hadoop.hdfs.protocol.datatransfer.Receiver.processOp(Receiver.java:89)
   at 
 org.apache.hadoop.hdfs.server.datanode.DataXceiver.run(DataXceiver.java:234)
   at java.lang.Thread.run(Thread.java:745)
 {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HDFS-7358) Clients may get stuck waiting when using ByteArrayManager

2014-11-08 Thread Tsz Wo Nicholas Sze (JIRA)

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

Tsz Wo Nicholas Sze commented on HDFS-7358:
---

Something wrong with test-patch?  This is no findbugs warning in the given 
newPatchFindbugsWarningshadoop-hdfs.html.

 Clients may get stuck waiting when using ByteArrayManager
 -

 Key: HDFS-7358
 URL: https://issues.apache.org/jira/browse/HDFS-7358
 Project: Hadoop HDFS
  Issue Type: Bug
  Components: hdfs-client
Reporter: Tsz Wo Nicholas Sze
Assignee: Tsz Wo Nicholas Sze
 Attachments: h7358_20141104.patch, h7358_20141104_wait_timeout.patch, 
 h7358_20141105.patch, h7358_20141106.patch, h7358_20141107.patch, 
 h7358_20141108.patch


 [~stack] reported that clients might get stuck waiting when using 
 ByteArrayManager; see [his 
 comments|https://issues.apache.org/jira/browse/HDFS-7276?focusedCommentId=14197036page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-14197036].



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HDFS-7312) Update DistCp v1 to optionally not use tmp location

2014-11-08 Thread Yongjun Zhang (JIRA)

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

Yongjun Zhang commented on HDFS-7312:
-

Hi [~jprosser],

Thanks for reporting the issue. FYI,  HADOOP-11281.described the same issue. 
It's nice that you are working on.



 Update DistCp v1 to optionally not use tmp location
 ---

 Key: HDFS-7312
 URL: https://issues.apache.org/jira/browse/HDFS-7312
 Project: Hadoop HDFS
  Issue Type: Improvement
  Components: tools
Affects Versions: 2.5.1
Reporter: Joseph Prosser
Assignee: Joseph Prosser
Priority: Minor
   Original Estimate: 72h
  Remaining Estimate: 72h

 DistCp v1 currently copies files to a tmp location and then renames that to 
 the specified destination.  This can cause performance issues on filesystems 
 such as S3.  A -skiptmp flag will be added to bypass this step and copy 
 directly to the destination.  This feature mirrors a similar one added to 
 HBase ExportSnapshot 
 [HBASE-9|https://issues.apache.org/jira/browse/HBASE-9]



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HDFS-7382) DataNode in secure mode may throw NullPointerException if client connects before DataNode registers itself with NameNode.

2014-11-08 Thread Yongjun Zhang (JIRA)

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

Yongjun Zhang commented on HDFS-7382:
-

HI [~cnauroth],

Thanks for reporting the issue and working on the solution. I reviewed the fix, 
it looks good to me. I happen to notice a typo in the related code at line 113:

{code}
   } else if (datanodeId.getXferPort()  1024) {
  LOG.debug(
SASL server skipping handshake in unsecured configuration for 
+ peer = {}, datanodeId = {}, peer, datanodeId);
  return new IOStreamPair(underlyingIn, underlyingOut);
} else if (dnConf.getSaslPropsResolver() != null) {
{code}
In the debug message above, the unsecured meant  secured.  Can we fix it in 
this patch?

Another thing is, I wish the 1024 is a constant defined in the security util 
somewhere, but it pre-existed. So I think it we were to replace it, it does not 
have to be the scope of this jira.

Thanks again.





 DataNode in secure mode may throw NullPointerException if client connects 
 before DataNode registers itself with NameNode.
 -

 Key: HDFS-7382
 URL: https://issues.apache.org/jira/browse/HDFS-7382
 Project: Hadoop HDFS
  Issue Type: Bug
  Components: datanode, security
Reporter: Chris Nauroth
Assignee: Chris Nauroth
Priority: Minor
 Attachments: HDFS-7382.1.patch


 {{SaslDataTransferServer#receive}} needs to check if the DataNode is 
 listening on a privileged port.  It does this by checking the address from 
 the {{DatanodeID}}.  However, there is a window of time when this will be 
 {{null}}.  If a client is still holding a {{LocatedBlock}} that references 
 that DataNode and chooses to connect, then there is a risk of getting a 
 {{NullPointerException}}.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HDFS-7383) DataNode.requestShortCircuitFdsForRead may throw NullPointerException

2014-11-08 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HDFS-7383:
-

{color:red}-1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12680441/h7383_20141108.patch
  against trunk revision 6caa810.

{color:green}+1 @author{color}.  The patch does not contain any @author 
tags.

{color:red}-1 tests included{color}.  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.

{color:green}+1 javac{color}.  The applied patch does not increase the 
total number of javac compiler warnings.

{color:green}+1 javadoc{color}.  There were no new javadoc warning messages.

{color:green}+1 eclipse:eclipse{color}.  The patch built with 
eclipse:eclipse.

{color:green}+1 findbugs{color}.  The patch does not introduce any new 
Findbugs (version 2.0.3) warnings.

{color:green}+1 release audit{color}.  The applied patch does not increase 
the total number of release audit warnings.

{color:red}-1 core tests{color}.  The patch failed these unit tests in 
hadoop-hdfs-project/hadoop-hdfs:

  org.apache.hadoop.hdfs.server.balancer.TestBalancer
  
org.apache.hadoop.hdfs.server.namenode.ha.TestFailureToReadEdits

  The following test timeouts occurred in 
hadoop-hdfs-project/hadoop-hdfs:

org.apache.hadoop.hdfs.util.TestByteArrayManager

{color:green}+1 contrib tests{color}.  The patch passed contrib unit tests.

Test results: 
https://builds.apache.org/job/PreCommit-HDFS-Build/8701//testReport/
Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/8701//console

This message is automatically generated.

 DataNode.requestShortCircuitFdsForRead may throw NullPointerException
 -

 Key: HDFS-7383
 URL: https://issues.apache.org/jira/browse/HDFS-7383
 Project: Hadoop HDFS
  Issue Type: Bug
  Components: datanode
Reporter: Tsz Wo Nicholas Sze
Assignee: Tsz Wo Nicholas Sze
 Attachments: h7383_20141108.patch


 {noformat}
 java.lang.NullPointerException
   at 
 org.apache.hadoop.hdfs.server.datanode.DataNode.requestShortCircuitFdsForRead(DataNode.java:1525)
   at 
 org.apache.hadoop.hdfs.server.datanode.DataXceiver.requestShortCircuitFds(DataXceiver.java:286)
   at 
 org.apache.hadoop.hdfs.protocol.datatransfer.Receiver.opRequestShortCircuitFds(Receiver.java:185)
   at 
 org.apache.hadoop.hdfs.protocol.datatransfer.Receiver.processOp(Receiver.java:89)
   at 
 org.apache.hadoop.hdfs.server.datanode.DataXceiver.run(DataXceiver.java:234)
   at java.lang.Thread.run(Thread.java:745)
 {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HDFS-7382) DataNode in secure mode may throw NullPointerException if client connects before DataNode registers itself with NameNode.

2014-11-08 Thread Chris Nauroth (JIRA)

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

Chris Nauroth updated HDFS-7382:

Attachment: HDFS-7382.2.patch

Thank you for the code reviews, Nicholas and Yongjun.

Yongjun, I'm uploading patch v2 with the log message fixed like you suggested.  
Nice catch!  I'm going to commit this version based on the prior +1 from 
Nicholas, and this new version is only changing a string literal in a log 
message.

You're right that there are a few different places checking for a port number  
1024.  We might benefit from a shared {{isPrivilegedPort}} method somewhere.  
One interesting aspect is that Windows has no concept of privileged ports, so 
perhaps this method would return false always on Windows.  I won't address it 
here, but I do encourage you to file a jira for that chnage.

bq. -1 tests included. The patch doesn't appear to include any new or modified 
tests.

This turned up in internal system testing that involved restart DataNodes while 
running jobs.  It would be very difficult to put this into a predictable test.

The failure in {{TestWebHDFSForHA}} looks unrelated.  The test passed locally.

 DataNode in secure mode may throw NullPointerException if client connects 
 before DataNode registers itself with NameNode.
 -

 Key: HDFS-7382
 URL: https://issues.apache.org/jira/browse/HDFS-7382
 Project: Hadoop HDFS
  Issue Type: Bug
  Components: datanode, security
Reporter: Chris Nauroth
Assignee: Chris Nauroth
Priority: Minor
 Attachments: HDFS-7382.1.patch, HDFS-7382.2.patch


 {{SaslDataTransferServer#receive}} needs to check if the DataNode is 
 listening on a privileged port.  It does this by checking the address from 
 the {{DatanodeID}}.  However, there is a window of time when this will be 
 {{null}}.  If a client is still holding a {{LocatedBlock}} that references 
 that DataNode and chooses to connect, then there is a risk of getting a 
 {{NullPointerException}}.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HDFS-7382) DataNode in secure mode may throw NullPointerException if client connects before DataNode registers itself with NameNode.

2014-11-08 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HDFS-7382:
-

{color:red}-1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12680464/HDFS-7382.2.patch
  against trunk revision 9ba8d8c.

{color:red}-1 patch{color}.  The patch command could not apply the patch.

Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/8703//console

This message is automatically generated.

 DataNode in secure mode may throw NullPointerException if client connects 
 before DataNode registers itself with NameNode.
 -

 Key: HDFS-7382
 URL: https://issues.apache.org/jira/browse/HDFS-7382
 Project: Hadoop HDFS
  Issue Type: Bug
  Components: datanode, security
Reporter: Chris Nauroth
Assignee: Chris Nauroth
Priority: Minor
 Attachments: HDFS-7382.1.patch, HDFS-7382.2.patch


 {{SaslDataTransferServer#receive}} needs to check if the DataNode is 
 listening on a privileged port.  It does this by checking the address from 
 the {{DatanodeID}}.  However, there is a window of time when this will be 
 {{null}}.  If a client is still holding a {{LocatedBlock}} that references 
 that DataNode and chooses to connect, then there is a risk of getting a 
 {{NullPointerException}}.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HDFS-7382) DataNode in secure mode may throw NullPointerException if client connects before DataNode registers itself with NameNode.

2014-11-08 Thread Hudson (JIRA)

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

Hudson commented on HDFS-7382:
--

FAILURE: Integrated in Hadoop-trunk-Commit #6494 (See 
[https://builds.apache.org/job/Hadoop-trunk-Commit/6494/])
HDFS-7382. DataNode in secure mode may throw NullPointerException if client 
connects before DataNode registers itself with NameNode. Contributed by Chris 
Nauroth. (cnauroth: rev 9ba8d8c7eb65eeb6fe673f04e493d9eedd95a822)
* 
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/datanode/DataXceiver.java
* 
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/protocol/datatransfer/sasl/SaslDataTransferServer.java
* hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt


 DataNode in secure mode may throw NullPointerException if client connects 
 before DataNode registers itself with NameNode.
 -

 Key: HDFS-7382
 URL: https://issues.apache.org/jira/browse/HDFS-7382
 Project: Hadoop HDFS
  Issue Type: Bug
  Components: datanode, security
Reporter: Chris Nauroth
Assignee: Chris Nauroth
Priority: Minor
 Attachments: HDFS-7382.1.patch, HDFS-7382.2.patch


 {{SaslDataTransferServer#receive}} needs to check if the DataNode is 
 listening on a privileged port.  It does this by checking the address from 
 the {{DatanodeID}}.  However, there is a window of time when this will be 
 {{null}}.  If a client is still holding a {{LocatedBlock}} that references 
 that DataNode and chooses to connect, then there is a risk of getting a 
 {{NullPointerException}}.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HDFS-7382) DataNode in secure mode may throw NullPointerException if client connects before DataNode registers itself with NameNode.

2014-11-08 Thread Chris Nauroth (JIRA)

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

Chris Nauroth updated HDFS-7382:

   Resolution: Fixed
Fix Version/s: 2.6.0
   Status: Resolved  (was: Patch Available)

I committed this to trunk, branch-2 and branch-2.6.  Nicholas and Yongjun, 
thank you again for the code reviews.

 DataNode in secure mode may throw NullPointerException if client connects 
 before DataNode registers itself with NameNode.
 -

 Key: HDFS-7382
 URL: https://issues.apache.org/jira/browse/HDFS-7382
 Project: Hadoop HDFS
  Issue Type: Bug
  Components: datanode, security
Reporter: Chris Nauroth
Assignee: Chris Nauroth
Priority: Minor
 Fix For: 2.6.0

 Attachments: HDFS-7382.1.patch, HDFS-7382.2.patch


 {{SaslDataTransferServer#receive}} needs to check if the DataNode is 
 listening on a privileged port.  It does this by checking the address from 
 the {{DatanodeID}}.  However, there is a window of time when this will be 
 {{null}}.  If a client is still holding a {{LocatedBlock}} that references 
 that DataNode and chooses to connect, then there is a risk of getting a 
 {{NullPointerException}}.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HDFS-7381) Decouple the management of block id and gen stamps from FSNamesystem

2014-11-08 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HDFS-7381:
-

{color:red}-1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12680304/HDFS-7381.000.patch
  against trunk revision 6caa810.

{color:green}+1 @author{color}.  The patch does not contain any @author 
tags.

{color:green}+1 tests included{color}.  The patch appears to include 2 new 
or modified test files.

{color:green}+1 javac{color}.  The applied patch does not increase the 
total number of javac compiler warnings.

{color:green}+1 javadoc{color}.  There were no new javadoc warning messages.

{color:red}-1 eclipse:eclipse{color}.  The patch failed to build with 
eclipse:eclipse.

{color:green}+1 findbugs{color}.  The patch does not introduce any new 
Findbugs (version 2.0.3) warnings.

{color:green}+1 release audit{color}.  The applied patch does not increase 
the total number of release audit warnings.

{color:green}+1 core tests{color}.  The patch passed unit tests in .

{color:green}+1 contrib tests{color}.  The patch passed contrib unit tests.

Test results: 
https://builds.apache.org/job/PreCommit-HDFS-Build/8702//testReport/
Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/8702//console

This message is automatically generated.

 Decouple the management of block id and gen stamps from FSNamesystem
 

 Key: HDFS-7381
 URL: https://issues.apache.org/jira/browse/HDFS-7381
 Project: Hadoop HDFS
  Issue Type: Improvement
Reporter: Haohui Mai
Assignee: Haohui Mai
 Attachments: HDFS-7381.000.patch


 The block layer should be responsible of managing block ids and generation 
 stamps. Currently the functionality is misplace into {{FSNamesystem}}.
 This jira proposes to decouple them from the {{FSNamesystem}} class.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HDFS-7383) DataNode.requestShortCircuitFdsForRead may throw NullPointerException

2014-11-08 Thread Chris Nauroth (JIRA)

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

Chris Nauroth updated HDFS-7383:

Hadoop Flags: Reviewed

+1 for the patch.  The test failures look unrelated, and I verified that the 
tests pass in my local environment.  Thank you for the patch, Nicholas.

 DataNode.requestShortCircuitFdsForRead may throw NullPointerException
 -

 Key: HDFS-7383
 URL: https://issues.apache.org/jira/browse/HDFS-7383
 Project: Hadoop HDFS
  Issue Type: Bug
  Components: datanode
Reporter: Tsz Wo Nicholas Sze
Assignee: Tsz Wo Nicholas Sze
 Attachments: h7383_20141108.patch


 {noformat}
 java.lang.NullPointerException
   at 
 org.apache.hadoop.hdfs.server.datanode.DataNode.requestShortCircuitFdsForRead(DataNode.java:1525)
   at 
 org.apache.hadoop.hdfs.server.datanode.DataXceiver.requestShortCircuitFds(DataXceiver.java:286)
   at 
 org.apache.hadoop.hdfs.protocol.datatransfer.Receiver.opRequestShortCircuitFds(Receiver.java:185)
   at 
 org.apache.hadoop.hdfs.protocol.datatransfer.Receiver.processOp(Receiver.java:89)
   at 
 org.apache.hadoop.hdfs.server.datanode.DataXceiver.run(DataXceiver.java:234)
   at java.lang.Thread.run(Thread.java:745)
 {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HDFS-6735) A minor optimization to avoid pread() be blocked by read() inside the same DFSInputStream

2014-11-08 Thread Lars Hofhansl (JIRA)

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

Lars Hofhansl updated HDFS-6735:

Attachment: HDFS-6735-v3.txt

I classified the state in DFSInputStream into state used by read only and state 
used by both read and pread.

With that here's a new proposed patch.
* makes LocatedBlocks immutable (which was intended it seems)
* pread no longer affects currentNode (that was unintended I think)
* guards state shared between read and pread with an extra sharedLock
(the state used for read only is still guarded by a lock on this, which we 
need to take anyway to avoid concurrent stateful reads against the same input 
stream)
* removed all synchronized on private method that were only called from methods 
already synchronized (good practice anyway)
* makes cachingStrategy volatile (made more sense than locking there)
* should be free of deadlocks (never acquire lock on this with sharedLock 
held, but the reverse is possible)
* pos, blockEnd, currentLocatedBlock are not updated in getBlockAt unless 
called on behalf of read (not for pread, hence locking on this not needed 
there)

I have not tested this, yet.
Please have a careful look and let me know what you think.
We might want to further disentangle the mixed state.

(And just maybe the best solution would be for HBase to have an input stream 
for each thread doing read and one for all threads doing preads - and not do 
any of this...?)

 A minor optimization to avoid pread() be blocked by read() inside the same 
 DFSInputStream
 -

 Key: HDFS-6735
 URL: https://issues.apache.org/jira/browse/HDFS-6735
 Project: Hadoop HDFS
  Issue Type: Improvement
  Components: hdfs-client
Affects Versions: 3.0.0
Reporter: Liang Xie
Assignee: Liang Xie
 Attachments: HDFS-6735-v2.txt, HDFS-6735-v3.txt, HDFS-6735.txt


 In current DFSInputStream impl, there're a couple of coarser-grained locks in 
 read/pread path, and it has became a HBase read latency pain point so far. In 
 HDFS-6698, i made a minor patch against the first encourtered lock, around 
 getFileLength, in deed, after reading code and testing, it shows still other 
 locks we could improve.
 In this jira, i'll make a patch against other locks, and a simple test case 
 to show the issue and the improved result.
 This is important for HBase application, since in current HFile read path, we 
 issue all read()/pread() requests in the same DFSInputStream for one HFile. 
 (Multi streams solution is another story i had a plan to do, but probably 
 will take more time than i expected)



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)