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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to