----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38546/#review99760 -----------------------------------------------------------
webapp/pom.xml (line 88) <https://reviews.apache.org/r/38546/#comment156747> Why is this change required? Also why do we have two dependency instances for falcon-common? - Sowmya Ramesh On Sept. 21, 2015, 7:28 a.m., Narayan Periwal wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/38546/ > ----------------------------------------------------------- > > (Updated Sept. 21, 2015, 7:28 a.m.) > > > Review request for Falcon. > > > Repository: falcon-git > > > Description > ------- > > As part of FALCON-1027 a few tests were introduced that hardcode "testuser" > as the user. Use FalconTestUtil.TEST_USER_2 instead. > > > Diffs > ----- > > common/src/test/java/org/apache/falcon/security/SecurityUtilTest.java > 76d9f45 > > prism/src/test/java/org/apache/falcon/security/FalconAuthenticationFilterTest.java > 5627e68 > webapp/pom.xml 828f7f5 > webapp/src/test/java/org/apache/falcon/cli/FalconCLIIT.java 577b60d > webapp/src/test/java/org/apache/falcon/resource/EntityManagerJerseyIT.java > 6d375c0 > > webapp/src/test/java/org/apache/falcon/resource/MetadataResourceJerseyIT.java > 749dfa5 > > Diff: https://reviews.apache.org/r/38546/diff/ > > > Testing > ------- > > Done. > > > Thanks, > > Narayan Periwal > >
