Re: Review Request 61557: AMBARI-21701. Add check for import from relocated packages

2017-08-10 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61557/#review182593 --- Ship it! Ship It! - Sebastian Toader On Aug. 10, 2017, 3:47

Re: Review Request 61557: AMBARI-21701. Add check for import from relocated packages

2017-08-10 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61557/#review182591 --- Ship it! Ship It! - Jonathan Hurley On Aug. 10, 2017, 9:47 a

Re: Review Request 61557: AMBARI-21701. Add check for import from relocated packages

2017-08-10 Thread Attila Doroszlai
> On Aug. 10, 2017, 4:17 p.m., Jonathan Hurley wrote: > > ambari-server/checkstyle.xml > > Lines 27 (patched) > > > > > > Are there legit cases where sun.* is needed (such as with LDAP or SSL > > stuff?) Not current

Re: Review Request 61557: AMBARI-21701. Add check for import from relocated packages

2017-08-10 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61557/#review182586 --- ambari-server/checkstyle.xml Lines 27 (patched)