Hive
  1. Hive
  2. HIVE-1255

Add mathematical UDFs PI, E, degrees, radians, tan, sign, and atan

    Details

    • Type: New Feature New Feature
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.5.0
    • Fix Version/s: 0.6.0
    • Component/s: Query Processor
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      Add support for PI, E, degrees, radians, tan, sign and atan

      1. hive-1255-patch-4.txt
        50 kB
        Edward Capriolo
      2. hive-1255-patch-3.txt
        49 kB
        Edward Capriolo
      3. hive-1255-patch-2.txt
        45 kB
        Edward Capriolo
      4. hive-1255-patch.txt
        34 kB
        Edward Capriolo

        Issue Links

          Activity

          Hide
          John Sichi added a comment -

          Committed. Thanks Edward!

          Show
          John Sichi added a comment - Committed. Thanks Edward!
          Hide
          John Sichi added a comment -

          +1. Will commit if tests pass.

          Show
          John Sichi added a comment - +1. Will commit if tests pass.
          Hide
          Paul Yang added a comment -

          Looks good +1

          Show
          Paul Yang added a comment - Looks good +1
          Hide
          Edward Capriolo added a comment -

          show_functions.q is patched

          Show
          Edward Capriolo added a comment - show_functions.q is patched
          Hide
          Paul Yang added a comment -

          Can you update show_functions.q? That test will fail with the addition of new UDF's.

          Show
          Paul Yang added a comment - Can you update show_functions.q? That test will fail with the addition of new UDF's.
          Hide
          Paul Yang added a comment -

          Taking a look.

          Show
          Paul Yang added a comment - Taking a look.
          Hide
          Edward Capriolo added a comment -

          Fixed test cases

          Show
          Edward Capriolo added a comment - Fixed test cases
          Hide
          Edward Capriolo added a comment -

          Fixes the casts. I noticed had "helped" me by doing that. I thought it did not matter. Good catch!

          Show
          Edward Capriolo added a comment - Fixes the casts. I noticed had "helped" me by doing that. I thought it did not matter. Good catch!
          Hide
          Zheng Shao added a comment -

          Edward, can you add back the (unnecessary) type casts in FunctionRegistry.java?
          These are required to get hive compilable with hadoop 0.17.

          Show
          Zheng Shao added a comment - Edward, can you add back the (unnecessary) type casts in FunctionRegistry.java? These are required to get hive compilable with hadoop 0.17.

            People

            • Assignee:
              Edward Capriolo
              Reporter:
              Edward Capriolo
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development