[jira] [Commented] (HIVE-20961) Retire NVL implementation

["Laszlo Bodor (JIRA)" <ji...@apache.org>]

    [ 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)