[GitHub] [druid] capistrant commented on pull request #12571: Set druid.processing.fifo to true by default

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

capistrant commented on PR #12571:
URL: https://github.com/apache/druid/pull/12571#issuecomment-1176658417

   https://github.com/apache/druid/pull/1809
   
   Went back in time to see if there was reasoning behind setting it to false by default. It seems like that was done simply to leave default behavior the same across versions. There is an note in there from implementer in PR description:
   > One downside to the approach in this PR is that a particularly long-running query has the ability to starve out cluster resources, where the prior implementation could interleave (though randomly) other tasks into a long-running query.
   
   This seems like a reasonable change in default to me, but I do wonder how many people may notice odd side effects from this after upgrade and struggle to put their finger on the cause. 


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