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

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

VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 1.12.0
    • None
    • 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.)

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

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

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment