Hive
  1. Hive
  2. HIVE-2288

Adding the oracle nvl function to the UDF

    Details

    • Type: New Feature New Feature
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 0.9.0
    • Fix Version/s: 0.11.0
    • Component/s: UDF
    • Labels:
    • Tags:
      UDF NVL

      Description

      It would be nice if we could use the nvl function, described at oracle:
      http://www.techonthenet.com/oracle/functions/nvl.php

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Patch Available Patch Available Open Open
        2d 3h 45m 1 Namit Jain 23/Apr/12 06:12
        Open Open Patch Available Patch Available
        279d 3h 33m 2 Edward Capriolo 23/Apr/12 14:57
        Patch Available Patch Available Resolved Resolved
        230d 15h 53m 1 Ashutosh Chauhan 10/Dec/12 05:50
        Resolved Resolved Closed Closed
        157d 15h 19m 1 Owen O'Malley 16/May/13 22:10
        Swarnim Kulkarni made changes -
        Labels hive TODOC11 hive
        Owen O'Malley made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Hide
        Hudson added a comment -

        Integrated in Hive-trunk-hadoop2 #54 (See https://builds.apache.org/job/Hive-trunk-hadoop2/54/)
        HIVE-2288 : Adding the oracle nvl function to the UDF (Ed Capriolo, Guy Doulberg via Ashutosh Chauhan) (Revision 1419204)

        Result = ABORTED
        hashutosh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1419204
        Files :

        • /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/FunctionRegistry.java
        • /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFNvl.java
        • /hive/trunk/ql/src/test/queries/clientpositive/udf_nvl.q
        • /hive/trunk/ql/src/test/results/clientpositive/show_functions.q.out
        • /hive/trunk/ql/src/test/results/clientpositive/udf_nvl.q.out
        Show
        Hudson added a comment - Integrated in Hive-trunk-hadoop2 #54 (See https://builds.apache.org/job/Hive-trunk-hadoop2/54/ ) HIVE-2288 : Adding the oracle nvl function to the UDF (Ed Capriolo, Guy Doulberg via Ashutosh Chauhan) (Revision 1419204) Result = ABORTED hashutosh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1419204 Files : /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/FunctionRegistry.java /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFNvl.java /hive/trunk/ql/src/test/queries/clientpositive/udf_nvl.q /hive/trunk/ql/src/test/results/clientpositive/show_functions.q.out /hive/trunk/ql/src/test/results/clientpositive/udf_nvl.q.out
        Hide
        Hudson added a comment -

        Integrated in Hive-trunk-h0.21 #1848 (See https://builds.apache.org/job/Hive-trunk-h0.21/1848/)
        HIVE-2288 : Adding the oracle nvl function to the UDF (Ed Capriolo, Guy Doulberg via Ashutosh Chauhan) (Revision 1419204)

        Result = FAILURE
        hashutosh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1419204
        Files :

        • /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/FunctionRegistry.java
        • /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFNvl.java
        • /hive/trunk/ql/src/test/queries/clientpositive/udf_nvl.q
        • /hive/trunk/ql/src/test/results/clientpositive/show_functions.q.out
        • /hive/trunk/ql/src/test/results/clientpositive/udf_nvl.q.out
        Show
        Hudson added a comment - Integrated in Hive-trunk-h0.21 #1848 (See https://builds.apache.org/job/Hive-trunk-h0.21/1848/ ) HIVE-2288 : Adding the oracle nvl function to the UDF (Ed Capriolo, Guy Doulberg via Ashutosh Chauhan) (Revision 1419204) Result = FAILURE hashutosh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1419204 Files : /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/FunctionRegistry.java /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFNvl.java /hive/trunk/ql/src/test/queries/clientpositive/udf_nvl.q /hive/trunk/ql/src/test/results/clientpositive/show_functions.q.out /hive/trunk/ql/src/test/results/clientpositive/udf_nvl.q.out
        Ashutosh Chauhan made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Fix Version/s 0.11 [ 12323587 ]
        Resolution Fixed [ 1 ]
        Hide
        Ashutosh Chauhan added a comment -

        Committed to trunk. Thanks, Ed and Guy!

        Show
        Ashutosh Chauhan added a comment - Committed to trunk. Thanks, Ed and Guy!
        Hide
        Ashutosh Chauhan added a comment -

        +1 will commit if tests pass.

        Show
        Ashutosh Chauhan added a comment - +1 will commit if tests pass.
        Edward Capriolo made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Affects Version/s 0.9.0 [ 12317742 ]
        Affects Version/s 0.8.1 [ 12319268 ]
        Hide
        Edward Capriolo added a comment -

        Namit, I added the test cases and related changed. Is there anyreason you reverted this back to OPEN? In my eyes it is read for review.

        Show
        Edward Capriolo added a comment - Namit, I added the test cases and related changed. Is there anyreason you reverted this back to OPEN? In my eyes it is read for review.
        Namit Jain made changes -
        Status Patch Available [ 10002 ] Open [ 1 ]
        Edward Capriolo made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Affects Version/s 0.8.1 [ 12319268 ]
        Assignee Edward Capriolo [ appodictic ]
        Edward Capriolo made changes -
        Attachment hive-2288.2.patch.txt [ 12523604 ]
        Hide
        Edward Capriolo added a comment -

        You need to include test cases will a patch. Are you still looking to work on this?

        Show
        Edward Capriolo added a comment - You need to include test cases will a patch. Are you still looking to work on this?
        Guy Doulberg made changes -
        Field Original Value New Value
        Attachment 0002-HIVE-2288-Adding-the-oracle-nvl-function-to-the-UDF.patch [ 12487014 ]
        Hide
        Guy Doulberg added a comment -

        I created a it myself...
        Attached a patch

        Show
        Guy Doulberg added a comment - I created a it myself... Attached a patch
        Guy Doulberg created issue -

          People

          • Assignee:
            Edward Capriolo
            Reporter:
            Guy Doulberg
          • Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development