kennknowles commented on PR #26444:
URL: https://github.com/apache/beam/pull/26444#issuecomment-1632594629

   BTW the reason I am suggesting the split: this code clearly needs review & 
revision to match the expectations of the Beam repo. That will be much easier 
to do in pieces.
   
    - We can merge the overall framework with TODOs in place / P1s filed to fix 
the disabled checks
    - Then we could do thorough reviews one at a time of the test modules and 
get them fixed up before merge


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