Re: Review Request 24297: Spark Explain should give useful information on dependencies

2014-08-05 Thread Chao Sun
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24297/ --- (Updated Aug. 5, 2014, 6:09 a.m.) Review request for hive and Brock Noland.

Re: Review Request 24297: Spark Explain should give useful information on dependencies

2014-08-05 Thread Brock Noland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24297/#review49570 --- Hi Chao, Can you share what the output looks like with the patch?

Re: Review Request 24297: Spark Explain should give useful information on dependencies

2014-08-05 Thread Chao Sun
On Aug. 5, 2014, 6:15 a.m., Brock Noland wrote: Hi Chao, Can you share what the output looks like with the patch? Hive has thousands of .q file tests (https://github.com/apache/hive/tree/trunk/ql/src/test/queries/clientpositive) and most of them do an EXPLAIN. Thus I think

Re: Review Request 24297: Spark Explain should give useful information on dependencies

2014-08-05 Thread Chao Sun
On Aug. 5, 2014, 6:15 a.m., Brock Noland wrote: Hi Chao, Can you share what the output looks like with the patch? Hive has thousands of .q file tests (https://github.com/apache/hive/tree/trunk/ql/src/test/queries/clientpositive) and most of them do an EXPLAIN. Thus I think

Re: Review Request 24297: Spark Explain should give useful information on dependencies

2014-08-05 Thread Chao Sun
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24297/ --- (Updated Aug. 5, 2014, 5:43 p.m.) Review request for hive and Brock Noland.