[ 
https://issues.apache.org/jira/browse/HIVE-931?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12780886#action_12780886
 ] 

Namit Jain commented on HIVE-931:
---------------------------------

1. Add new parameter in hive-default.xml
2. Utilities.java: change function names - extractColumnNamesFromSortCols
   variable name: bucketCol: line 813
3. Remove all the tabs
4. Given        the fact that we are not doing  this optimization across 
sub-queries right now,
   would it be simpler to maintain the group by operator to table mapping via a 
separate walker
   instead of getting while generating the group by operator ? -- I am fine 
with        the current approach
   also, but just a question.
5. You are still doing partition pruning in GroupByOptimizer - why cant we 
reuse the mapping from
   ParseContext. That was the whole reason for storing it in ParseContext.


Sorry about being so picky...

> Sorted Group By
> ---------------
>
>                 Key: HIVE-931
>                 URL: https://issues.apache.org/jira/browse/HIVE-931
>             Project: Hadoop Hive
>          Issue Type: New Feature
>          Components: Query Processor
>            Reporter: Namit Jain
>            Assignee: He Yongqiang
>             Fix For: 0.5.0
>
>         Attachments: hive-931-2009-11-18.patch, hive-931-2009-11-19.patch, 
> hive-931-2009-11-20.3.patch
>
>
> If the table is sorted by a given key, we don't use that for group by. That 
> can be very useful.
> For eg: if T is sorted by column c1,
> For select c1, aggr() from T group by c1
> we always use a single map-reduce job. No hash table is needed on the mapper, 
> since the data is sorted by c1 anyway.
> This will reduce the memory pressure on the mapper and also remove overhead 
> of maintaining the hash table.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to