manupa-arm edited a comment on issue #9057:
URL: https://github.com/apache/tvm/issues/9057#issuecomment-927981725


   -1 
   
   It feels like the wrong solution to a valid problem. The reason I object 
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]


Reply via email to