Re: Review Request 60034: Support KNOX SSO Token based authentication on Ranger REST API calls

2017-06-13 Thread Ankita Sinha
> On June 13, 2017, 8:49 a.m., Colm O hEigeartaigh wrote: > > Please put curly brackets around the two one-line if statements: > > if(LOG.isDebugEnabled()) Thanks, updated with the suggested changes. - Ankita --- This is an

Re: Review Request 60034: Support KNOX SSO Token based authentication on Ranger REST API calls

2017-06-13 Thread Ankita Sinha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60034/ --- (Updated June 13, 2017, 9:55 a.m.) Review request for ranger, Don Bosco Durai,

Re: Review Request 60034: Support KNOX SSO Token based authentication on Ranger REST API calls

2017-06-13 Thread Colm O hEigeartaigh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60034/#review177715 --- Please put curly brackets around the two one-line if statements:

Review Request 60034: Support KNOX SSO Token based authentication on Ranger REST API calls

2017-06-13 Thread Ankita Sinha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60034/ --- Review request for ranger, Don Bosco Durai, Gautam Borad, Abhay Kulkarni,