> On Aug. 5, 2019, 10:01 p.m., Ashutosh Mestry wrote: > > intg/src/main/java/org/apache/atlas/type/AtlasEntityType.java > > Lines 1040 (patched) > > <https://reviews.apache.org/r/71127/diff/4/?file=2159837#file2159837line1040> > > > > What is the penalty imposed on current entity creation flow with this > > change?
All of this runs upon startup of Atlas since it's just reordering a list within each entity type. So the entity creation flow is untouched. - Merryle ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/71127/#review217086 ----------------------------------------------------------- On Aug. 5, 2019, 10:19 p.m., Merryle Wang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/71127/ > ----------------------------------------------------------- > > (Updated Aug. 5, 2019, 10:19 p.m.) > > > Review request for atlas, Ashutosh Mestry, Aadarsh Jajodia, Sridhar K, Le Ma, > Madhan Neethiraj, and Sarath Subramanian. > > > Repository: atlas > > > Description > ------- > > ATLAS-3343: Ordering of dynAttr evaluation > > > Diffs > ----- > > intg/src/main/java/org/apache/atlas/type/AtlasEntityType.java > 23eaa0a2e88fd348d2347314170726ebb5cb4393 > intg/src/test/java/org/apache/atlas/type/TestAtlasEntityType.java > c114bdf7888519cdd396fc8501bbcd92b00ef0bb > > > Diff: https://reviews.apache.org/r/71127/diff/5/ > > > Testing > ------- > > Created dummy dynamic attributes to test the correct ordering. > > > Thanks, > > Merryle Wang > >
