----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/70913/#review216095 -----------------------------------------------------------
intg/src/main/java/org/apache/atlas/type/AtlasEntityType.java Lines 679 (patched) <https://reviews.apache.org/r/70913/#comment303078> Create 'private static final' field. intg/src/main/java/org/apache/atlas/type/AtlasEntityType.java Lines 680 (patched) <https://reviews.apache.org/r/70913/#comment303079> Abstract this in a separate method, say resolveDynamicAttribute(AtlasAttribute attr); - Ashutosh Mestry On June 22, 2019, 12:40 a.m., Merryle Wang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/70913/ > ----------------------------------------------------------- > > (Updated June 22, 2019, 12:40 a.m.) > > > Review request for atlas, Ashutosh Mestry, Aadarsh Jajodia, Sridhar K, Le Ma, > Madhan Neethiraj, and Sarath Subramanian. > > > Repository: atlas > > > Description > ------- > > ATLAS-3286: Populated dynamic attribute flags for AtlasAttributes > > > Diffs > ----- > > addons/models/1000-Hadoop/1030-hive_model.json > 8901aa4aa86fb2802a9e9b1e65c2ff8aad8855ad > intg/src/main/java/org/apache/atlas/type/AtlasEntityType.java > d9ae9e36773146fad652a1e28fc8822bae5c8557 > intg/src/main/java/org/apache/atlas/type/AtlasStructType.java > 254eee4c9d18ab87a8862ce479fc2274227c7c29 > intg/src/test/java/org/apache/atlas/type/TestAtlasEntityType.java > 3c53c02b48747515217c9327c98209a48ee84237 > > > Diff: https://reviews.apache.org/r/70913/diff/2/ > > > Testing > ------- > > Planning to add unit tests > > > Thanks, > > Merryle Wang > >
