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

Parser for JSON-serialized plans should be more liberal

    XMLWordPrintableJSON

Details

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

    Description

      Following CALCITE-812, the writer and reader for serialized plans (RelJsonWriter, RelJsonReader) adhere to strict JSON. The writer should continue to emit strict JSON but the reader should accept extended JSON: unquoted keys, strings enclosed in single quotes, and comments.

      Per the robustness principle https://en.wikipedia.org/wiki/Robustness_principle we should "be conservative in what we send, be liberal in what we accept".

      One of the reasons we chose JSON is because it can be written by humans. If we require strict JSON we diminish that.

      Similar arguments apply to the JSON model, but are outside the scope of this issue.

      Attachments

        Issue Links

          is related to

          Bug - A problem which impairs or prevents the functions of the product. CALCITE-812 RelJsonWriter outputs invalid JSON

          • Major - Major loss of function.
          • Closed

          Activity

            People

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

              Dates

                Created:
                Updated: