anujmodi2021 commented on code in PR #7582:
URL: https://github.com/apache/hadoop/pull/7582#discussion_r2031189514
##########
hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/ITestAzureBlobFileSystemListStatus.java:
##########
@@ -176,6 +183,31 @@ public void testListPathTracingContext() throws Exception {
Mockito.verify(spiedTracingContext, times(0)).constructHeader(any(),
any(), any());
}
+ @Test
+ public void testListPathParsingFailure() throws Exception {
+ assumeBlobServiceType();
+ AzureBlobFileSystem spiedFs = Mockito.spy(getFileSystem());
+ AzureBlobFileSystemStore spiedStore = Mockito.spy(spiedFs.getAbfsStore());
+ AbfsBlobClient spiedClient = Mockito.spy(spiedStore.getClientHandler()
+ .getBlobClient());
+ Mockito.doReturn(spiedStore).when(spiedFs).getAbfsStore();
+ Mockito.doReturn(spiedClient).when(spiedStore).getClient();
+
+ Mockito.doThrow(new
SocketException(CONNECTION_RESET_MESSAGE)).when(spiedClient).filterDuplicateEntriesAndRenamePendingFiles(any(),
any());
Review Comment:
This particular test is to make sure retry does not happen as here exception
is injected in parsing list result from local stream.
I have modified the test `testListPathTracingContext` to assert that
exception thrown within httpOperation.processResponse() is retried.
--
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
