You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hive.apache.org by "Pengcheng Xiong (JIRA)" <ji...@apache.org> on 2016/02/01 13:32:39 UTC

[jira] [Commented] (HIVE-12839) Upgrade Hive to Calcite 1.6

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

Pengcheng Xiong commented on HIVE-12839:
----------------------------------------

The test failures are mainly due to the change in "[CALCITE-1058] Add method RelBuilder.empty, and rewrite LIMIT 0 to it". For example, limit0.q. Due to calcite-1058, which added
{code}
ee283cad (Julian Hyde             2016-01-13 21:37:27 -0800 1299)     if (offsetNode == null && fetch == 0) {
ee283cad (Julian Hyde             2016-01-13 21:37:27 -0800 1300)       return empty();
ee283cad (Julian Hyde             2016-01-13 21:37:27 -0800 1301)     }
{code}
It will return "LogicalValues" rather than a HiveSort. However, Hive ASTConverter can not deal with LogicalValues and it throws exception. [~julianhyde], I was surprised that we did not discover this when we were testing with 1.6.0 snapshot. Is it something newly added into 1.6.0 but not in 1.6.0-snapshot? Thanks. ccing [~jpullokkaran].

> Upgrade Hive to Calcite 1.6
> ---------------------------
>
>                 Key: HIVE-12839
>                 URL: https://issues.apache.org/jira/browse/HIVE-12839
>             Project: Hive
>          Issue Type: Improvement
>            Reporter: Pengcheng Xiong
>            Assignee: Pengcheng Xiong
>         Attachments: HIVE-12839.01.patch, HIVE-12839.02.patch, HIVE-12839.03.patch
>
>
> CLEAR LIBRARY CACHE
> Upgrade Hive to Calcite 1.6.0-incubating.



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