Only bootstrap the zinc worker pool if there is work to do
Review Request #3559 - Created March 10, 2016 and submitted
|Nick Howard (Twitter)|
|benjyw, gmalmquist, jsirois, stuhood|
Currently, jvm compile will always create a worker pool. This updates JvmCompile so that it only bootstraps the pool if there are invalid targets. It also renames compile_chunk to do_compile as compile_chunk implies chunking behavior that is no longer used.
Wrote an integration test that checks for the presence and abscence of the workunit label in output and made it pass. CI away on the PR.
Remove timing improvement assertions. Stu pointed out on the PR that the time is likely in hashing which is right. That said, I think this is still an improvement because it reduces some console output noise and removes some weird shared state.