Harish Butani 2014-04-25, 22:57
-Re: Review Request 20737: HIVE-6031: explain subquery rewrite for where clause predicates
Ashutosh Chauhan 2014-04-28, 18:28
This is an automatically generated e-mail. To reply, visit:
Few comments. Major one is to drop subquery from explain subquery rewrite syntax.
will be good to add a comment here for this replacement.
Is this needed ? Looks redundant code. If not, good to put some comments here.
I think we should not have explain subquery rewrite .. rather explain rewrite .. because later on we may want to extend this explain functionality beyond just subquery rewrites.
Doesn't make lot of difference, but QBSQRewrite extends QBSQRewriteNoop seems more natural.
Also, good to add a note here saying its non-explain path.
- Ashutosh Chauhan
On April 25, 2014, 10:57 p.m., Harish Butani wrote:
Harish Butani 2014-04-29, 00:44
Harish Butani 2014-04-29, 00:43