You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hive.apache.org by "Matt McCline (JIRA)" <ji...@apache.org> on 2018/04/30 06:14:00 UTC
[jira] [Commented] (HIVE-19357) Vectorization: assert_true
HiveException erroneously get suppressed to NULL
[ https://issues.apache.org/jira/browse/HIVE-19357?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16458343#comment-16458343 ]
Matt McCline commented on HIVE-19357:
-------------------------------------
[~kgyrtkirk] please code review and please give it a +1 tests pending. Key code change is in VectorUDFAdaptor. Adding "throws HiveException" to VectorExpression.evaluate rippled upward.
> Vectorization: assert_true HiveException erroneously get suppressed to NULL
> ---------------------------------------------------------------------------
>
> Key: HIVE-19357
> URL: https://issues.apache.org/jira/browse/HIVE-19357
> Project: Hive
> Issue Type: Bug
> Components: Vectorization
> Reporter: Zoltan Haindrich
> Assignee: Matt McCline
> Priority: Major
> Attachments: HIVE-19357.01.patch
>
>
> This could be limited to assert exceptions; but might interfere with other exceptions...discovered while "fixing" testreopt after HIVE-19269
> {code}
> create table tu(id_uv int,id_uw int,u int);
> create table tv(id_uv int,v int);
> create table tw(id_uw int,w int);
> insert into tu values (10,10,10),(1,1,1),(2,2,2),(3,3,3),(4,4,4),(5,5,5),(6,6,6);
> insert into tv values (10,10),(1,1),(2,2),(3,3);
> insert into tw values (10,10),(1,1),(2,2),(3,3),(4,4),(5,5),(6,6),(7,7),(8,8),(9,9);
> set zzz=0;
> set hive.vectorized.execution.enabled=false;
> select assert_true(${hiveconf:zzz}>sum(1)) from tu join tv on (tu.id_uv=tv.id_uv) where u<10 and v>1;
> -- fails as expected
> set hive.vectorized.execution.enabled=true;
> select assert_true(${hiveconf:zzz}>sum(1)) from tu join tv on (tu.id_uv=tv.id_uv) where u<10 and v>1;
> -- there is a result set
> {code}
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)