----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/70265/#review213897 -----------------------------------------------------------
intg/pom.xml Line 105 (original) <https://reviews.apache.org/r/70265/#comment300012> Avoid adding 'guava' dependency in 'intg' project, as this will push 'guava' to all consumers of 'intg' library - including Ranger and plugins. It seems only use from this library is StopWatch. Please find an alternate (like AtlasPerfTracer) and remove dependency on 'guava'. - madhan On March 21, 2019, 11:36 p.m., Ashutosh Mestry wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/70265/ > ----------------------------------------------------------- > > (Updated March 21, 2019, 11:36 p.m.) > > > Review request for atlas, Madhan Neethiraj, Nixon Rodrigues, and Sarath > Subramanian. > > > Bugs: ATLAS-3089 > https://issues.apache.org/jira/browse/ATLAS-3089 > > > Repository: atlas > > > Description > ------- > > Move refactor applied. > > > Diffs > ----- > > > graphdb/janus/src/main/java/org/apache/atlas/repository/graphdb/janus/migration/JsonNodeProcessManager.java > e2f418e44 > > graphdb/janus/src/main/java/org/apache/atlas/repository/graphdb/janus/migration/PostProcessManager.java > 7046f8c80 > > graphdb/janus/src/main/java/org/apache/atlas/repository/graphdb/janus/migration/pc/WorkItemBuilder.java > 257b63397 > > graphdb/janus/src/main/java/org/apache/atlas/repository/graphdb/janus/migration/pc/WorkItemConsumer.java > 39b50de7d > > graphdb/janus/src/main/java/org/apache/atlas/repository/graphdb/janus/migration/pc/WorkItemManager.java > d50ada435 > > graphdb/janus/src/test/java/org/apache/atlas/repository/graphdb/janus/migration/WorkItemConsumerTest.java > 4cd4c2c15 > > graphdb/janus/src/test/java/org/apache/atlas/repository/graphdb/janus/migration/WorkItemManagerTest.java > 25488bd3f > intg/pom.xml 7582549eb > > > Diff: https://reviews.apache.org/r/70265/diff/1/ > > > Testing > ------- > > Unit tests executed. > > > Thanks, > > Ashutosh Mestry > >