diff options
author | Benjamin Peterson <bp@benjamin.pe> | 2017-08-11 13:34:52 +0200 |
---|---|---|
committer | Marcel Hlopko <hlopko@google.com> | 2017-08-11 15:43:45 +0200 |
commit | 740cd903ee7666a7ddef512c65b8455bde46abae (patch) | |
tree | 997d58e3a891c4f14d240aa13cb7d94d9d93d249 /src/package-bazel.sh | |
parent | a3acb6630b7e0693b42781b8ec7ff6860efc7c3f (diff) |
Refactor persistent workers to use SpawnRunner.
Change the persistent worker spawn strategy to extend
AbstractSpawnStrategy and put the actual logic into
WorkerSpawnRunner. WorkerTestStrategy is unaffected.
I had to extend SpawnPolicy with a speculating() method. Persistent
workers need to know if speculation is happening in order to require
sandboxing.
Additionally, I added java_test rules for the local runner tests and
worker tests. See https://github.com/bazelbuild/bazel/issues/3481.
NOTE: ulfjack@ made some changes to this change before merging:
- changed Reporter to EventHandler; added TODO about its usage
- reverted non-semantic indentation change in AbstractSpawnStrategy
- reverted a non-semantic indentation change in WorkerSpawnRunner
- updated some internal classes to match
- removed catch IOException in WorkerSpawnRunner in some cases,
removed verboseFailures flag from WorkerSpawnRunner, updated callers
- disable some tests on Windows; we were previously not running them,
now that we do, they fail :-(
Change-Id: I207b3938f0dc84d374ab052d5030020886451d47
PiperOrigin-RevId: 164965398
Diffstat (limited to 'src/package-bazel.sh')
0 files changed, 0 insertions, 0 deletions