From 7a0903b7436e7f3fb4f07c40bed2218a2653ba38 Mon Sep 17 00:00:00 2001 From: Mehakmeet Singh Date: Thu, 16 Feb 2023 10:09:06 +0530 Subject: [PATCH] HADOOP-18633. fix test AbstractContractDistCpTest#testDistCpUpdateCheckFileSkip (#5401) Contributed by: Mehakmeet Singh --- hadoop-tools/hadoop-distcp/src/site/markdown/DistCp.md.vm | 2 +- .../hadoop/tools/contract/AbstractContractDistCpTest.java | 3 +-- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/hadoop-tools/hadoop-distcp/src/site/markdown/DistCp.md.vm b/hadoop-tools/hadoop-distcp/src/site/markdown/DistCp.md.vm index 2d77619d5c..0e11b24529 100644 --- a/hadoop-tools/hadoop-distcp/src/site/markdown/DistCp.md.vm +++ b/hadoop-tools/hadoop-distcp/src/site/markdown/DistCp.md.vm @@ -653,7 +653,7 @@ checksums if the checksum algorithm between the two stores is different. * `distcp.update.modification.time` would only be used if either of the two stores don't have checksum validation resulting in incompatible checksum comparison between the two. Even if the property is set to true, it won't - be used if their is valid checksum comparison between the two stores. + be used if there is valid checksum comparison between the two stores. To turn off the modification time check, set this in your core-site.xml ```xml diff --git a/hadoop-tools/hadoop-distcp/src/test/java/org/apache/hadoop/tools/contract/AbstractContractDistCpTest.java b/hadoop-tools/hadoop-distcp/src/test/java/org/apache/hadoop/tools/contract/AbstractContractDistCpTest.java index 532abc2aa4..aa42cb968d 100644 --- a/hadoop-tools/hadoop-distcp/src/test/java/org/apache/hadoop/tools/contract/AbstractContractDistCpTest.java +++ b/hadoop-tools/hadoop-distcp/src/test/java/org/apache/hadoop/tools/contract/AbstractContractDistCpTest.java @@ -899,8 +899,7 @@ public void testDistCpUpdateCheckFileSkip() throws Exception { // Creating a source file with certain dataset. byte[] sourceBlock = dataset(10, 'a', 'z'); - // Write the dataset and as well create the target path. - ContractTestUtils.createFile(localFS, dest, true, sourceBlock); + // Write the dataset. ContractTestUtils .writeDataset(remoteFS, source, sourceBlock, sourceBlock.length, 1024, true);