anujmodi2021 opened a new pull request, #6003:
URL: https://github.com/apache/hadoop/pull/6003

   ### Description of PR
   Jira Ticket: https://issues.apache.org/jira/browse/HADOOP-18869 
   
   Following changes are included in this PR:
   1. HDFS.create() was failing on root path with Invalid URL. As a fix a check 
is added to avoid network call if root path is passed. Instead, an Exception 
will be thrown with relevant message.
   2. HDFS.createNonRecursive() was failing on root path with 
NullPointerException. As a fix a check is added to avoid network call if root 
path is passed. Instead, an Exception will be thrown with relevant message.
   3. HDFS.getXAttr() was failing for root path with Invalid URL. As a fix 
support for setting File System Properties added if root path is passed.
   4. HDFS.setXAttr() was failing for root path with Invalid URL. As a fix 
support for getting File System Properties added if root path is passed.
   5. Tests for getsetXAttr() were trying to set Unicode characters in 
attribute value encoded in ISO_8859-1 which does not support unicode, hence the 
value set by setXAttr() were going as undefined character. getXAttr() also 
retrieved them as undefined characters only. As a fix, assertions were changed 
to compare the both encoded and decoded values.
   Also, ABFS used DFS endpoint to interact with Azure services. As per the 
official documentation, these operations on DFS endpoint supports only ASCII 
characters. As a ix tests were modified to work only with ASCII characters.
   
   ### How was this patch tested?
   Existing tests were modified and more tests were added.
   Whole test suite was ran to verify the changes. Test results pasted below.
   
   ### For code changes:
   
   - [ ] Does the title or this PR starts with the corresponding JIRA issue id 
(e.g. 'HADOOP-17799. Your PR title ...')?
   - [ ] Object storage: have the integration tests been executed and the 
endpoint declared according to the connector-specific documentation?
   - [ ] If adding new dependencies to the code, are these dependencies 
licensed in a way that is compatible for inclusion under [ASF 
2.0](http://www.apache.org/legal/resolved.html#category-a)?
   - [ ] If applicable, have you updated the `LICENSE`, `LICENSE-binary`, 
`NOTICE-binary` files?
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to