> On Sept. 23, 2016, 6:19 a.m., Shwetha GS wrote: > > In case the attribute is added again, the data will be wrong. We can't > > discount that as an edge case, as it causes data inconsistency. Follow up > > jiras will hardly be taken up. Its fine even if we decide that we don't > > allow adding the attribute again(the deleted attribute). But we should > > address the issue completely > > > > Add UTs in all your patches. Just an end to end test for this patch now > > will not ensure that this functionality won't be broken in further commits
One sugestion to resolve this case would be 1. Add a isDeleted flag to the attribute. It may be useful to know that this atribute was deleted and in case of upgrade/rollbacks , we could potentially use this flag to resurface it back again and roll back the type update. 2. Rename the existing property key - http://s3.thinkaurelius.com/docs/titan/0.5.1/schema.html#_changing_schema_elements. Not sure how much time this operation would take. If its an expensive operation, we would need to run this as an asynchronous thread. - Suma ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52166/#review150154 ----------------------------------------------------------- On Sept. 22, 2016, 6:22 p.m., Sarath Kumar Subramanian wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52166/ > ----------------------------------------------------------- > > (Updated Sept. 22, 2016, 6:22 p.m.) > > > Review request for atlas, Madhan Neethiraj, Shwetha GS, and Suma Shivaprasad. > > > Bugs: ATLAS-1175 > https://issues.apache.org/jira/browse/ATLAS-1175 > > > Repository: atlas > > > Description > ------- > > we restrict optional attributes to be removed from the type system. This > needs to be allowed > > > Diffs > ----- > > typesystem/src/main/java/org/apache/atlas/typesystem/types/TypeUtils.java > f5c2ce9 > > Diff: https://reviews.apache.org/r/52166/diff/ > > > Testing > ------- > > tested using REST calls to delete optional and required attributes and only > optional attributes were removed. > > > Thanks, > > Sarath Kumar Subramanian > >
