big-data/hadoop
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

HDFS-7997. The first non-existing xattr should also throw IOException. (zhouyingchao via yliu)

Browse Source
This commit is contained in:
yliu 2015-03-31 21:17:44 +08:00
parent b5a22e9838
commit 3df61f303a
3 changed files with 7 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
View File

@ -392,6 +392,9 @@ Release 2.8.0 - UNRELEASED
DatanodeDescriptor#updateHeartbeatState() (Brahma Reddy Battula via Colin DatanodeDescriptor#updateHeartbeatState() (Brahma Reddy Battula via Colin
P. McCabe) P. McCabe)
HDFS-7997. The first non-existing xattr should also throw IOException.
(zhouyingchao via yliu)
Release 2.7.0 - UNRELEASED Release 2.7.0 - UNRELEASED
INCOMPATIBLE CHANGES INCOMPATIBLE CHANGES

3
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/FSDirXAttrOp.java
View File

@ -108,7 +108,8 @@ static List<XAttr> getXAttrs(FSDirectory fsd, final String srcArg,
return filteredAll; return filteredAll;
} }
if (filteredAll == null || filteredAll.isEmpty()) { if (filteredAll == null || filteredAll.isEmpty()) {
return null; throw new IOException(
"At least one of the attributes provided was not found.");
} }
List<XAttr> toGet = Lists.newArrayListWithCapacity(xAttrs.size()); List<XAttr> toGet = Lists.newArrayListWithCapacity(xAttrs.size());
for (XAttr xAttr : xAttrs) { for (XAttr xAttr : xAttrs) {

4
hadoop-hdfs-project/hadoop-hdfs/src/test/resources/testXAttrConf.xml
View File

@ -420,8 +420,8 @@
</cleanup-commands> </cleanup-commands>
<comparators> <comparators>
<comparator> <comparator>
<type>ExactComparator</type> <type>SubstringComparator</type>
<expected-output># file: /file1#LF#</expected-output> <expected-output>At least one of the attributes provided was not found</expected-output>
</comparator> </comparator>
</comparators> </comparators>
</test> </test>