anmolanmol1234 commented on code in PR #7265:
URL: https://github.com/apache/hadoop/pull/7265#discussion_r1937002186
##########
hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/services/TestAbfsRenameRetryRecovery.java:
##########
@@ -434,6 +437,9 @@ public void testExistingPathCorrectlyRejected() throws
Exception {
@Test
public void testRenameRecoveryUnsupportedForFlatNamespace() throws Exception
{
Assume.assumeTrue(!isNamespaceEnabled);
+ // In DFS endpoint, renamePath is O(1) API call and idempotency issue can
happen.
+ // For blob endpoint, client orchestrates the rename operation.
+ Assume.assumeTrue(getFileSystem().getAbfsStore().getClient() instanceof
AbfsDfsClient);
Review Comment:
Now we have getAbfsServiceType() method in trunk so can be use everywhere
where we are checking for client, we can directly check for service type
--
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]
