hive-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Laszlo Bodor (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HIVE-20961) Retire NVL implementation
Date Mon, 03 Dec 2018 13:01:00 GMT

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

Laszlo Bodor commented on HIVE-20961:
-------------------------------------

patch set 01 uploaded
explain plans show that nvl is aliased to coalesce, which is fine
negative test shows an error message regarding coalesce, when user is using NVL, it's could
be a little confusing, however I don't think it's an issue, because the error message tells
the error correctly

> Retire NVL implementation
> -------------------------
>
>                 Key: HIVE-20961
>                 URL: https://issues.apache.org/jira/browse/HIVE-20961
>             Project: Hive
>          Issue Type: Improvement
>            Reporter: Zoltan Haindrich
>            Assignee: Laszlo Bodor
>            Priority: Major
>         Attachments: HIVE-20961.01.patch
>
>
> Right now we have coalesce and nvl implemented separetly; it might be better to remove
one of them as they are doing the same. Because Coalesce is in the standard -  I think NVL
have to go...and became an alias to Coalesce.
> Further benefit is: that optimizations which coalesce already recieves; will be done
on NVL calls as well.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Mime
View raw message