Skip to content
This repository has been archived by the owner. It is now read-only.
Permalink
Browse files
FALCON-2333 FilteredCopyListing#shouldCopy override for new function …
…signature

Build succeeds on local with checkstyle enabled

[INFO] Reactor Summary:
[INFO]
[INFO] Apache Falcon ...................................... SUCCESS [ 29.596 s]
[INFO] Apache Falcon UI ................................... SUCCESS [ 30.419 s]
[INFO] Build Tools ........................................ SUCCESS [ 20.110 s]
[INFO] Apache Falcon Java common types .................... SUCCESS [ 54.352 s]
[INFO] Apache Falcon Metrics .............................. SUCCESS [ 21.369 s]
[INFO] Apache Falcon Hadoop Dependencies .................. SUCCESS [ 23.484 s]
[INFO] Apache Falcon Test Utility ......................... SUCCESS [ 19.320 s]
[INFO] Apache Falcon Commons .............................. SUCCESS [05:45 min]
[INFO] Apache Falcon extensions ........................... SUCCESS [01:19 min]
[INFO] Apache Falcon Java client .......................... SUCCESS [ 46.655 s]
[INFO] Apache Falcon Shell ................................ SUCCESS [ 28.133 s]
[INFO] Apache Falcon CLI client ........................... SUCCESS [ 30.136 s]
[INFO] Apache Falcon Titan 1.0 support .................... SUCCESS [ 14.624 s]
[INFO] Apache Falcon Oozie EL Extension ................... SUCCESS [ 24.562 s]
[INFO] Apache Falcon Embedded Hadoop - Test Cluster ....... SUCCESS [01:02 min]
[INFO] Apache Falcon Sharelib Hive - Test Cluster ......... SUCCESS [  7.371 s]
[INFO] Apache Falcon Sharelib Pig - Test Cluster .......... SUCCESS [  2.580 s]
[INFO] Apache Falcon Sharelib Hcatalog - Test Cluster ..... SUCCESS [  2.128 s]
[INFO] Apache Falcon Sharelib Oozie - Test Cluster ........ SUCCESS [  0.899 s]
[INFO] Apache Falcon Test Tools - Test Cluster ............ SUCCESS [  0.789 s]
[INFO] Apache Falcon Messaging ............................ SUCCESS [01:03 min]
[INFO] Apache Falcon LIfecycle Module ..................... SUCCESS [ 38.893 s]
[INFO] Apache Falcon Oozie Adaptor ........................ SUCCESS [04:16 min]
[INFO] Apache Falcon Scheduler ............................ SUCCESS [02:24 min]
[INFO] Apache Falcon Acquisition .......................... SUCCESS [ 51.916 s]
[INFO] Apache Falcon Distcp Replication ................... SUCCESS [ 35.040 s]
[INFO] Apache Falcon Retention ............................ SUCCESS [ 32.609 s]
[INFO] Apache Falcon Archival ............................. SUCCESS [  2.186 s]
[INFO] Apache Falcon Rerun ................................ SUCCESS [ 40.828 s]
[INFO] Apache Falcon Prism ................................ SUCCESS [08:04 min]
[INFO] Apache Falcon Unit ................................. SUCCESS [04:04 min]
[INFO] Apache Falcon Examples ............................. SUCCESS [ 35.224 s]
[INFO] Apache Falcon Web Application ...................... SUCCESS [26:32 min]
[INFO] Apache Falcon Documentation ........................ SUCCESS [ 23.240 s]
[INFO] Apache Falcon Distro ............................... SUCCESS [03:09 min]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------

Author: Raghavendra Singh <raghavendra.singh@inmobi.com>

Reviewers: @pallavi-rao

Closes #407 from RaghavendraSingh/master and squashes the following commits:

febf03f [Raghavendra Singh] FilteredCopyListing: fix checkstyle issue
0595e79 [Raghavendra Singh] FilteredCopyListing: shouldCopy override for new function signature
  • Loading branch information
Raghavendra Singh authored and pallavi-rao committed Apr 17, 2018
1 parent 1a5b4f6 commit 272e999c1bd5f288470adeeb6ff7d6af96ca9101
Showing 1 changed file with 10 additions and 0 deletions.
@@ -69,6 +69,16 @@ protected FilteredCopyListing(Configuration configuration, Credentials credentia

@Override
protected boolean shouldCopy(Path path, DistCpOptions options) {
return shouldCopy(path);
}

/**
* From hadoop 2.8.0 onwards, the function signature has been changed to not use DistCpOptions.
* hence added another function that works with the new implementation.
*
* Preserving previous implementation as well for backward compatibility.
*/
protected boolean shouldCopy(Path path) {
if (path.getName().equals(availabilityFlag)) {
return false;
}

0 comments on commit 272e999

Please sign in to comment.