paul-rogers commented on issue #2000: DRILL-7607: support dynamic credit based 
flow control
URL: https://github.com/apache/drill/pull/2000#issuecomment-593159644
 
 
   @weijietong, see the comment above about using config options to enable and 
configure this feature. If we can add those, then we can fall back to existing 
behavior if the new behavior causes problems. For now, I'd leave the feature 
turned off. Then, as you use the feature, and work out any issues, we can add 
additional RM (ideas discussed above), and enable the feature by default.
   
   Does this sound reasonable? If so, please let me know when you're ready for 
another review and I'll be happy to do one. 

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


With regards,
Apache Git Services

Reply via email to