[GitHub] [druid] clintropolis commented on pull request #13939: Lower default maxRowsInMemory for realtime ingestion.

["clintropolis (via GitHub)" <gi...@apache.org>]

clintropolis commented on PR #13939:
URL: https://github.com/apache/druid/pull/13939#issuecomment-1546366288

   > @gianm in the draft release notes we indicate this change improves throughput, but from reading your reasoning it sounds like this actually improves query performance at the expense of throughput, am I reading this right? cc @clintropolis
   
   yeah, this is optimizing for query perf by flushing to disk more frequently, fixed the release notes to say instead
   
   >Optimized query performance by lowering the default maxRowsInMemory for real-time ingestion, which might lower overall ingestion throughput https://github.com/apache/druid/pull/13939
   


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