----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26751/#review56723 -----------------------------------------------------------
falcon-regression/merlin/src/main/java/org/apache/falcon/regression/testHelper/BaseTestClass.java <https://reviews.apache.org/r/26751/#comment97112> This variable is needed only for specific test. Why are we adding fields to BaseTestClass ? - Raghav Gautam On Oct. 15, 2014, 8:05 a.m., Ruslan Ostafiychuk wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/26751/ > ----------------------------------------------------------- > > (Updated Oct. 15, 2014, 8:05 a.m.) > > > Review request for Falcon. > > > Bugs: FALCON-808 > https://issues.apache.org/jira/browse/FALCON-808 > > > Repository: falcon-git > > > Description > ------- > > Check replication to *WASB* and other external file systems > > > Diffs > ----- > > > falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/enumsAndConstants/MerlinConstants.java > e877803 > > falcon-regression/merlin/src/main/java/org/apache/falcon/regression/testHelper/BaseTestClass.java > c16c8f4 > > falcon-regression/merlin/src/test/java/org/apache/falcon/regression/ExternalFSTest.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/26751/diff/ > > > Testing > ------- > > invalidCredentialsExtFS works fine > replicateToExternalFS is not properly tested > > > Thanks, > > Ruslan Ostafiychuk > >
