hundreds-father-404
03/28/2020, 5:14 AMscheduler.ExecutionError
when calling ./v2 --changed-since=master fmt
that only includes the very long Computing Select
clause and then <None>
Has anyone else seen this? I started noticing it yesterday.enough-analyst-54434
03/29/2020, 7:48 PM--changed-since
is not required.
$ git bisect bad
6db9ab4becf5b520c12db297b10c9e6e78aa994e is the first bad commit
commit 6db9ab4becf5b520c12db297b10c9e6e78aa994e
Author: Henry Fuller <<mailto:hfuller@twitter.com|hfuller@twitter.com>>
Date: Thu Mar 26 12:01:39 2020 -0700
Add notify fs watcher to engine. (#9318)
...
@early-needle-54791 fyi. I'll file an issue and see if there is not a workaround so we don't need a full revert.enough-analyst-54434
03/29/2020, 7:57 PMearly-needle-54791
03/29/2020, 8:04 PMearly-needle-54791
03/29/2020, 8:05 PMenough-analyst-54434
03/29/2020, 8:08 PMenough-analyst-54434
03/29/2020, 8:08 PMenough-analyst-54434
03/29/2020, 8:09 PMearly-needle-54791
03/29/2020, 11:29 PMearly-needle-54791
03/29/2020, 11:30 PMI’m not sure multiple times has anything to do with it unless you are.As eric pointed out, and is visible in your bug report, the black tool is being run multiple times, when it should be run once. To me this indicates that the fs watcher is invalidating nodes either while they are running or right after, causing them to run again presumably. I’m not sure exactly where the code that determines why / when to run a node is but I plan to pursue that. If the build graph must have every node in a completed state for the build to succeed, that would be where the problem is.