manupa-arm edited a comment on issue #9057: URL: https://github.com/apache/tvm/issues/9057#issuecomment-927981725
-1 It feels like a wrong solution to a valid problem. The objection would be for mainly two reasons as follows : 1) The round-robin assignment could miss out interested reviewers. Giving everyone (who have working and contributed the specific component) the opportunity for review, IMO, should be the fair solution. If the problem is that the granularity is too coarse, the actual action might have been make the CODEOWNERS bit more specific. It would be interesting to know why the committers are getting pinged for far too many pull requests between (a) there are lot of activity going on the specific component the committer owning vs (b) there is lot of activity going on many components that are grouped to a single bucket where committers may not necessarily be interested in. Are we sure its (a) ? 2) I feel picking two reviewers per PR makes it bit non-voluntary. -- 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]
