F64116045 opened a new pull request, #11469: URL: https://github.com/apache/ozone/pull/11469
## What changes were proposed in this pull request? `TestSstFileSetReader` creates 1,000 keys per SST file, but only ten of them are used to generate the read boundaries. The other keys make the tests slower without adding more test cases. This change reduces the number of keys per file to 10. Local test time decreased from 131.9 seconds to 4.836 seconds. ## What is the link to the Apache JIRA https://issues.apache.org/jira/browse/HDDS-16821 ## How was this patch tested? <details> <summary>Local</summary> **Before** ```text [INFO] Tests run: 18, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 131.9 s -- in org.apache.hadoop.hdds.utils.db.TestSstFileSetReader org.apache.hadoop.hdds.utils.db.TestSstFileSetReader.testGetKeyStreamWithTombstone(int)[1] -- Time elapsed: 0.295 s org.apache.hadoop.hdds.utils.db.TestSstFileSetReader.testGetKeyStreamWithTombstone(int)[2] -- Time elapsed: 3.585 s org.apache.hadoop.hdds.utils.db.TestSstFileSetReader.testGetKeyStreamWithTombstone(int)[3] -- Time elapsed: 6.321 s org.apache.hadoop.hdds.utils.db.TestSstFileSetReader.testGetKeyStreamWithTombstone(int)[4] -- Time elapsed: 9.76 s org.apache.hadoop.hdds.utils.db.TestSstFileSetReader.testGetKeyStreamWithTombstone(int)[5] -- Time elapsed: 22.457 s org.apache.hadoop.hdds.utils.db.TestSstFileSetReader.testGetKeyStreamWithTombstone(int)[6] -- Time elapsed: 32.044 s org.apache.hadoop.hdds.utils.db.TestSstFileSetReader.testGetKeyStream(int)[1] -- Time elapsed: 0.026 s org.apache.hadoop.hdds.utils.db.TestSstFileSetReader.testGetKeyStream(int)[2] -- Time elapsed: 2.414 s org.apache.hadoop.hdds.utils.db.TestSstFileSetReader.testGetKeyStream(int)[3] -- Time elapsed: 5.027 s org.apache.hadoop.hdds.utils.db.TestSstFileSetReader.testGetKeyStream(int)[4] -- Time elapsed: 7.175 s org.apache.hadoop.hdds.utils.db.TestSstFileSetReader.testGetKeyStream(int)[5] -- Time elapsed: 17.391 s org.apache.hadoop.hdds.utils.db.TestSstFileSetReader.testGetKeyStream(int)[6] -- Time elapsed: 25.151 s org.apache.hadoop.hdds.utils.db.TestSstFileSetReader.testDuplicateKeyHandlingWithLatestFilePrecedence(int)[1] -- Time elapsed: 0.031 s org.apache.hadoop.hdds.utils.db.TestSstFileSetReader.testDuplicateKeyHandlingWithLatestFilePrecedence(int)[2] -- Time elapsed: 0.023 s org.apache.hadoop.hdds.utils.db.TestSstFileSetReader.testDuplicateKeyHandlingWithLatestFilePrecedence(int)[3] -- Time elapsed: 0.022 s org.apache.hadoop.hdds.utils.db.TestSstFileSetReader.testMinHeapWithOverlappingSstFiles(int)[1] -- Time elapsed: 0.013 s org.apache.hadoop.hdds.utils.db.TestSstFileSetReader.testMinHeapWithOverlappingSstFiles(int)[2] -- Time elapsed: 0.02 s org.apache.hadoop.hdds.utils.db.TestSstFileSetReader.testMinHeapWithOverlappingSstFiles(int)[3] -- Time elapsed: 0.031 s ``` **After** ```text [INFO] Tests run: 18, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 4.836 s -- in org.apache.hadoop.hdds.utils.db.TestSstFileSetReader org.apache.hadoop.hdds.utils.db.TestSstFileSetReader.testGetKeyStreamWithTombstone(int)[1] -- Time elapsed: 0.284 s org.apache.hadoop.hdds.utils.db.TestSstFileSetReader.testGetKeyStreamWithTombstone(int)[2] -- Time elapsed: 0.432 s org.apache.hadoop.hdds.utils.db.TestSstFileSetReader.testGetKeyStreamWithTombstone(int)[3] -- Time elapsed: 0.325 s org.apache.hadoop.hdds.utils.db.TestSstFileSetReader.testGetKeyStreamWithTombstone(int)[4] -- Time elapsed: 0.42 s org.apache.hadoop.hdds.utils.db.TestSstFileSetReader.testGetKeyStreamWithTombstone(int)[5] -- Time elapsed: 0.806 s org.apache.hadoop.hdds.utils.db.TestSstFileSetReader.testGetKeyStreamWithTombstone(int)[6] -- Time elapsed: 1.125 s org.apache.hadoop.hdds.utils.db.TestSstFileSetReader.testGetKeyStream(int)[1] -- Time elapsed: 0.007 s org.apache.hadoop.hdds.utils.db.TestSstFileSetReader.testGetKeyStream(int)[2] -- Time elapsed: 0.112 s org.apache.hadoop.hdds.utils.db.TestSstFileSetReader.testGetKeyStream(int)[3] -- Time elapsed: 0.167 s org.apache.hadoop.hdds.utils.db.TestSstFileSetReader.testGetKeyStream(int)[4] -- Time elapsed: 0.171 s org.apache.hadoop.hdds.utils.db.TestSstFileSetReader.testGetKeyStream(int)[5] -- Time elapsed: 0.347 s org.apache.hadoop.hdds.utils.db.TestSstFileSetReader.testGetKeyStream(int)[6] -- Time elapsed: 0.389 s org.apache.hadoop.hdds.utils.db.TestSstFileSetReader.testDuplicateKeyHandlingWithLatestFilePrecedence(int)[1] -- Time elapsed: 0.021 s org.apache.hadoop.hdds.utils.db.TestSstFileSetReader.testDuplicateKeyHandlingWithLatestFilePrecedence(int)[2] -- Time elapsed: 0.019 s org.apache.hadoop.hdds.utils.db.TestSstFileSetReader.testDuplicateKeyHandlingWithLatestFilePrecedence(int)[3] -- Time elapsed: 0.02 s org.apache.hadoop.hdds.utils.db.TestSstFileSetReader.testMinHeapWithOverlappingSstFiles(int)[1] -- Time elapsed: 0.012 s org.apache.hadoop.hdds.utils.db.TestSstFileSetReader.testMinHeapWithOverlappingSstFiles(int)[2] -- Time elapsed: 0.013 s org.apache.hadoop.hdds.utils.db.TestSstFileSetReader.testMinHeapWithOverlappingSstFiles(int)[3] -- Time elapsed: 0.02 s ``` </details> CI: -- 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]
