Uploaded image for project: 'Calcite'
  1. Calcite
  2. CALCITE-1623

Make sure the Calendar inside DATE, TIME and TIMESTAMP literals has timezone UTC

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.12.0
    • Component/s: None
    • Labels:
      None

      Description

      At query preparation time, Calcite uses a Calendar to hold the value of DATE, TIME, TIMESTAMP literals. It assumes that Calendar has a UTC (GMT) time zone. Bad things probably happen if it does not. We should check, and throw IllegalArgumentException if not.

      (For the record, Calendar is not used at run time. Those values become int or long, or Integer or Long if nullable.)

        Activity

        Hide
        julianhyde Julian Hyde added a comment -

        Resolved in release 1.12.0 (2017-03-24).

        Show
        julianhyde Julian Hyde added a comment - Resolved in release 1.12.0 (2017-03-24).
        Show
        julianhyde Julian Hyde added a comment - Fixed in http://git-wip-us.apache.org/repos/asf/calcite/commit/a269a2b7 .

          People

          • Assignee:
            julianhyde Julian Hyde
            Reporter:
            julianhyde Julian Hyde
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development