You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2022/09/15 23:30:20 UTC

[GitHub] [druid] jon-wei commented on pull request #13089: Add JsonInputFormat option to assume newline delimited JSON, improve parse exception handling for multiline JSON

jon-wei commented on PR #13089:
URL: https://github.com/apache/druid/pull/13089#issuecomment-1248753600

   @FrankChen021 Hm, I'm inclined to keep the internal `lineSplittable` property for now, since we don't support multi-line JSON for batch ingestion, I feel like we'd still need some kind of thing to force batch ingestion to use the `JsonLineReader` (like if the user were to specify `assumeNewlineDelimited` = false in a batch job)
   
   @vogievetsky I added some docs for the new properties


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org