paul-rogers opened a new pull request #2045: DRILL-7683: Add "message parsing" 
to new JSON loader
URL: https://github.com/apache/drill/pull/2045
 
 
   # [DRILL-7683](https://issues.apache.org/jira/browse/DRILL-7683): Add 
"message parsing" to new JSON loader
   
   ## 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.
   
   Note that, aside from the private plugin mentioned above, no other code uses 
the JSON loader yet.
   
   ## Documentation
   
   N/A
   
   ## Testing
   
   Added unit tests. Reran all existing tests.

----------------------------------------------------------------
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:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to