Details

    • Type: Sub-task
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.8.0
    • Component/s: Catalog
    • Labels:
      None
    1. TAJO-61.patch
      12 kB
      DaeMyung Kang

      Activity

      Hide
      charsyam DaeMyung Kang added a comment -

      This time, I don't implement for timezone, like Date or Timestamp.
      Please Review this seriously.

      Show
      charsyam DaeMyung Kang added a comment - This time, I don't implement for timezone, like Date or Timestamp. Please Review this seriously.
      Hide
      hyunsik Hyunsik Choi added a comment -

      +1

      Show
      hyunsik Hyunsik Choi added a comment - +1
      Hide
      hyunsik Hyunsik Choi added a comment -

      committed this patch to master. Thank you for your contribution.

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

      The patch was great. Actually, more work would be needed in order to enable users to use date time types. I'll create more issues for them.

      Show
      hyunsik Hyunsik Choi added a comment - The patch was great. Actually, more work would be needed in order to enable users to use date time types. I'll create more issues for them.
      Hide
      hudson Hudson added a comment -

      FAILURE: Integrated in Tajo-trunk-postcommit #612 (See https://builds.apache.org/job/Tajo-trunk-postcommit/612/)
      TAJO-61: Implement Time Datum Type. (DaeMyung Kang via hyunsik) (hyunsik: https://git-wip-us.apache.org/repos/asf?p=incubator-tajo.git&a=commit&h=2ff0eeaccb1ff37c5f54d1ac8aafedda7bdf6855)

      • tajo-common/src/test/java/org/apache/tajo/datum/TestTimeDatum.java
      • tajo-common/src/main/java/org/apache/tajo/datum/DatumFactory.java
      • CHANGES.txt
      • tajo-common/src/main/java/org/apache/tajo/json/TimeDatumAdapter.java
      • tajo-common/src/main/java/org/apache/tajo/datum/TimeDatum.java
      • tajo-common/src/main/java/org/apache/tajo/json/CommonGsonHelper.java
      Show
      hudson Hudson added a comment - FAILURE: Integrated in Tajo-trunk-postcommit #612 (See https://builds.apache.org/job/Tajo-trunk-postcommit/612/ ) TAJO-61 : Implement Time Datum Type. (DaeMyung Kang via hyunsik) (hyunsik: https://git-wip-us.apache.org/repos/asf?p=incubator-tajo.git&a=commit&h=2ff0eeaccb1ff37c5f54d1ac8aafedda7bdf6855 ) tajo-common/src/test/java/org/apache/tajo/datum/TestTimeDatum.java tajo-common/src/main/java/org/apache/tajo/datum/DatumFactory.java CHANGES.txt tajo-common/src/main/java/org/apache/tajo/json/TimeDatumAdapter.java tajo-common/src/main/java/org/apache/tajo/datum/TimeDatum.java tajo-common/src/main/java/org/apache/tajo/json/CommonGsonHelper.java

        People

        • Assignee:
          charsyam DaeMyung Kang
          Reporter:
          hyunsik Hyunsik Choi
        • Votes:
          0 Vote for this issue
          Watchers:
          2 Start watching this issue

          Dates

          • Created:
            Updated:
            Resolved:

            Development