Re: Review Request 63838: ambari-server setup should surface GPL software agreement

2017-11-16 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63838/#review191189 --- I have committed a follow up patch to fix review comment. As i

Re: Review Request 63838: ambari-server setup should surface GPL software agreement

2017-11-16 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63838/#review191162 --- Thanks for your reviews guys. - Andrew Onischuk On Nov. 16,

Re: Review Request 63838: ambari-server setup should surface GPL software agreement

2017-11-15 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63838/#review191077 --- Fix it, then Ship it!

Re: Review Request 63838: ambari-server setup should surface GPL software agreement

2017-11-15 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63838/#review191067 --- Fix it, then Ship it!

Re: Review Request 63838: ambari-server setup should surface GPL software agreement

2017-11-15 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63838/#review191064 --- Fix it, then Ship it! Overall, the patch looks fine, just a

Review Request 63838: ambari-server setup should surface GPL software agreement

2017-11-15 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63838/ --- Review request for Ambari, Myroslav Papirkovskyy and Sid Wagle. Bugs: