RenameDir and DeleteDir on blob to push number of blobs operated on last API call.#76
Conversation
…marker copy/delete
…rimaryIdIsSameInAllTheStepsOfBlobRename
|
Example where in rename failed and resumed from getFileStatus HttpRequest: DeleteBlob: 202,,,cid=:eb667826-a694-4562-bd99-a1ff0534b861:3b9df2a2-bcde-44d7-b5f2-7d65803f30f4:4ffe3621-21d7-49b3-b9ab-a9300e9cf2da::RN:0:B,rid=c2e0ffa0-101e-0046-0490-a91f5800000 failure happened, now via getFileStatus 2023-06-28 00:17:12,020 DEBUG [JUnit-testBlobRenameResumeWithGetFileStatus]: services.AbfsClient (AbfsRestOperation.java:executeHttpOperation(383)) - HttpRequest: GetBlobProperties: 200,,,cid=:9e9fa237-4bb9-4f2c-a1f1-c2c1cfa42f59:3b9df2a2-bcde-44d7-b5f2-7d65803f30f4:e76becea-4dd5-4d0c-8362-d7a78a996267::GF:0:B,rid=c2e110e6-101e-0046-3390-a91f58000000,sent=0,recv=0,HEAD,https://pranavsaxenanonhns.blob.core.windows.net/abfs-testcontainer-8589e309-e8c5-4317-9835-4ce19886f932/hbase/testDir?timeout=90 2023-06-28 00:17:12,949 DEBUG [JUnit-testBlobRenameResumeWithGetFileStatus]: services.AbfsClient (AbfsRestOperation.java:completeExecute(230)) - First execution of REST operation - LeaseBlob |
|
Example where rename failed is resumed from listStatus HttpRequest: CopyBlob: 202,,,cid=:f668e2e5-4ef4-472f-a0cf-f0f7b9b14c70:53441220-a477-48e6-9f84-b1c9d9b5dc12:ad767c15-1b65-4de0-b2c9-0791fa3b4152::RN:0:B,rid=7bd53d60-701e-006f-1292-a9212c000000,sent=0,recv=0,PUT HttpRequest: DeleteBlob: 202,,,cid=:2a8facea-96b7-472d-89e0-62bdb93b3308:53441220-a477-48e6-9f84-b1c9d9b5dc12:ad767c15-1b65-4de0-b2c9-0791fa3b4152::RN:0:B,rid=ddf48981-f01e-0061-4e92-a9089c000000,sent=0,recv=0,DELETE Failure happens, now will be taken from listStatus: UG [JUnit-testBlobRenameResumeWithListStatus]: services.AbfsClient (AbfsRestOperation.java:signRequest(290)) - Signing request with shared key |
| } | ||
| renameBlobExecutorService.shutdown(); | ||
|
|
||
| tracingContext.setOperatedBlobCount(renamedBlob.get() + 1); |
There was a problem hiding this comment.
Since we are already doing an increment and get, why +1 here ?
There was a problem hiding this comment.
Increment is being done only for blobs in renamed in parallel
This +1 is for blob which would contain the count value in CID.
| deleteBlobExecutorService.shutdown(); | ||
| } | ||
|
|
||
| tracingContext.setOperatedBlobCount(deletedBlobCount.get() + 1); |
There was a problem hiding this comment.
Increment is being done only for blobs in renamed in parallel
This +1 is for blob which would contain the count value in CID.
| Assertions.assertThat(lmt).isEqualTo(newLmt); | ||
| } | ||
|
|
||
| @Test |
There was a problem hiding this comment.
Can you please add a small description for each test as in what the test is trying to achieve ?
There was a problem hiding this comment.
Added as javadoc in all new tests.
* tracingContextValidator impl addition * testBlobRenameResumeWithListStatus * Done for getFileStatus as well * general refactor * javadoc; updateRequestId
|
:::: AGGREGATED TEST RESULT :::: HNS-OAuth[INFO] Results: HNS-SharedKey[INFO] Results: NonHNS-SharedKey[INFO] Results: NonHNS-OAuth[INFO] Results: AppendBlob-HNS-OAuth[INFO] Results: Time taken: 43 mins 39 secs. azureuser@Hadoop-VM-EAST2:~/hadoop/hadoop-tools/hadoop-azure$ git log |
|
What has changed in this revision:
|
PR Details:
Example of final CID in listStatus:
Example in getFileStatus:
:::: AGGREGATED TEST RESULT ::::
HNS-OAuth
[INFO] Results:
[INFO]
[WARNING] Tests run: 140, Failures: 0, Errors: 0, Skipped: 4
[INFO] Results:
[INFO]
[ERROR] Errors:
[ERROR] ITestAzureBlobFileSystemLease.testAcquireRetry:383 » TestTimedOut test timed o...
[INFO]
[ERROR] Tests run: 772, Failures: 0, Errors: 1, Skipped: 182
[INFO] Results:
[INFO]
[WARNING] Tests run: 278, Failures: 0, Errors: 0, Skipped: 44
HNS-SharedKey
[INFO] Results:
[INFO]
[ERROR] Failures:
[ERROR] TestAbfsClientThrottlingAnalyzer.testManySuccessAndErrorsAndWaiting:181->fuzzyValidate:64 The actual value 9 is not within the expected range: [5.60, 8.40].
[INFO]
[ERROR] Tests run: 140, Failures: 1, Errors: 0, Skipped: 4
[INFO] Results:
[INFO]
[ERROR] Failures:
[ERROR] ITestAzureBlobFileSystemRandomRead.testValidateSeekBounds:276->Assert.assertTrue:42->Assert.fail:89 There should not be any network I/O (elapsedTimeMs=28).
[ERROR] Errors:
[ERROR] ITestAzureBlobFileSystemLease.testAcquireRetry:383 » TestTimedOut test timed o...
[INFO]
[ERROR] Tests run: 772, Failures: 1, Errors: 1, Skipped: 182
[INFO] Results:
[INFO]
[WARNING] Tests run: 278, Failures: 0, Errors: 0, Skipped: 44
NonHNS-SharedKey
[INFO] Results:
[INFO]
[WARNING] Tests run: 140, Failures: 0, Errors: 0, Skipped: 4
[INFO] Results:
[INFO]
[ERROR] Failures:
[ERROR] ITestAbfsStatistics.testCreateStatistics:108->AbstractAbfsIntegrationTest.assertAbfsStatistics:510->Assert.assertEquals:647->Assert.failNotEquals:835->Assert.fail:89 Mismatch in directories_created expected:<2> but was:<1>
[ERROR] ITestAzureBlobFileSystemRandomRead.testValidateSeekBounds:276->Assert.assertTrue:42->Assert.fail:89 There should not be any network I/O (elapsedTimeMs=140).
[INFO]
[ERROR] Tests run: 772, Failures: 2, Errors: 0, Skipped: 276
[INFO] Results:
[INFO]
[WARNING] Tests run: 278, Failures: 0, Errors: 0, Skipped: 45
NonHNS-OAuth
[INFO] Results:
[INFO]
[WARNING] Tests run: 140, Failures: 0, Errors: 0, Skipped: 4
[INFO] Results:
[INFO]
[ERROR] Failures:
[ERROR] ITestAbfsStatistics.testCreateStatistics:108->AbstractAbfsIntegrationTest.assertAbfsStatistics:510->Assert.assertEquals:647->Assert.failNotEquals:835->Assert.fail:89 Mismatch in directories_created expected:<2> but was:<1>
[INFO]
[ERROR] Tests run: 772, Failures: 1, Errors: 0, Skipped: 276
[INFO] Results:
[INFO]
[WARNING] Tests run: 278, Failures: 0, Errors: 0, Skipped: 45
AppendBlob-HNS-OAuth
[INFO] Results:
[INFO]
[WARNING] Tests run: 140, Failures: 0, Errors: 0, Skipped: 4
[INFO] Results:
[INFO]
[ERROR] Errors:
[ERROR] ITestAzureBlobFileSystemLease.testAcquireRetry:371 » TestTimedOut test timed o...
[INFO]
[ERROR] Tests run: 772, Failures: 0, Errors: 1, Skipped: 182
[INFO] Results:
[INFO]
[WARNING] Tests run: 278, Failures: 0, Errors: 0, Skipped: 44