Re: Review Request 27247: convert joinOp to MapJoinOp and generate MapWorks only

2014-10-28 Thread Suhas Satish
> On Oct. 27, 2014, 11:03 p.m., Szehon Ho wrote: > > ql/src/java/org/apache/hadoop/hive/ql/parse/spark/SparkCompiler.java, line > > 151 > > > > > > I don't see this class attached This was already included as a part

Re: Review Request 27247: convert joinOp to MapJoinOp and generate MapWorks only

2014-10-28 Thread Suhas Satish
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27247/ --- (Updated Oct. 29, 2014, 6:24 a.m.) Review request for hive, Rui Li, Szehon Ho,

Re: Review Request 27247: convert joinOp to MapJoinOp and generate MapWorks only

2014-10-28 Thread Xuefu Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27247/#review58833 --- ql/src/java/org/apache/hadoop/hive/ql/optimizer/spark/SparkConvertJ

Re: Review Request 27247: convert joinOp to MapJoinOp and generate MapWorks only

2014-10-27 Thread Szehon Ho
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27247/#review58712 --- Hi Suhas, thanks a lot for this great work! I haven't looked at all