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

Parser for JSON-serialized plans should be more liberal

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Open
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      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

            Activity

              People

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

                Dates

                • Created:
                  Updated: