youngoli commented on pull request #14134: URL: https://github.com/apache/beam/pull/14134#issuecomment-790264150
I'm giving `mustRunAfter` a try in #14144, and while I'm still running Precommits to see if the flake is really fixed, I'm not sold on the approach. If I could figure out how to add all the `mustRunAfter` calls in the definition of the Precommit task so they were all in one place and only applied locally to that task, then I'd be ok with it. But I couldn't figure that out, and instead had to spread out all the calls over six build.gradle files which is a bit too spaghetti code for me (enforcing ordering for seemingly unrelated tasks in multiple files to work around a mysterious flake). I think this change still seems preferable to that. Thoughts? ---------------------------------------------------------------- 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]
