GitHub user zhztheplayer added a comment to the discussion: Infra: Remove the 
approval restriction of merging PRs

Code conflict is not the key. Code always conflicts, and we don't merge PRs 
only because of that.

> We should consider whether to break it down into smaller PRs.

Actually, if we have an approval restriction, committers will tend to file 
large PRs than splitting them up. Because  the smallest PR still need to wait 
for approval. Sometimes it's not needed.


GitHub link: 
https://github.com/apache/incubator-gluten/discussions/8556#discussioncomment-11864837

----
This is an automatically sent email for [email protected].
To unsubscribe, please send an email to: [email protected]


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

Reply via email to