kennknowles commented on pull request #12899:
URL: https://github.com/apache/beam/pull/12899#issuecomment-700950966


   Coming back to my review, I just want to merge:
   
   1. Having this checked in and running against even a single runner is an 
improvement and leaves the codebase in a consistent state. Adding any other 
runners that might be relevant can be separated.
   2. Seed job only runs against master so getting the job definition it 
checked in first and fixing it up is fine and helpful for any followup PR.


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