You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@drill.apache.org by GitBox <gi...@apache.org> on 2020/03/01 23:02:55 UTC

[GitHub] [drill] paul-rogers commented on issue #2000: DRILL-7607: support dynamic credit based flow control

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:
users@infra.apache.org


With regards,
Apache Git Services