DonalEvans commented on pull request #6386:
URL: https://github.com/apache/geode/pull/6386#issuecomment-830254616


   > Closing PR because I don't want to waste the time of CODEOWNERs to handle 
a PR that is going to feature/GEODE-7665 instead of develop. I also have lots 
of conflicts to handle again.
   
   For what it's worth, I don't consider it a waste of time to review commits 
that are going to a feature branch that will at some point be merged to 
develop. By reviewing commits as they go into the feature branch, far less work 
will be needed when it comes time to merge the branch in, as codeowners can be 
confident that all the changes have been previously reviewed and can be given a 
thumbs-up without having to re-review the entire colossal commit (which 
currently would include 114 files changed, 10964 insertions and 1801 deletions).


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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to