Uploaded image for project: 'Apache Drill'
  1. Apache Drill
  2. DRILL-7683

Add "message parsing" to new JSON loader

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 1.18.0
    • Fix Version/s: 1.18.0
    • Component/s: None
    • Labels:

      Description

      Worked on a project that uses the new JSON loader to parse a REST response that includes a set of "wrapper" fields around the JSON payload. Example:

      { "status": "ok", "results: [ data here ]}
      

      To solve this cleanly, added the ability to specify a "message parser" to consume JSON tokens up to the start of the data. This parser can be written as needed for each different data source.

      Since this change adds one more parameter to the JSON structure parser, added builders to gather the needed parameters rather than making the constructor even larger.

        Attachments

          Activity

            People

            • Assignee:
              Paul.Rogers Paul Rogers
              Reporter:
              Paul.Rogers Paul Rogers
              Reviewer:
              Arina Ielchiieva

              Dates

              • Created:
                Updated:
                Resolved:

                Issue deployment