vladhlinsky commented on issue #89: ATLAS-3646 Create new 
'spark_ml_model_dataset','spark_ml_pipeline_dataset' relationship defs
URL: https://github.com/apache/atlas/pull/89#issuecomment-599593065
 
 
   @sarathsubramanian sorry, i just realized that i tested proposed by you 
changes with previously existing relationship definitions and thus I missed 
that changing  "isContainer" => false leads to:
   ```
   org.apache.atlas.exception.AtlasBaseException: AGGREGATION relationshipDef 
spark_ml_model_dataset creation attempted without an end specifying isContainer
        at 
org.apache.atlas.type.AtlasRelationshipType.validateAtlasRelationshipDef(AtlasRelationshipType.java:309)
        at 
org.apache.atlas.type.AtlasRelationshipType.resolveReferences(AtlasRelationshipType.java:110)
        at 
org.apache.atlas.type.AtlasTypeRegistry$AtlasTransientTypeRegistry.resolveReferences(AtlasTypeRegistry.java:426)
        at 
org.apache.atlas.type.AtlasTypeRegistry$AtlasTransientTypeRegistry.addTypes(AtlasTypeRegistry.java:500)
        at 
org.apache.atlas.repository.store.graph.AtlasTypeDefGraphStore.tryTypeCreation(AtlasTypeDefGraphStore.java:1136)
        at 
org.apache.atlas.repository.store.graph.AtlasTypeDefGraphStore.createUpdateTypesDef(AtlasTypeDefGraphStore.java:410)
        at 
org.apache.atlas.repository.store.graph.AtlasTypeDefGraphStore$$FastClassBySpringCGLIB$$5226c80b.invoke(<generated>)
        at 
org.springframework.cglib.proxy.MethodProxy.invoke(MethodProxy.java:204)
   ```
   
   changes reverted back to "isContainer" => true. 

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to