[
https://issues.apache.org/jira/browse/DRILL-7683?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17072770#comment-17072770
]
ASF GitHub Bot commented on DRILL-7683:
---------------------------------------
cgivre commented on issue #2045: DRILL-7683: Add "message parsing" to new JSON
loader
URL: https://github.com/apache/drill/pull/2045#issuecomment-607246450
@paul-rogers
Thank you very much for submitting this. It's funny, the original REST
plugin did this but not very well. This is a really useful functionality.
In any event, I don't want to hold up this PR, but would you mind please
writing up a paragraph documenting this functionality?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
> Add "message parsing" to new JSON loader
> ----------------------------------------
>
> Key: DRILL-7683
> URL: https://issues.apache.org/jira/browse/DRILL-7683
> Project: Apache Drill
> Issue Type: Improvement
> Affects Versions: 1.18.0
> Reporter: Paul Rogers
> Assignee: Paul Rogers
> Priority: Minor
> Labels: ready-to-commit
> Fix For: 1.18.0
>
>
> 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:
> {code:json}
> { "status": "ok", "results: [ data here ]}
> {code}
> 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.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
