c-w commented on a change in pull request #452: HADOOP-16005: Add XAttr support
to WASB and ABFS
URL: https://github.com/apache/hadoop/pull/452#discussion_r356202673
##########
File path:
hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azure/NativeAzureFileSystem.java
##########
@@ -3549,6 +3550,90 @@ public void setOwner(Path p, String username, String
groupname)
}
}
+ /**
+ * Set the value of an attribute for a path.
+ *
+ * @param path The path on which to set the attribute
+ * @param xAttrName The attribute to set
+ * @param value The byte value of the attribute to set (encoded in utf-8)
+ * @param flag The mode in which to set the attribute
+ * @throws IOException If there was an issue setting the attribute on Azure
+ */
+ @Override
+ public void setXAttr(Path path, String xAttrName, byte[] value,
EnumSet<XAttrSetFlag> flag) throws IOException {
Review comment:
Thanks for the pointers. I was previously following the style of
[setOwner](https://github.com/apache/hadoop/blob/29049e6fee271298f28ff2c484c940527370c96d/hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azure/NativeAzureFileSystem.java#L3498-L3551).
I removed the newlines you mentioned in f7c0d26.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
