You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2021/09/27 15:24:29 UTC

[GitHub] [tvm] manupa-arm commented on issue #9057: [VOTE] Adopt round-robin assignment of reviewers for GitHub pull request reviewer assignment.

manupa-arm commented 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 necessarily 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: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org