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

Add parseDateTime method with TimeZone in DateTimeFormat class

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Critical
    • Resolution: Won't Fix
    • Affects Version/s: 0.11.2
    • Fix Version/s: 0.12.0, 0.11.2
    • Component/s: Data Type
    • Labels:
      None

      Description

      I added the UDF for my application like below.

      TimeMeta tm = DateTimeFormat.parseDateTime(pastDateText, "YYYYMMDD");
      DateTimeUtil.toUserTimezone(tm, timezone);

      I expected to return the date to apply a time zone but It wasn't working.
      So I hope to add parseDateTime method with TimeZone.

      Thanks.

        Activity

        Hide
        githubbot ASF GitHub Bot added a comment -

        GitHub user combineads opened a pull request:

        https://github.com/apache/tajo/pull/970

        TAJO-2085 Add parseDateTime method with TimeZone in DateTimeFormat class

        I added the UDF for my application like below.

        TimeMeta tm = DateTimeFormat.parseDateTime(pastDateText, "YYYYMMDD");
        DateTimeUtil.toUserTimezone(tm, timezone);

        I expected to return the date to apply a time zone but It wasn't working.
        So I hope to add parseDateTime method with TimeZone.
        Thanks.

        You can merge this pull request into a Git repository by running:

        $ git pull https://github.com/combineads/tajo TAJO-2085

        Alternatively you can review and apply these changes as the patch at:

        https://github.com/apache/tajo/pull/970.patch

        To close this pull request, make a commit to your master/trunk branch
        with (at least) the following in the commit message:

        This closes #970


        commit b6631087926145732e5c61b45c48089e8fc163ad
        Author: Byunghwa Yun <combine@combineads.co.kr>
        Date: 2016-03-02T01:34:02Z

        TAJO-2085
        Add parseDateTime method with TimeZone in DateTimeFormat class


        Show
        githubbot ASF GitHub Bot added a comment - GitHub user combineads opened a pull request: https://github.com/apache/tajo/pull/970 TAJO-2085 Add parseDateTime method with TimeZone in DateTimeFormat class I added the UDF for my application like below. TimeMeta tm = DateTimeFormat.parseDateTime(pastDateText, "YYYYMMDD"); DateTimeUtil.toUserTimezone(tm, timezone); I expected to return the date to apply a time zone but It wasn't working. So I hope to add parseDateTime method with TimeZone. Thanks. You can merge this pull request into a Git repository by running: $ git pull https://github.com/combineads/tajo TAJO-2085 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/tajo/pull/970.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #970 commit b6631087926145732e5c61b45c48089e8fc163ad Author: Byunghwa Yun <combine@combineads.co.kr> Date: 2016-03-02T01:34:02Z TAJO-2085 Add parseDateTime method with TimeZone in DateTimeFormat class
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user combineads closed the pull request at:

        https://github.com/apache/tajo/pull/970

        Show
        githubbot ASF GitHub Bot added a comment - Github user combineads closed the pull request at: https://github.com/apache/tajo/pull/970
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user combineads commented on the pull request:

        https://github.com/apache/tajo/pull/970#issuecomment-191013694

        It was to resolve useing DateTimeUtil.toUTCTimezone(tm, tz);
        Thanks a lot.

        Show
        githubbot ASF GitHub Bot added a comment - Github user combineads commented on the pull request: https://github.com/apache/tajo/pull/970#issuecomment-191013694 It was to resolve useing DateTimeUtil.toUTCTimezone(tm, tz); Thanks a lot.
        Hide
        combine Byunghwa Yun added a comment -

        It was to resolve useing DateTimeUtil.toUTCTimezone(tm, tz);
        Thanks a lot.

        Show
        combine Byunghwa Yun added a comment - It was to resolve useing DateTimeUtil.toUTCTimezone(tm, tz); Thanks a lot.

          People

          • Assignee:
            combine Byunghwa Yun
            Reporter:
            combine Byunghwa Yun
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Due:
              Created:
              Updated:
              Resolved:

              Development