Re: doc on predicate pushdown in joins

2013-12-16 Thread Lefty Leverenz
> > This is old code, hard to attribute to jiras > Okay, HIVE-2337 was way back in release 0.8.0 so I think we can safely omit version information. If anyone objects, I can link the wikidoc to that ticket. (And I've marked HIVE-2337 as being related to HIVE-279 in the JIRA.) Thanks for all the

Re: doc on predicate pushdown in joins

2013-12-11 Thread Harish Butani
I see. Let's leave it in. This is old code, hard to attribute to jiras: - The PPD code comes from: HIVE-279, HIVE-2337 - I cannot tell when the join condition parsing code was added. regards, Harish. On Dec 11, 2013, at 5:17 PM, Lefty Leverenz wrote: > > Maybe we should remove the section on

Re: doc on predicate pushdown in joins

2013-12-11 Thread Lefty Leverenz
> Maybe we should remove the section on Hive Implementation here. > It is in the Design doc; this information only concerns developers. But this is the Design doc (unless there's another one somewhere -- maybe attached to a JIRA ticket?) and it's in the Resources for Contributors part of the wiki,

Re: doc on predicate pushdown in joins

2013-12-11 Thread Harish Butani
getQualifiedAliases is a private method in JoinPPD. Maybe we should remove the section on Hive Implementation here. It is in the Design doc; this information only concerns developers. regards, Harish. On Dec 11, 2013, at 3:05 AM, Lefty Leverenz wrote: > Happy to fix the sentence and the link

Re: doc on predicate pushdown in joins

2013-12-11 Thread Lefty Leverenz
Happy to fix the sentence and the link. I pointed out the name change just so you would review it, so please don't apologize! One more question: why am I not finding getQualifiedAliases() in the SemanticAnalyzer class? It turns up in OpProcFactory.java with javadoc comments, but I can't find it

Re: doc on predicate pushdown in joins

2013-12-10 Thread Harish Butani
I can see why you would rename. But this sentence is not correct: 'Hive enforces the predicate pushdown rules by these methods in the SemanticAnalyzer and JoinPPD classes:' It should be: Hive enforces the rules by these methods in the SemanticAnalyzer and JoinPPD classes: (The implementation i

Re: doc on predicate pushdown in joins

2013-12-10 Thread Lefty Leverenz
How's this? Hive Implementation Also, I moved the link on the Design Docs pagefrom *Proposed* to *Other*. (It's called SQL Outer J

Re: doc on predicate pushdown in joins

2013-12-10 Thread Harish Butani
You are correct, it is plural. regards, Harish. On Dec 10, 2013, at 4:03 AM, Lefty Leverenz wrote: > Okay, then monospace with "()" after the method name is a good way to show > them: parseJoinCondition() and getQualifiedAlias() ... but I only found the > latter pluralized, instead of singula

Re: doc on predicate pushdown in joins

2013-12-10 Thread Lefty Leverenz
Okay, then monospace with "()" after the method name is a good way to show them: parseJoinCondition() and getQualifiedAlias() ... but I only found the latter pluralized, instead of singular, so should it be getQualifiedAliases() or am I missing something? trunk> *grep -nr 'getQualifiedAlias' ./ql

Re: doc on predicate pushdown in joins

2013-12-09 Thread Harish Butani
Looks good. Thanks for doing this. Minor point: Rule 1: During QBJoinTree construction in Plan Gen, the parse Join Condition logic applies this rule. Rule 2: During JoinPPD (Join Predicate Pushdown) the get Qualified Alias logic applies this rule. FYI 'parseJoinCondition' and 'getQualifiedAli

Re: doc on predicate pushdown in joins

2013-12-09 Thread Thejas Nair
Looks great! Thanks for also linking to it from the other relevant docs! On Mon, Dec 9, 2013 at 1:52 AM, Lefty Leverenz wrote: > The Outer Join Behavior > wikidocis > done, with links from the Design > Docs

Re: doc on predicate pushdown in joins

2013-12-09 Thread Lefty Leverenz
The Outer Join Behavior wikidocis done, with links from the Design Docs page and the Joins doc

Re: doc on predicate pushdown in joins

2013-12-02 Thread Thejas Nair
:) On Mon, Dec 2, 2013 at 6:18 PM, Lefty Leverenz wrote: > Easy as 3.14159 (I can take a hint.) > > -- Lefty > > > On Mon, Dec 2, 2013 at 5:34 PM, Thejas Nair wrote: > >> FYI, Harish has a written a very nice doc describing predicate push >> down rules for join. I have attached it to the d

Re: doc on predicate pushdown in joins

2013-12-02 Thread Lefty Leverenz
Easy as 3.14159 (I can take a hint.) -- Lefty On Mon, Dec 2, 2013 at 5:34 PM, Thejas Nair wrote: > FYI, Harish has a written a very nice doc describing predicate push > down rules for join. I have attached it to the design doc page. It > will be very useful for anyone looking at joins. >

doc on predicate pushdown in joins

2013-12-02 Thread Thejas Nair
FYI, Harish has a written a very nice doc describing predicate push down rules for join. I have attached it to the design doc page. It will be very useful for anyone looking at joins. https://cwiki.apache.org/confluence/download/attachments/27362075/OuterJoinBehavior.html (any help converting it t