You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@atlas.apache.org by Suma Shivaprasad <su...@gmail.com> on 2016/08/03 17:55:41 UTC

Review Request 50760: ATLAS-1060 enable composite indexes with typeName and superTypeNames for selected attributes

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50760/
-----------------------------------------------------------

Review request for atlas and Shwetha GS.


Bugs: ATLAS-1060
    https://issues.apache.org/jira/browse/ATLAS-1060


Repository: atlas


Description
-------

The composite index is enabled for selected attributes like name, owner and Data model specific ones like hive_column.type , user etc


Diffs
-----

  addons/falcon-bridge/src/main/java/org/apache/atlas/falcon/model/FalconDataModelGenerator.java 3250a23 
  addons/hdfs-model/src/main/scala/org/apache/atlas/fs/model/FSDataModel.scala 4941a5f 
  addons/hive-bridge/src/main/java/org/apache/atlas/hive/model/HiveDataModelGenerator.java a3d97eb 
  addons/hive-bridge/src/test/java/org/apache/atlas/hive/hook/HiveHookIT.java 573f7f5 
  addons/sqoop-bridge/src/main/java/org/apache/atlas/sqoop/model/SqoopDataModelGenerator.java 0ea4d7f 
  addons/storm-bridge/src/main/scala/org/apache/atlas/storm/model/StormDataModel.scala 005f441 
  repository/src/main/java/org/apache/atlas/repository/graph/GraphBackedSearchIndexer.java c77004d 
  repository/src/main/java/org/apache/atlas/services/DefaultMetadataService.java e6b75f4 
  repository/src/test/java/org/apache/atlas/discovery/GraphBackedDiscoveryServiceTest.java df3fe87 
  repository/src/test/java/org/apache/atlas/repository/graph/GraphBackedSearchIndexerTest.java 1fa0619 
  typesystem/src/main/java/org/apache/atlas/typesystem/types/AttributeDefinition.java 29c3450 
  typesystem/src/main/java/org/apache/atlas/typesystem/types/AttributeInfo.java 9cb0d0d 
  typesystem/src/main/java/org/apache/atlas/typesystem/types/utils/TypesUtil.java ef8448d 
  typesystem/src/main/scala/org/apache/atlas/typesystem/builders/TypesBuilder.scala f18151a 
  typesystem/src/main/scala/org/apache/atlas/typesystem/json/TypesSerialization.scala 5618938 

Diff: https://reviews.apache.org/r/50760/diff/


Testing
-------


Thanks,

Suma Shivaprasad


Re: Review Request 50760: ATLAS-1060 enable composite indexes with typeName and superTypeNames for selected attributes

Posted by Suma Shivaprasad <su...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50760/
-----------------------------------------------------------

(Updated Aug. 5, 2016, 4:21 a.m.)


Review request for atlas and Shwetha GS.


Changes
-------

removed indexes for some attributes in HDFS Data model


Bugs: ATLAS-1060
    https://issues.apache.org/jira/browse/ATLAS-1060


Repository: atlas


Description
-------

The composite index is enabled for selected attributes like name, owner and Data model specific ones like hive_column.type , user etc


Diffs (updated)
-----

  addons/falcon-bridge/src/main/java/org/apache/atlas/falcon/model/FalconDataModelGenerator.java 3250a23 
  addons/hdfs-model/src/main/scala/org/apache/atlas/fs/model/FSDataModel.scala 4941a5f 
  addons/hive-bridge/src/main/java/org/apache/atlas/hive/model/HiveDataModelGenerator.java a3d97eb 
  addons/hive-bridge/src/test/java/org/apache/atlas/hive/bridge/HiveMetastoreBridgeIT.java 7e658a7 
  addons/hive-bridge/src/test/java/org/apache/atlas/hive/hook/HiveHookIT.java e61e916 
  addons/sqoop-bridge/src/main/java/org/apache/atlas/sqoop/model/SqoopDataModelGenerator.java 0ea4d7f 
  repository/src/main/java/org/apache/atlas/repository/graph/GraphBackedSearchIndexer.java c77004d 
  repository/src/main/java/org/apache/atlas/services/DefaultMetadataService.java e6b75f4 
  repository/src/test/java/org/apache/atlas/discovery/GraphBackedDiscoveryServiceTest.java df3fe87 
  repository/src/test/java/org/apache/atlas/repository/graph/GraphBackedSearchIndexerTest.java 1fa0619 
  repository/src/test/scala/org/apache/atlas/query/GremlinTest.scala 8fbcdbc 
  repository/src/test/scala/org/apache/atlas/query/LineageQueryTest.scala 021a8d2 
  typesystem/src/main/java/org/apache/atlas/typesystem/types/AttributeDefinition.java 29c3450 

Diff: https://reviews.apache.org/r/50760/diff/


Testing
-------


Thanks,

Suma Shivaprasad


Re: Review Request 50760: ATLAS-1060 enable composite indexes with typeName and superTypeNames for selected attributes

Posted by Suma Shivaprasad <su...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50760/
-----------------------------------------------------------

(Updated Aug. 4, 2016, 9:20 p.m.)


Review request for atlas and Shwetha GS.


Changes
-------

Making isIndexable false by default and turning it on selectively for attributes which need a composite index


Bugs: ATLAS-1060
    https://issues.apache.org/jira/browse/ATLAS-1060


Repository: atlas


Description
-------

The composite index is enabled for selected attributes like name, owner and Data model specific ones like hive_column.type , user etc


Diffs (updated)
-----

  addons/falcon-bridge/src/main/java/org/apache/atlas/falcon/model/FalconDataModelGenerator.java 3250a23 
  addons/hive-bridge/src/main/java/org/apache/atlas/hive/model/HiveDataModelGenerator.java a3d97eb 
  addons/hive-bridge/src/test/java/org/apache/atlas/hive/bridge/HiveMetastoreBridgeIT.java 7e658a7 
  addons/hive-bridge/src/test/java/org/apache/atlas/hive/hook/HiveHookIT.java e61e916 
  addons/sqoop-bridge/src/main/java/org/apache/atlas/sqoop/model/SqoopDataModelGenerator.java 0ea4d7f 
  repository/src/main/java/org/apache/atlas/repository/graph/GraphBackedSearchIndexer.java c77004d 
  repository/src/main/java/org/apache/atlas/services/DefaultMetadataService.java e6b75f4 
  repository/src/test/java/org/apache/atlas/discovery/GraphBackedDiscoveryServiceTest.java df3fe87 
  repository/src/test/java/org/apache/atlas/repository/graph/GraphBackedSearchIndexerTest.java 1fa0619 
  repository/src/test/scala/org/apache/atlas/query/GremlinTest.scala 8fbcdbc 
  repository/src/test/scala/org/apache/atlas/query/LineageQueryTest.scala 021a8d2 
  typesystem/src/main/java/org/apache/atlas/typesystem/types/AttributeDefinition.java 29c3450 

Diff: https://reviews.apache.org/r/50760/diff/


Testing
-------


Thanks,

Suma Shivaprasad


Re: Review Request 50760: ATLAS-1060 enable composite indexes with typeName and superTypeNames for selected attributes

Posted by Suma Shivaprasad <su...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50760/
-----------------------------------------------------------

(Updated Aug. 3, 2016, 6:09 p.m.)


Review request for atlas and Shwetha GS.


Changes
-------

Renamed flags in createIndexes


Bugs: ATLAS-1060
    https://issues.apache.org/jira/browse/ATLAS-1060


Repository: atlas


Description
-------

The composite index is enabled for selected attributes like name, owner and Data model specific ones like hive_column.type , user etc


Diffs (updated)
-----

  addons/falcon-bridge/src/main/java/org/apache/atlas/falcon/model/FalconDataModelGenerator.java 3250a23 
  addons/hdfs-model/src/main/scala/org/apache/atlas/fs/model/FSDataModel.scala 4941a5f 
  addons/hive-bridge/src/main/java/org/apache/atlas/hive/model/HiveDataModelGenerator.java a3d97eb 
  addons/hive-bridge/src/test/java/org/apache/atlas/hive/hook/HiveHookIT.java e61e916 
  addons/sqoop-bridge/src/main/java/org/apache/atlas/sqoop/model/SqoopDataModelGenerator.java 0ea4d7f 
  addons/storm-bridge/src/main/scala/org/apache/atlas/storm/model/StormDataModel.scala 005f441 
  repository/src/main/java/org/apache/atlas/repository/graph/GraphBackedSearchIndexer.java c77004d 
  repository/src/main/java/org/apache/atlas/services/DefaultMetadataService.java e6b75f4 
  repository/src/test/java/org/apache/atlas/discovery/GraphBackedDiscoveryServiceTest.java df3fe87 
  repository/src/test/java/org/apache/atlas/repository/graph/GraphBackedSearchIndexerTest.java 1fa0619 
  typesystem/src/main/java/org/apache/atlas/typesystem/types/AttributeDefinition.java 29c3450 
  typesystem/src/main/java/org/apache/atlas/typesystem/types/AttributeInfo.java 9cb0d0d 
  typesystem/src/main/java/org/apache/atlas/typesystem/types/utils/TypesUtil.java ef8448d 
  typesystem/src/main/scala/org/apache/atlas/typesystem/builders/TypesBuilder.scala f18151a 
  typesystem/src/main/scala/org/apache/atlas/typesystem/json/TypesSerialization.scala 5618938 

Diff: https://reviews.apache.org/r/50760/diff/


Testing
-------


Thanks,

Suma Shivaprasad