jroesch commented on pull request #8806:
URL: https://github.com/apache/tvm/pull/8806#issuecomment-902993040


   Thanks for the contribution guys!
   
   I know how painful of a request this is, but is there any way you guys could 
logically split this up into smaller sets of changes? I know I am one of the 
worst offenders of very large PRs but an 8k PR is very hard/nearly impossible 
to review well in one shot. 
   
   My hope is for us to build wider understanding of these big changes so we 
can spread the future review and maintenance burden across many people. For 
example @areusch has the most context but is on vacation for the next couple of 
weeks. 
   
   


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