----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67082/#review202898 -----------------------------------------------------------
graphdb/janus/src/main/java/org/apache/atlas/repository/graphdb/janus/AtlasJanusGraphDatabase.java Lines 50 (patched) <https://reviews.apache.org/r/67082/#comment284934> this class doesn't seem to be used here. Please review and remove this import. repository/src/main/java/org/apache/atlas/repository/graph/GraphBackedSearchIndexer.java Lines 334 (patched) <https://reviews.apache.org/r/67082/#comment284935> Property-key is not needed if attribute is array<object> or array<struct> - right? repository/src/main/java/org/apache/atlas/repository/graph/GraphBackedSearchIndexer.java Line 330 (original), 339 (patched) <https://reviews.apache.org/r/67082/#comment284936> Property-key is not needed if attribute is map<primitive,object> or map<primitive,struct> - right? repository/src/main/java/org/apache/atlas/repository/graph/GraphBackedSearchIndexer.java Line 700 (original), 725 (patched) <https://reviews.apache.org/r/67082/#comment284937> cleanupIndex() needs to be callec for primitive array & maps type attribuges as well - to delete the propertyKey (line #742 below). repository/src/main/java/org/apache/atlas/repository/graph/GraphHelper.java Line 1581 (original), 1581 (patched) <https://reviews.apache.org/r/67082/#comment284938> Instead of calling GraphHelper.encodePropertyKey() on every read, consider using the encoded name in 'attribute'. Same for line #1571 above. - Madhan Neethiraj On May 11, 2018, 6:49 a.m., Sarath Subramanian wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/67082/ > ----------------------------------------------------------- > > (Updated May 11, 2018, 6:49 a.m.) > > > Review request for atlas, Apoorv Naik, Ashutosh Mestry, and Madhan Neethiraj. > > > Bugs: ATLAS-2672 > https://issues.apache.org/jira/browse/ATLAS-2672 > > > Repository: atlas > > > Description > ------- > > Currently primitive map type in atlas is stored in vertex as: > > typeName.mapAttr = [key1, key2, key3] > typeName.mapAttr.key1 = value1 > typeName.mapAttr.key2 = value2 > typeName.mapAttr.key3 = value3 > Since JanusGraph supports Map datatype, we can store map value within a > single vertex property. > > Also we need to create edge label and property key for array and map > primitive types. > > > Diffs > ----- > > > graphdb/janus/src/main/java/org/apache/atlas/repository/graphdb/janus/AtlasJanusGraphDatabase.java > 3636ca65a > intg/src/test/java/org/apache/atlas/TestUtilsV2.java f48ccef05 > > repository/src/main/java/org/apache/atlas/repository/graph/GraphBackedSearchIndexer.java > f75f7862d > repository/src/main/java/org/apache/atlas/repository/graph/GraphHelper.java > 7490a15b1 > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/DeleteHandlerV1.java > d285e6964 > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityGraphMapper.java > cd00639de > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityGraphRetriever.java > 183a2f6c0 > > repository/src/test/java/org/apache/atlas/repository/store/graph/v1/AtlasComplexAttributesTest.java > e4947282a > > repository/src/test/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityTestBase.java > 56dc5bdaf > > > Diff: https://reviews.apache.org/r/67082/diff/1/ > > > Testing > ------- > > https://builds.apache.org/view/A/view/Atlas/job/PreCommit-ATLAS-Build-Test/385/console > > > Thanks, > > Sarath Subramanian > >
