anmolanmol1234 commented on code in PR #7777:
URL: https://github.com/apache/hadoop/pull/7777#discussion_r2213048850
##########
hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azurebfs/services/AbfsBlobClient.java:
##########
@@ -1914,7 +1920,11 @@ private List<AbfsHttpHeader>
getMetadataHeadersList(final Hashtable<String, Stri
// AzureBlobFileSystem supports only ASCII Characters in property values.
if (isPureASCII(value)) {
try {
- value = encodeMetadataAttribute(value);
+ // URL encoding this JSON metadata, set by the WASB Client during
file creation, causes compatibility issues.
Review Comment:
Create via WASB, setXAttr via ABFS, create overwrite via WASB -> there is a
test added for this scenario where this flow is getting tested, do you suggest
adding a mocked test as well ?
--
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: common-issues-unsubscr...@hadoop.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org
