[jira] [Commented] (HIVE-3241) GenericUDAF when packaged as a JAR raises IllegalAccessException unless the JAR is placed in lib dir

2015-04-09 Thread Hive QA (JIRA)

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

Hive QA commented on HIVE-3241:
---



{color:red}Overall{color}: -1 at least one tests failed

Here are the results of testing the latest attachment:
https://issues.apache.org/jira/secure/attachment/12724020/HIVE-3241.1.patch

{color:red}ERROR:{color} -1 due to 13 failed/errored test(s), 8665 tests 
executed
*Failed tests:*
{noformat}
TestMinimrCliDriver-bucketmapjoin6.q-constprog_partitioner.q-infer_bucket_sort_dyn_part.q-and-1-more
 - did not produce a TEST-*.xml file
TestMinimrCliDriver-external_table_with_space_in_location_path.q-infer_bucket_sort_merge.q-auto_sortmerge_join_16.q-and-1-more
 - did not produce a TEST-*.xml file
TestMinimrCliDriver-groupby2.q-import_exported_table.q-bucketizedhiveinputformat.q-and-1-more
 - did not produce a TEST-*.xml file
TestMinimrCliDriver-index_bitmap3.q-stats_counter_partitioned.q-temp_table_external.q-and-1-more
 - did not produce a TEST-*.xml file
TestMinimrCliDriver-infer_bucket_sort_map_operators.q-join1.q-bucketmapjoin7.q-and-1-more
 - did not produce a TEST-*.xml file
TestMinimrCliDriver-infer_bucket_sort_num_buckets.q-disable_merge_for_bucketing.q-uber_reduce.q-and-1-more
 - did not produce a TEST-*.xml file
TestMinimrCliDriver-infer_bucket_sort_reducers_power_two.q-scriptfile1.q-scriptfile1_win.q-and-1-more
 - did not produce a TEST-*.xml file
TestMinimrCliDriver-leftsemijoin_mr.q-load_hdfs_file_with_space_in_the_name.q-root_dir_external_table.q-and-1-more
 - did not produce a TEST-*.xml file
TestMinimrCliDriver-list_bucket_dml_10.q-bucket_num_reducers.q-bucket6.q-and-1-more
 - did not produce a TEST-*.xml file
TestMinimrCliDriver-load_fs2.q-file_with_header_footer.q-ql_rewrite_gbtoidx_cbo_1.q-and-1-more
 - did not produce a TEST-*.xml file
TestMinimrCliDriver-parallel_orderby.q-reduce_deduplicate.q-ql_rewrite_gbtoidx_cbo_2.q-and-1-more
 - did not produce a TEST-*.xml file
TestMinimrCliDriver-ql_rewrite_gbtoidx.q-smb_mapjoin_8.q - did not produce a 
TEST-*.xml file
TestMinimrCliDriver-schemeAuthority2.q-bucket4.q-input16_cc.q-and-1-more - did 
not produce a TEST-*.xml file
{noformat}

Test results: 
http://ec2-174-129-184-35.compute-1.amazonaws.com/jenkins/job/PreCommit-HIVE-TRUNK-Build/3349/testReport
Console output: 
http://ec2-174-129-184-35.compute-1.amazonaws.com/jenkins/job/PreCommit-HIVE-TRUNK-Build/3349/console
Test logs: 
http://ec2-174-129-184-35.compute-1.amazonaws.com/logs/PreCommit-HIVE-TRUNK-Build-3349/

Messages:
{noformat}
Executing org.apache.hive.ptest.execution.PrepPhase
Executing org.apache.hive.ptest.execution.ExecutionPhase
Executing org.apache.hive.ptest.execution.ReportingPhase
Tests exited with: TestsFailedException: 13 tests failed
{noformat}

This message is automatically generated.

ATTACHMENT ID: 12724020 - PreCommit-HIVE-TRUNK-Build

> GenericUDAF when packaged as a JAR raises IllegalAccessException unless the 
> JAR is placed in lib dir
> 
>
> Key: HIVE-3241
> URL: https://issues.apache.org/jira/browse/HIVE-3241
> Project: Hive
>  Issue Type: Bug
>  Components: UDF
>Affects Versions: 0.9.0
>Reporter: Shreepadma Venugopalan
> Attachments: HIVE-3241.1.patch, compute-stats.jar
>
>
> GenericUDAFEvaluator has a member variable "Mode" which is protected. 
> Accessing mode in a GenericUDAF class raises IllegalAccessException when 
> packaged as a jar. This looks like a class loader issue because the error 
> doesn't occur when the jar is placed in HIVE_LIB directory. The fix for this 
> issue is to provide public getters for member variable mode in 
> GenericUDAFEvaluator.
> Steps to reproduce,
> Copy the attached jar to a dir other than HIVE_LIB
> On a hive shell,
> add jar 
> create temporary function compute_stats as 
> 'org.apache.hadoop.hive.ql.udf.generic.GenericUDAFComputeStats';
>  select compute_stats(some_col) from some_table;



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HIVE-3241) GenericUDAF when packaged as a JAR raises IllegalAccessException unless the JAR is placed in lib dir

2015-04-08 Thread Alexander Pivovarov (JIRA)

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

Alexander Pivovarov commented on HIVE-3241:
---

"Mode mode" does not have any access modified. So, it is not protected, it's 
"default" from the day it was added.
https://github.com/apache/hive/commit/f376e632ce4796e4ffb740434b3cfcdae590a1ec#diff-fa86fb58b7827b5e9569d609e8a0eac0

Because mode field is used by Subclasses it should be protected (not "default")
GenericUDAFEvaluator subclasses can access mode field because they are located 
in the same package as GenericUDAFEvaluator.

But protected modifier is better because it clearly says that the field can be 
accessed by child classes


> GenericUDAF when packaged as a JAR raises IllegalAccessException unless the 
> JAR is placed in lib dir
> 
>
> Key: HIVE-3241
> URL: https://issues.apache.org/jira/browse/HIVE-3241
> Project: Hive
>  Issue Type: Bug
>  Components: UDF
>Affects Versions: 0.9.0
>Reporter: Shreepadma Venugopalan
> Attachments: compute-stats.jar
>
>
> GenericUDAFEvaluator has a member variable "Mode" which is protected. 
> Accessing mode in a GenericUDAF class raises IllegalAccessException when 
> packaged as a jar. This looks like a class loader issue because the error 
> doesn't occur when the jar is placed in HIVE_LIB directory. The fix for this 
> issue is to provide public getters for member variable mode in 
> GenericUDAFEvaluator.
> Steps to reproduce,
> Copy the attached jar to a dir other than HIVE_LIB
> On a hive shell,
> add jar 
> create temporary function compute_stats as 
> 'org.apache.hadoop.hive.ql.udf.generic.GenericUDAFComputeStats';
>  select compute_stats(some_col) from some_table;



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)