[GitHub] [druid] suneet-s commented on pull request #10962: Fix streaming ingestion fails if it encounters empty rows (Regression)

[GitBox <gi...@apache.org>]

suneet-s commented on pull request #10962:
URL: https://github.com/apache/druid/pull/10962#issuecomment-793980460


   Is it possible to add an integration test for this? Maybe update `SyntheticStreamGenerator#run` to send a null event a configurable number of times per run - then the tests can check how many parse exceptions were thrown.
   
   I think a test like this can help flush out whether similar issues exist with other data formats... To be fair, I don't know why a null event would ever end up in a stream


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



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