Re: Review Request 52495: AMBARI-18245 Upgrade node to version 4.x

2016-10-18 Thread Richard Zang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52495/#review153220 --- Ship it! Ship It! - Richard Zang On Oct. 18, 2016, 11:37

Re: Review Request 52495: AMBARI-18245 Upgrade node to version 4.x

2016-10-18 Thread Zhe (Joe) Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52495/ --- (Updated Oct. 18, 2016, 11:37 p.m.) Review request for Ambari, Jaimin Jetly,

Re: Review Request 52495: AMBARI-18245 Upgrade node to version 4.x

2016-10-05 Thread Richard Zang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52495/#review151604 --- Ship it! Ship It! - Richard Zang On Oct. 4, 2016, 12:51

Re: Review Request 52495: AMBARI-18245 Upgrade node to version 4.x

2016-10-04 Thread Zhe (Joe) Wang
> On Oct. 3, 2016, 10:57 p.m., Yusaku Sako wrote: > > When you say "local testing done", what did that entail? > > Also, did you explicitly build all the contrib/views modules with the > > change? Added the details. Thank you. - Zhe (Joe)

Re: Review Request 52495: AMBARI-18245 Upgrade node to version 4.x

2016-10-03 Thread Zhe (Joe) Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52495/ --- (Updated Oct. 4, 2016, 12:51 a.m.) Review request for Ambari, Jaimin Jetly,

Re: Review Request 52495: AMBARI-18245 Upgrade node to version 4.x

2016-10-03 Thread Yusaku Sako
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52495/#review151248 --- When you say "local testing done", what did that entail? Also,

Review Request 52495: AMBARI-18245 Upgrade node to version 4.x

2016-10-03 Thread Zhe (Joe) Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52495/ --- Review request for Ambari, Jaimin Jetly, Richard Zang, Vivek Ratnavel