[GitHub] [druid] maytasm edited a comment on pull request #10689: Multiphase segment merge for IndexMergerV9

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

maytasm edited a comment on pull request #10689:
URL: https://github.com/apache/druid/pull/10689#issuecomment-749242327


   Also integration test might be useful and easy to add. i.e. a IT that sets the `maxColumnsToMerge` to some > 0 number and make sure that the ingestion succeeded and run a test query to make sure data is correct. This can builds on existing IT to reuse the raw data and query


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