-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.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/apache/tvm/issues/9057#issuecomment-927981725

Reply via email to