----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59264/#review174961 -----------------------------------------------------------
Looks good overall, can you also cover the test clasess for consistency ? - Apoorv Naik On May 14, 2017, 4:58 p.m., Jan Hentschel wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/59264/ > ----------------------------------------------------------- > > (Updated May 14, 2017, 4:58 p.m.) > > > Review request for atlas. > > > Bugs: ATLAS-1787 > https://issues.apache.org/jira/browse/ATLAS-1787 > > > Repository: atlas > > > Description > ------- > > Replaced the remaining places where explicit types are used. > > > Diffs > ----- > > > graphdb/titan1/src/main/java/org/apache/atlas/repository/graphdb/titan1/Titan1Graph.java > 6a61075 > > graphdb/titan1/src/main/java/org/apache/atlas/repository/graphdb/titan1/Titan1GraphIndex.java > 5ec1180 > > graphdb/titan1/src/main/java/org/apache/atlas/repository/graphdb/titan1/Titan1Vertex.java > 1670e92 > > graphdb/titan1/src/main/java/org/apache/atlas/repository/graphdb/titan1/graphson/AtlasElementPropertyConfig.java > df67a83 > > graphdb/titan1/src/main/java/org/apache/atlas/repository/graphdb/titan1/graphson/AtlasGraphSONUtility.java > 26abe2e > > graphdb/titan1/src/main/java/org/apache/atlas/repository/graphdb/titan1/serializer/StringListSerializer.java > eb99fae > > graphdb/titan1/src/test/java/org/apache/atlas/repository/graphdb/titan1/GraphQueryTest.java > 45fbbea > > graphdb/titan1/src/test/java/org/apache/atlas/repository/graphdb/titan1/Titan1DatabaseTest.java > ab2c8c9 > intg/src/test/java/org/apache/atlas/type/TestAtlasObjectIdType.java fb4a183 > > repository/src/main/java/org/apache/atlas/gremlin/optimizer/ExpandOrsOptimization.java > a48a007 > > repository/src/main/java/org/apache/atlas/repository/converters/TypeConverterUtil.java > 7902100 > > repository/src/main/java/org/apache/atlas/repository/graph/GraphBackedMetadataRepository.java > d1e8cb7 > repository/src/main/java/org/apache/atlas/repository/graph/GraphHelper.java > ca7fad0 > > repository/src/main/java/org/apache/atlas/repository/typestore/GraphBackedTypeStore.java > ac13586 > > repository/src/main/java/org/apache/atlas/repository/typestore/TypeVertexFinder.java > 8b38152 > > repository/src/main/java/org/apache/atlas/util/AtlasRepositoryConfiguration.java > a4a596a > > repository/src/test/java/org/apache/atlas/discovery/GraphBackedDiscoveryServiceTest.java > 18573fc > > repository/src/test/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1Test.java > 1dd7276 > > webapp/src/main/java/org/apache/atlas/web/security/AtlasAbstractAuthenticationProvider.java > e55d2cf > > > Diff: https://reviews.apache.org/r/59264/diff/1/ > > > Testing > ------- > > > Thanks, > > Jan Hentschel > >
