[
https://issues.apache.org/jira/browse/DRILL-3715?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14715725#comment-14715725
]
ASF GitHub Bot commented on DRILL-3715:
---------------------------------------
Github user jaltekruse commented on the pull request:
https://github.com/apache/drill/pull/136#issuecomment-135203792
We should have a plan verification test to make sure that this change
actually has the desired effect. This should be able to execute fine without
your change as well, but the plan will have and unnecessary
SelectionVectorRemover in it today. You can see examples of plan verification
tests in
/Users/jaltekruse/drill/exec/java-exec/src/test/java/org/apache/drill/TestCTASPartitionFilter.java
and other subclasses of PlanTestBase
> Enable selection vector sv2 and sv4 in hash aggregator
> ------------------------------------------------------
>
> Key: DRILL-3715
> URL: https://issues.apache.org/jira/browse/DRILL-3715
> Project: Apache Drill
> Issue Type: Bug
> Components: Execution - Relational Operators
> Reporter: amit hadke
> Assignee: amit hadke
> Priority: Minor
>
> HashAggregator already can read sv2 and sv4 vectors. Enable support for all
> of them.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
