infvg commented on PR #12368: URL: https://github.com/apache/gluten/pull/12368#issuecomment-4804821586
@philo-he IMO the goal isn’t to make assigned reviewers the only reviewers, it’s mainly to make sure PRs get an initial look and encourage more people to participate. Anyone else can still review, and then committers can do a final pass before they merge. For example, Presto has a bot that adds 1-2 people to every PR just for an initial pass. Those people generally aren't the ones who merge/approve (and sometimes they don't even review), but it encourages more participation. I can also change this to use reviewer groups per area, for example like an Iceberg group, Clickhouse group or a global group, instead of one global round-robin. Anyone interested in an area could add themselves to that group. That should help keep reviews closer to the right person while also helping people stay involved with whichever part of the codebase they want. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
