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

Make type conversion documentation consistent with behavior and tests

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Open
    • Major
    • Resolution: Unresolved
    • None
    • None
    • None
    • None

    Description

      Make type conversion documentation consistent with behavior and tests. Use the following steps:
       * Create an internal map that, for each pair of data types, says whether conversion is implicit, explicit, or disallowed
       * Run tests based on that map
       * In DocumentationTest, generate an HTML table of that mapping, and check that that table is consistent with the one in the SQL reference.

      I know of two errors in the table:
       * The table says that you can explicitly cast from INT to TIMESTAMP (see CALCITE-5483); this should be disallowed;

      • The table says that you can explicitly cast from DECIMAL to BOOLEAN (see CALCITE-4777); this should be disallowed.

      There may be others.

      Attachments

        Issue Links

          is related to

          Sub-task - The sub-task of the issue CALCITE-3311 Add doc to site for implicit type coercion

          • Major - Major loss of function.
          • Closed

          Bug - A problem which impairs or prevents the functions of the product. CALCITE-3053 Add a test to ensure that all functions are documented in the SQL reference

          • Major - Major loss of function.
          • Closed

          Bug - A problem which impairs or prevents the functions of the product. CALCITE-4777 Casting from DECIMAL to BOOLEAN throws an exception

          • Major - Major loss of function.
          • Closed

          Bug - A problem which impairs or prevents the functions of the product. CALCITE-5483 ProjectAggregateMergeRule throws exception if literal is non-numeric

          • Major - Major loss of function.
          • Closed

          Activity

            People

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

              Dates

                Created:
                Updated: