Re: Review Request 55813: Porting performance and stability changes made in 0.7 branch into master

2017-01-25 Thread Madhan Neethiraj
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55813/#review162981 --- Ship it! The fix looks good! With this patch, following DSL

Re: Review Request 55813: Porting performance and stability changes made in 0.7 branch into master

2017-01-25 Thread Sarath Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55813/ --- (Updated Jan. 25, 2017, 1:32 a.m.) Review request for atlas, Madhan Neethiraj

Re: Review Request 55813: Porting performance and stability changes made in 0.7 branch into master

2017-01-24 Thread Sarath Subramanian
> On Jan. 23, 2017, 8:36 p.m., Suma Shivaprasad wrote: > > repository/src/main/java/org/apache/atlas/discovery/DataSetLineageService.java, > > line 99 > > > > > > This shoudnt have superTypeNames check directly in

Re: Review Request 55813: Porting performance and stability changes made in 0.7 branch into master

2017-01-24 Thread Sarath Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55813/ --- (Updated Jan. 24, 2017, 12:08 a.m.) Review request for atlas, Madhan Neethiraj

Re: Review Request 55813: Porting performance and stability changes made in 0.7 branch into master

2017-01-23 Thread Suma Shivaprasad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55813/#review162765 ---

Re: Review Request 55813: Porting performance and stability changes made in 0.7 branch into master

2017-01-23 Thread Sarath Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55813/ --- (Updated Jan. 23, 2017, 3:17 p.m.) Review request for atlas, Madhan Neethiraj

Re: Review Request 55813: Porting performance and stability changes made in 0.7 branch into master

2017-01-23 Thread Madhan Neethiraj
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55813/#review162725 --- Fix it, then Ship it!

Re: Review Request 55813: Porting performance and stability changes made in 0.7 branch into master

2017-01-23 Thread Sarath Subramanian
> On Jan. 20, 2017, 7:11 p.m., Suma Shivaprasad wrote: > > repository/src/main/scala/org/apache/atlas/query/GremlinEvaluator.scala, > > line 25 > > > > > > will also need to change code in Gremlin2ExpressionFactory

Re: Review Request 55813: Porting performance and stability changes made in 0.7 branch into master

2017-01-23 Thread Sarath Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55813/ --- (Updated Jan. 23, 2017, 2:09 p.m.) Review request for atlas, Madhan Neethiraj

Re: Review Request 55813: Porting performance and stability changes made in 0.7 branch into master

2017-01-20 Thread Suma Shivaprasad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55813/#review162536 ---

Review Request 55813: Porting performance and stability changes made in 0.7 branch into master

2017-01-20 Thread Sarath Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55813/ --- Review request for atlas, Madhan Neethiraj and Suma Shivaprasad. Bugs: