Re: Review Request 61526: ATLAS-2029: Restrict entities, classifications can be applied to

2017-09-14 Thread Madhan Neethiraj
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61526/#review185423 --- Ship it! Ship It! - Madhan Neethiraj On Sept. 13, 2017, 1:38

Re: Review Request 61526: ATLAS-2029: Restrict entities, classifications can be applied to

2017-09-13 Thread David Radley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61526/ --- (Updated Sept. 13, 2017, 1:38 p.m.) Review request for atlas, Graham Wallis, Ma

Re: Review Request 61526: ATLAS-2029: Restrict entities, classifications can be applied to

2017-09-12 Thread Madhan Neethiraj
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61526/#review185236 --- intg/src/main/java/org/apache/atlas/type/AtlasClassificationType.

Re: Review Request 61526: ATLAS-2029: Restrict entities, classifications can be applied to

2017-09-12 Thread Apoorv Naik
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61526/#review185201 --- repository/src/main/java/org/apache/atlas/repository/store/graph/

Re: Review Request 61526: ATLAS-2029: Restrict entities, classifications can be applied to

2017-09-12 Thread David Radley
> On Sept. 8, 2017, 6:34 p.m., Apoorv Naik wrote: > > intg/src/main/java/org/apache/atlas/type/AtlasEntityType.java > > Lines 450 (patched) > > > > > > return type void for a get method ? > > > > Doesn't loo

Re: Review Request 61526: ATLAS-2029: Restrict entities, classifications can be applied to

2017-09-12 Thread David Radley
> On Sept. 1, 2017, 6:54 p.m., Sarath Subramanian wrote: > > intg/src/main/java/org/apache/atlas/type/AtlasClassificationType.java > > Lines 114 (patched) > > > > > > you can iterate through 's' List (computed in >

Re: Review Request 61526: ATLAS-2029: Restrict entities, classifications can be applied to

2017-09-12 Thread David Radley
> On Sept. 1, 2017, 8:18 p.m., Madhan Neethiraj wrote: > > intg/src/main/java/org/apache/atlas/AtlasErrorCode.java > > Lines 99 (patched) > > > > > > I think we should treat classification's empty entity-type to mean

Re: Review Request 61526: ATLAS-2029: Restrict entities, classifications can be applied to

2017-09-12 Thread David Radley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61526/ --- (Updated Sept. 12, 2017, 12:09 p.m.) Review request for atlas, Graham Wallis, M

Re: Review Request 61526: ATLAS-2029: Restrict entities, classifications can be applied to

2017-09-08 Thread Apoorv Naik
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61526/#review184993 --- intg/src/main/java/org/apache/atlas/type/AtlasEntityType.java Lin

Re: Review Request 61526: ATLAS-2029: Restrict entities, classifications can be applied to

2017-09-08 Thread David Radley
> On Sept. 1, 2017, 8:18 p.m., Madhan Neethiraj wrote: > > intg/src/main/java/org/apache/atlas/type/AtlasClassificationType.java > > Lines 394 (patched) > > > > > > With the changes suggested in resolveReferences(),

Re: Review Request 61526: ATLAS-2029: Restrict entities, classifications can be applied to

2017-09-08 Thread David Radley
> On Sept. 1, 2017, 6:54 p.m., Sarath Subramanian wrote: > > intg/src/main/java/org/apache/atlas/model/typedef/AtlasClassificationDef.java > > Lines 181 (patched) > > > > > > NPE here when 's' is null. Consider refac

Re: Review Request 61526: ATLAS-2029: Restrict entities, classifications can be applied to

2017-09-01 Thread Madhan Neethiraj
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61526/#review184373 --- intg/src/main/java/org/apache/atlas/AtlasErrorCode.java Lines 99

Re: Review Request 61526: ATLAS-2029: Restrict entities, classifications can be applied to

2017-09-01 Thread Sarath Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61526/#review184375 --- intg/src/main/java/org/apache/atlas/AtlasErrorCode.java Lines 116

Re: Review Request 61526: ATLAS-2029: Restrict entities, classifications can be applied to

2017-09-01 Thread David Radley
> On Aug. 20, 2017, 6:56 p.m., Madhan Neethiraj wrote: > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1.java > > Lines 742 (patched) > > > > > > Ensure that references are re

Re: Review Request 61526: ATLAS-2029: Restrict entities, classifications can be applied to

2017-09-01 Thread David Radley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61526/ --- (Updated Sept. 1, 2017, 9:50 a.m.) Review request for atlas, Graham Wallis, Mad

Re: Review Request 61526: ATLAS-2029: Restrict entities, classifications can be applied to

2017-08-21 Thread David Radley
> On Aug. 20, 2017, 6:56 p.m., Madhan Neethiraj wrote: > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1.java > > Lines 742 (patched) > > > > > > Ensure that references are re

Re: Review Request 61526: ATLAS-2029: Restrict entities, classifications can be applied to

2017-08-21 Thread David Radley
> On Aug. 20, 2017, 6:56 p.m., Madhan Neethiraj wrote: > > intg/src/main/java/org/apache/atlas/type/AtlasClassificationType.java > > Lines 54 (patched) > > > > > > >> we need to store the entityTypes specified in our

Re: Review Request 61526: ATLAS-2029: Restrict entities, classifications can be applied to

2017-08-21 Thread Madhan Neethiraj
> On Aug. 20, 2017, 6:56 p.m., Madhan Neethiraj wrote: > > intg/src/main/java/org/apache/atlas/type/AtlasClassificationType.java > > Lines 54 (patched) > > > > > > >> we need to store the entityTypes specified in our

Re: Review Request 61526: ATLAS-2029: Restrict entities, classifications can be applied to

2017-08-21 Thread David Radley
> On Aug. 20, 2017, 6:56 p.m., Madhan Neethiraj wrote: > > intg/src/main/java/org/apache/atlas/type/AtlasClassificationType.java > > Lines 386 (patched) > > > > > > blocks between #387 and #392 can be folded into a t

Re: Review Request 61526: ATLAS-2029: Restrict entities, classifications can be applied to

2017-08-21 Thread David Radley
> On Aug. 20, 2017, 6:56 p.m., Madhan Neethiraj wrote: > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1.java > > Lines 742 (patched) > > > > > > Ensure that references are re

Re: Review Request 61526: ATLAS-2029: Restrict entities, classifications can be applied to

2017-08-20 Thread David Radley
> On Aug. 20, 2017, 6:56 p.m., Madhan Neethiraj wrote: > > intg/src/main/java/org/apache/atlas/type/AtlasClassificationType.java > > Lines 54 (patched) > > > > > > >> we need to store the entityTypes specified in our

Re: Review Request 61526: ATLAS-2029: Restrict entities, classifications can be applied to

2017-08-20 Thread Madhan Neethiraj
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61526/#review183296 --- intg/src/main/java/org/apache/atlas/type/AtlasClassificationType.

Re: Review Request 61526: ATLAS-2029: Restrict entities, classifications can be applied to

2017-08-20 Thread David Radley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61526/ --- (Updated Aug. 20, 2017, 11:34 a.m.) Review request for atlas, Graham Wallis, Ma

Re: Review Request 61526: ATLAS-2029: Restrict entities, classifications can be applied to

2017-08-20 Thread David Radley
> On Aug. 17, 2017, 5:54 p.m., Madhan Neethiraj wrote: > > intg/src/main/java/org/apache/atlas/model/typedef/AtlasClassificationDef.java > > Lines 176 (patched) > > > > > > Classes in 'org.apache.atlas.model' are mea

Re: Review Request 61526: ATLAS-2029: Restrict entities, classifications can be applied to

2017-08-17 Thread Madhan Neethiraj
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61526/#review183142 --- intg/src/main/java/org/apache/atlas/model/typedef/AtlasClassifica

Review Request 61526: ATLAS-2029: Restrict entities, classifications can be applied to

2017-08-11 Thread David Radley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61526/ --- Review request for atlas. Repository: atlas Description --- ATLAS-2029: