[jira] [Assigned] (HIVE-24018) Review necessity of AggregationDesc#setGenericUDAFWritableEvaluator for bloom filter aggregations

["Stamatis Zampetakis (Jira)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/HIVE-24018?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Stamatis Zampetakis reassigned HIVE-24018:
------------------------------------------


> Review necessity of AggregationDesc#setGenericUDAFWritableEvaluator for bloom filter aggregations
> -------------------------------------------------------------------------------------------------
>
>                 Key: HIVE-24018
>                 URL: https://issues.apache.org/jira/browse/HIVE-24018
>             Project: Hive
>          Issue Type: Improvement
>            Reporter: Stamatis Zampetakis
>            Assignee: Stamatis Zampetakis
>            Priority: Minor
>
> Few places in the code have following pattern 
> {code:java}
> GenericUDAFBloomFilterEvaluator bloomFilterEval = new GenericUDAFBloomFilterEvaluator();
> ...
> AggregationDesc bloom = new AggregationDesc("bloom_filter", bloomFilterEval, p, false, mode);
> bloom.setGenericUDAFWritableEvaluator(bloomFilterEval);
> {code}
> where the bloom filter evaluator is passed in the constructor of the aggregation and  directly after using a setter. The use of the setter is necessary otherwise there are runtime failures of the query however the pattern is a bit confusing. 
> Investigate if there is a way to avoid the double passing of the evaluator. 
> To reproduce the failure remove the setter and run the following test.
> {noformat}
> mvn test -Dtest=TestMiniLlapLocalCliDriver -Dqfile=vectorized_dynamic_semijoin_reduction.q -Dtest.output.overwrite 
> {noformat}



--
This message was sent by Atlassian Jira
(v8.3.4#803005)