Re: Review Request 52257: Return system attributes in get entity definition

2016-11-18 Thread Suma Shivaprasad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52257/#review156288 --- Ship it! Ship It! - Suma Shivaprasad On Nov. 18, 2016, 7:15

Re: Review Request 52257: Return system attributes in get entity definition

2016-11-17 Thread Vimal Sharma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52257/ --- (Updated Nov. 18, 2016, 7:15 a.m.) Review request for atlas. Bugs: ATLAS-916

Re: Review Request 52257: Return system attributes in get entity definition

2016-11-16 Thread Vimal Sharma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52257/#review156180 ---

Re: Review Request 52257: Return system attributes in get entity definition

2016-11-16 Thread Shwetha GS
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52257/#review156176 ---

Re: Review Request 52257: Return system attributes in get entity definition

2016-11-16 Thread Vimal Sharma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52257/ --- (Updated Nov. 16, 2016, 1:33 p.m.) Review request for atlas. Changes ---

Re: Review Request 52257: Return system attributes in get entity definition

2016-11-15 Thread Shwetha GS
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52257/#review156026 --- Fix it, then Ship it!

Re: Review Request 52257: Return system attributes in get entity definition

2016-11-14 Thread Suma Shivaprasad
> On Nov. 14, 2016, 7:49 p.m., Suma Shivaprasad wrote: > > repository/src/main/java/org/apache/atlas/repository/graph/GraphBackedSearchIndexer.java, > > line 161 > > > > > > do we need composite for modification

Re: Review Request 52257: Return system attributes in get entity definition

2016-11-13 Thread Vimal Sharma
> On Oct. 24, 2016, 10:27 p.m., Suma Shivaprasad wrote: > > repository/src/main/java/org/apache/atlas/repository/graph/GraphBackedSearchIndexer.java, > > line 137 > > > > > > The Index should be of String type..not

Re: Review Request 52257: Return system attributes in get entity definition

2016-11-13 Thread Vimal Sharma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52257/ --- (Updated Nov. 14, 2016, 4:50 a.m.) Review request for atlas. Changes ---

Re: Review Request 52257: Return system attributes in get entity definition

2016-11-13 Thread Vimal Sharma
> On Oct. 24, 2016, 11:51 p.m., Suma Shivaprasad wrote: > > repository/src/test/java/org/apache/atlas/discovery/GraphBackedDiscoveryServiceTest.java, > > line 188 > > > > > > can you add test for querying by these

Re: Review Request 52257: Return system attributes in get entity definition

2016-10-24 Thread Suma Shivaprasad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52257/#review153763 ---

Re: Review Request 52257: Return system attributes in get entity definition

2016-10-24 Thread Suma Shivaprasad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52257/#review153762 ---

Re: Review Request 52257: Return system attributes in get entity definition

2016-10-24 Thread Suma Shivaprasad
> On Oct. 24, 2016, 8:10 p.m., Suma Shivaprasad wrote: > > Ship It! Pls fix below comments - Suma --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52257/#review153729

Re: Review Request 52257: Return system attributes in get entity definition

2016-10-24 Thread Suma Shivaprasad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52257/#review153749 ---

Re: Review Request 52257: Return system attributes in get entity definition

2016-10-24 Thread Suma Shivaprasad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52257/#review153729 --- Ship it! Ship It! - Suma Shivaprasad On Oct. 17, 2016,

Re: Review Request 52257: Return system attributes in get entity definition

2016-10-04 Thread Suma Shivaprasad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52257/#review151364 ---

Re: Review Request 52257: Return system attributes in get entity definition

2016-10-04 Thread Suma Shivaprasad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52257/#review151362 ---

Re: Review Request 52257: Return system attributes in get entity definition

2016-10-04 Thread Suma Shivaprasad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52257/#review151360 ---

Re: Review Request 52257: Return system attributes in get entity definition

2016-10-04 Thread Suma Shivaprasad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52257/#review151357 ---

Re: Review Request 52257: Return system attributes in get entity definition

2016-10-04 Thread Vimal Sharma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52257/ --- (Updated Oct. 4, 2016, 11:13 a.m.) Review request for atlas. Changes ---

Re: Review Request 52257: Return system attributes in get entity definition

2016-09-27 Thread Suma Shivaprasad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52257/#review150656 ---