----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34931/#review86513 -----------------------------------------------------------
Ship it! Ship It! - Raghav Gautam On June 3, 2015, 4:58 a.m., Paul Isaychuk wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/34931/ > ----------------------------------------------------------- > > (Updated June 3, 2015, 4:58 a.m.) > > > Review request for Falcon. > > > Bugs: FALCON-1246 > https://issues.apache.org/jira/browse/FALCON-1246 > > > Repository: falcon-git > > > Description > ------- > > Effective time is wrong parameter which is not being taken into account by > API. Removed it from update signature and related tests. > UpdateAtSpecificTimeTest#invalidCharFeed(),invalidCharProcess() were based on > enteting invalid effective time, disabling them. > > > Diffs > ----- > > > falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/helpers/entity/AbstractEntityHelper.java > 4dce3f3 > > falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/helpers/entity/ClusterEntityHelper.java > 4ea6728 > > falcon-regression/merlin/src/test/java/org/apache/falcon/regression/AuthorizationTest.java > 24af21f > > falcon-regression/merlin/src/test/java/org/apache/falcon/regression/FeedInstanceListingTest.java > 813300d > > falcon-regression/merlin/src/test/java/org/apache/falcon/regression/prism/EntityDryRunTest.java > 32d5e24 > > falcon-regression/merlin/src/test/java/org/apache/falcon/regression/prism/UpdateAtSpecificTimeTest.java > 9ff8016 > > Diff: https://reviews.apache.org/r/34931/diff/ > > > Testing > ------- > > FeedInstanceListingTest > > > Thanks, > > Paul Isaychuk > >
