andygrove opened a new issue, #3957:
URL: https://github.com/apache/arrow-datafusion/issues/3957

   **Is your feature request related to a problem or challenge? Please describe 
what you are trying to do.**
   DataFusion is a very active fast-moving project and I am starting to see 
some issues with the pace. We often have PRs being merged within 12 hours of 
being filed ( know I am guilty of doing this) and this means that contributors 
in other timezones do not have the opportunity to review. I know that I have 
missed out on reviewing PRs that impact my projects because I was busy on the 
day that the PR got created and merged and it can be frustrating. I am seeing 
this happen to other contributors as well.
   
   I'd like to propose that we start leaving major PRs open for at least 24 
hours. This is at least enough time for contributors to comment if they need 
more time to review. I am not sure what the definition of a "major PR" is, but 
I think it should include PRs that have breaking changes (such as API changes 
or optimizer changes that cause plans to be re-written differently, which is 
effectively an API change as well).
   
   We should write something up in the contributing guide explaining whatever 
policy we decide on.
   
   **Describe the solution you'd like**
   See above.
   
   **Describe alternatives you've considered**
   None
   
   **Additional context**
   None
   


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

Reply via email to