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

Extend INTERVAL literal syntax

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Open
    • Major
    • Resolution: Unresolved
    • 1.23.0
    • None
    • core
    • None

    Description

      Using this parser configuration: 

      final FrameworkConfig config = Frameworks.newConfigBuilder() .parserConfig(SqlParser.configBuilder()
      .setConformance(MYSQL_5).setLex(MYSQL).build())
      .programs(Programs.ofRules(RULE_SET))
      .build();
      

      Case 1. When parsing : 

      SELECT DATE_ADD('2008-01-02', INTERVAL '31' WEEK)
      

      fails with :

      Encountered "WEEK" at line 1, column 45.
      

      Case 2. When parsing:

      SELECT DATE_ADD('2008-01-02', INTERVAL 31 DAY)
      

      fails with

      Encountered "INTERVAL 31" at line 1, column 31.
      

      Case 3. When parsing

      SELECT DATE_ADD('1970-01-01 12:00:00', INTERVAL CAST(6/4 AS DECIMAL(3,1)) HOUR_MINUTE)
      

      or

      SELECT DATE_ADD('2008-01-02', INTERVAL ((select 1 from dual) - '31') DAY);
      

      fails with:

      Encountered "INTERVAL CAST".    
      
      Encountered "INTERVAL (
      

      Conclusion: INTERVAL is not fully supported:
      Case 1 : WEEK time unit argument is not allowed, for other seems to work fine
      Case 2: expressions(expr) that are nonStrings are not allowed
      Case 3: expressions(expr) are not allowed at all, only string expr are allowed

      According to MySql documentation https://dev.mysql.com/doc/refman/8.0/en/expressions.html#temporal-intervals

      all the above cases should be parsed with success.

      Attachments

        Activity

          People

            julianhyde Julian Hyde
            dan.cojocaru Dan Cojocaru
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated: