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

Review request for atlas, Jayendra Parab, Mandar Ambawane, Pinal Shah, and 
Sheetal Shah.


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


Repository: atlas


Description
-------

Problem:
--------
When tag is created with "description" as name it was throwing 
ClassCastException.

Solution:
--------
The above issue was seen in all typeDef creation so, we have modified default 
property keys format by adding "__" before some typeDef attributes.

For eg: 
Earlier the name of the typeDef was stored as:
"__type.name"

Now, the name of the typeDef was stored as:
"__type.__name"

These changes was done as when any typeDef vertex was created it was also 
adding another propertyKey with the format "__type.typeDef_name" having "List" 
as datatype and was storing attributes name of the respected typeDef. So, while 
creating vertex of description tag, when new property key i.e 
"__type.typeDef_name" gets added to the vertex it override its value to the 
existing default property "__type.description" and while fetching the 
atrributes names of the typeDef it was throwing this exception.

Note: All typeDefs vertex has "__type" prefix for some default property keys.


Diffs
-----

  common/src/main/java/org/apache/atlas/repository/Constants.java 2134133f5 
  
repository/src/main/java/org/apache/atlas/repository/patches/AddMandatoryAttributesPatch.java
 310251698 
  
repository/src/main/java/org/apache/atlas/repository/patches/AtlasPatchManager.java
 c41b75dc3 
  
repository/src/main/java/org/apache/atlas/repository/patches/ClassificationTextPatch.java
 a3415c003 
  
repository/src/main/java/org/apache/atlas/repository/patches/ConcurrentPatchProcessor.java
 e5dcb2ef6 
  
repository/src/main/java/org/apache/atlas/repository/patches/ProcessImpalaNamePatch.java
 7eb918481 
  
repository/src/main/java/org/apache/atlas/repository/patches/ProcessNamePatch.java
 9889d4680 
  
repository/src/main/java/org/apache/atlas/repository/patches/SuperTypesUpdatePatch.java
 a77ca70cb 
  
repository/src/main/java/org/apache/atlas/repository/patches/TypeDefAttributePatch.java
 PRE-CREATION 
  
repository/src/main/java/org/apache/atlas/repository/patches/UniqueAttributePatch.java
 c30bd2596 


Diff: https://reviews.apache.org/r/75336/diff/1/


Testing
-------

Manual testing done.


Thanks,

Priyanshi Shah

Reply via email to