mukund-thakur commented on a change in pull request #3240:
URL: https://github.com/apache/hadoop/pull/3240#discussion_r678038991
##########
File path:
hadoop-tools/hadoop-distcp/src/test/java/org/apache/hadoop/tools/contract/AbstractContractDistCpTest.java
##########
@@ -659,6 +683,41 @@ private int getTotalFiles() {
return totalFiles;
}
+ /**
+ * Override point: should direct write always be used?
+ * false by default; enable for stores where rename is slow.
+ * @return true if direct write should be used in all tests.
+ */
+ protected boolean directWriteAlways() {
Review comment:
nit: method name contains Always but default is false, a bit ambiguous
no?
##########
File path:
hadoop-tools/hadoop-distcp/src/test/java/org/apache/hadoop/tools/contract/AbstractContractDistCpTest.java
##########
@@ -758,7 +828,7 @@ public void testDistCpWithUpdateExistFile() throws
Exception {
verifyPathExists(remoteFS, "", source);
verifyPathExists(localFS, "", dest);
DistCpTestUtils.assertRunDistCp(DistCpConstants.SUCCESS, source.toString(),
- dest.toString(), "-delete -update", conf);
+ dest.toString(), "-delete -update" + getDefaultCLIOptions(), conf);
Review comment:
nit : options can be refactored like it is done above.
--
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]
