Re: Review Request 48686: Tez View: Improve proxy logging

2016-06-23 Thread Sreenath Somarajapuram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48686/ --- (Updated June 24, 2016, 4:45 a.m.) Review request for Ambari, DIPAYAN BHOWMICK,

Re: Review Request 48686: Tez View: Improve proxy logging

2016-06-23 Thread Sreenath Somarajapuram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48686/ --- (Updated June 23, 2016, 9:36 p.m.) Review request for Ambari, DIPAYAN BHOWMICK,

Re: Review Request 48686: Tez View: Improve proxy logging

2016-06-23 Thread Hitesh Shah
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48686/#review139298 --- Ship it! Looks good apart from the minor typo to be addressed.

Re: Review Request 48686: Tez View: Improve proxy logging

2016-06-23 Thread Sreenath Somarajapuram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48686/ --- (Updated June 23, 2016, 9:14 p.m.) Review request for Ambari, DIPAYAN BHOWMICK,

Re: Review Request 48686: Tez View: Improve proxy logging

2016-06-23 Thread Sreenath Somarajapuram
> On June 23, 2016, 8:24 p.m., Hitesh Shah wrote: > > contrib/views/tez/src/main/java/org/apache/ambari/view/tez/utils/ProxyHelper.java, > > line 64 > > > > > > Is there a debug message which logs which url was being

Re: Review Request 48686: Tez View: Improve proxy logging

2016-06-23 Thread Hitesh Shah
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48686/#review139293 --- contrib/views/tez/src/main/java/org/apache/ambari/view/tez/except

Re: Review Request 48686: Tez View: Improve proxy logging

2016-06-23 Thread Hitesh Shah
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48686/#review139292 --- contrib/views/tez/src/main/java/org/apache/ambari/view/tez/ViewCo

Re: Review Request 48686: Tez View: Improve proxy logging

2016-06-23 Thread Sreenath Somarajapuram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48686/ --- (Updated June 23, 2016, 8:04 p.m.) Review request for Ambari, DIPAYAN BHOWMICK

Re: Review Request 48686: Tez View: Improve proxy logging

2016-06-20 Thread Sreenath Somarajapuram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48686/ --- (Updated June 20, 2016, 11:21 a.m.) Review request for Ambari, DIPAYAN BHOWMICK

Re: Review Request 48686: Tez View: Improve proxy logging

2016-06-16 Thread Sreenath Somarajapuram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48686/ --- (Updated June 16, 2016, 5:17 p.m.) Review request for Ambari, DIPAYAN BHOWMICK

Re: Review Request 48686: Tez View: Improve proxy logging

2016-06-14 Thread Hitesh Shah
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48686/#review137513 --- Does logging need to change in any of other classes of the tez vie

Review Request 48686: Tez View: Improve proxy logging

2016-06-14 Thread Sreenath Somarajapuram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48686/ --- Review request for Ambari, DIPAYAN BHOWMICK and Pallav Kulshreshtha. Bugs: AMBA