Details

    • Type: New Feature
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.8.0
    • Component/s: None
    • Labels:
      None

      Description

      I'd like to develop this function.

      1. TAJO-368_2.patch
        4 kB
        Hyunsik Choi
      2. TAJO-368.patch
        4 kB
        Seungun Choe

        Activity

        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Tajo-trunk-postcommit #605 (See https://builds.apache.org/job/Tajo-trunk-postcommit/605/)
        TAJO-368: Implement quote_ident function. (Seungun Choe via hyunsik) (hyunsik: https://git-wip-us.apache.org/repos/asf?p=incubator-tajo.git&a=commit&h=25944a91290781774b7566fe64439d33cac1b18a)

        • tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/master/TajoMaster.java
        • tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/function/string/QuoteIdent.java
        • CHANGES.txt
        • tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/engine/function/TestStringOperatorsAndFunctions.java
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Tajo-trunk-postcommit #605 (See https://builds.apache.org/job/Tajo-trunk-postcommit/605/ ) TAJO-368 : Implement quote_ident function. (Seungun Choe via hyunsik) (hyunsik: https://git-wip-us.apache.org/repos/asf?p=incubator-tajo.git&a=commit&h=25944a91290781774b7566fe64439d33cac1b18a ) tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/master/TajoMaster.java tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/function/string/QuoteIdent.java CHANGES.txt tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/engine/function/TestStringOperatorsAndFunctions.java
        Hide
        hyunsik Hyunsik Choi added a comment -

        committed the latest patch to master. Thank you for your contribution.

        Show
        hyunsik Hyunsik Choi added a comment - committed the latest patch to master. Thank you for your contribution.
        Hide
        hyunsik Hyunsik Choi added a comment -

        +1

        This function proposal is good for me. But, this patch gone stale a little bit. So, I rebased and slightly improved your patch.

        Show
        hyunsik Hyunsik Choi added a comment - +1 This function proposal is good for me. But, this patch gone stale a little bit. So, I rebased and slightly improved your patch.
        Hide
        seungunchoe Seungun Choe added a comment -

        I attach patch file for quote_ident function.
        Confirmed with mvn.
        Plz review my patch

        Show
        seungunchoe Seungun Choe added a comment - I attach patch file for quote_ident function. Confirmed with mvn. Plz review my patch
        Hide
        blrunner Jaehwa Jung added a comment -

        Thank you, Seungun Choe. I've assigned this issue to you.

        Show
        blrunner Jaehwa Jung added a comment - Thank you, Seungun Choe . I've assigned this issue to you.

          People

          • Assignee:
            seungunchoe Seungun Choe
            Reporter:
            seungunchoe Seungun Choe
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development