anmolanmol1234 commented on code in PR #6314:
URL: https://github.com/apache/hadoop/pull/6314#discussion_r1439323259
##########
hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/services/TestAbfsRestOperationMockFailures.java:
##########
@@ -188,7 +187,7 @@ private void testClientRequestIdForStatusRetry(int status,
int[] statusCount = new int[1];
statusCount[0] = 0;
Mockito.doAnswer(answer -> {
- if (statusCount[0] <= 5) {
+ if (statusCount[0] <= 10) {
Review Comment:
Since we are updating the metrics based on status code, the getStatusCode()
method is called more than the above mentioned number. I think we should not
have code where we specifically mention number of times a method is called as
logic to return something as that can change frequently.
--
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]
