Re: Review Request 48839: HIVE-14029: Update Spark version to 2.0.0

2016-09-20 Thread cheng xu
> On Sept. 21, 2016, 8:54 a.m., Szehon Ho wrote: > > This looks straight-forward and good to me (once 2.0.0 is the version in > > pom) Thanks Sezhon for your review. I have updated some versions required by Spark side. - cheng ---

Re: Review Request 48839: HIVE-14029: Update Spark version to 2.0.0

2016-09-20 Thread cheng xu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48839/ --- (Updated Sept. 21, 2016, 1:27 p.m.) Review request for hive, Rui Li, Sergio

Re: Review Request 48839: HIVE-14029: Update Spark version to 2.0.0

2016-09-20 Thread cheng xu
> On Sept. 21, 2016, 3:44 a.m., Sahil Takiar wrote: > > pom.xml, line 179 > > > > > > Can this be changed to `2.0.0` instead of `2.0.0-preview` > > Sahil Takiar wrote: > Looked at your updated patch, seems like

Re: Review Request 48839: HIVE-14029: Update Spark version to 2.0.0

2016-09-20 Thread Szehon Ho
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48839/#review149771 --- Ship it! This looks straight-forward and good to me (once

Re: Review Request 48839: HIVE-14029: Update Spark version to 2.0.0

2016-09-20 Thread Sahil Takiar
> On Sept. 20, 2016, 7:44 p.m., Sahil Takiar wrote: > > pom.xml, line 179 > > > > > > Can this be changed to `2.0.0` instead of `2.0.0-preview` Looked at your updated patch, seems like you already did this. -

Re: Review Request 48839: HIVE-14029: Update Spark version to 2.0.0

2016-09-20 Thread Sahil Takiar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48839/#review149717 --- pom.xml (line 179)

Review Request 48839: HIVE-14029: Update Spark version to 2.0.0

2016-06-16 Thread cheng xu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48839/ --- Review request for hive, Rui Li, Szehon Ho, and Xuefu Zhang. Bugs: HIVE-14029