alamb commented on issue #3957: URL: https://github.com/apache/arrow-datafusion/issues/3957#issuecomment-1290812835
> I'd like to propose that we start leaving major PRs open for at least 24 hours. I agree 100% with this. Making it common sense so we can merge small changes is also fine with me 👍 Anything marked as API change is for sure should be left open for 24 hours. > @andygrove imho increasing min required reviewers to 2 might help also I would be in favor of this change only if more people in the community can step up to review PRs (especially larger or trickier changes). I falready a struggle to get them reviewed in a timely manner. Adding additional review burden when I think it is one of our most limited resources already does not seem like a good idea to me Not only have we have added several additional committers over the last 12 months, I found reviews by anyone are super helpful as they can identify issues (like lack of tests, other features, etc) that can be addressed and make future reviews faster. -- 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]
