Uploaded image for project: 'Tajo'
  1. Tajo
  2. TAJO-569

Add max(TEXT) function

        Details

        • Type: Bug
        • Status: Resolved
        • Priority: Major
        • Resolution: Fixed
        • Affects Version/s: None
        • Fix Version/s: 0.8.0
        • Component/s: Function/UDF
        • Labels:
          None

          Description

          There is min function for text parameter, where there is no max function for text parameter. We need to add it.

          1. Text File
            TAJO-569.patch
            6 kB
            Hyunsik Choi

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            hyunsik Hyunsik Choi added a comment -

            Created a review request against branch master in reviewboard
            https://reviews.apache.org/r/17485/

            Show
            hyunsik Hyunsik Choi added a comment - Created a review request against branch master in reviewboard https://reviews.apache.org/r/17485/
            Hide
            Permalink
            hyunsik Hyunsik Choi added a comment -

            It got +1 on RB. committed it to master.

            Show
            hyunsik Hyunsik Choi added a comment - It got +1 on RB. committed it to master.
            Hide
            Permalink
            hudson Hudson added a comment -

            ABORTED: Integrated in Tajo-master-build #47 (See https://builds.apache.org/job/Tajo-master-build/47/)
            TAJO-569: Add max(TEXT) function. (hyunsik) (hyunsik: https://git-wip-us.apache.org/repos/asf?p=incubator-tajo.git&a=commit&h=d516fc499d646b230fb38ebb1fdd72113d46a536)

            • tajo-core/tajo-core-backend/src/test/resources/results/TestBuiltinFunctions/testMaxString.result
            • tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/engine/function/TestBuiltinFunctions.java
            • tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/function/builtin/MaxString.java
            • tajo-core/tajo-core-backend/src/test/resources/results/TestBuiltinFunctions/testMinString.result
            • tajo-core/tajo-core-backend/src/test/resources/queries/TestBuiltinFunctions/testMinString.sql
            • tajo-core/tajo-core-backend/src/test/resources/queries/TestBuiltinFunctions/testMaxString.sql
            • CHANGES.txt
            Show
            hudson Hudson added a comment - ABORTED: Integrated in Tajo-master-build #47 (See https://builds.apache.org/job/Tajo-master-build/47/ ) TAJO-569 : Add max(TEXT) function. (hyunsik) (hyunsik: https://git-wip-us.apache.org/repos/asf?p=incubator-tajo.git&a=commit&h=d516fc499d646b230fb38ebb1fdd72113d46a536 ) tajo-core/tajo-core-backend/src/test/resources/results/TestBuiltinFunctions/testMaxString.result tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/engine/function/TestBuiltinFunctions.java tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/function/builtin/MaxString.java tajo-core/tajo-core-backend/src/test/resources/results/TestBuiltinFunctions/testMinString.result tajo-core/tajo-core-backend/src/test/resources/queries/TestBuiltinFunctions/testMinString.sql tajo-core/tajo-core-backend/src/test/resources/queries/TestBuiltinFunctions/testMaxString.sql CHANGES.txt

              People

              • Assignee:
                hyunsik Hyunsik Choi
                Reporter:
                hyunsik Hyunsik Choi
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Development