Tim Wojtulewicz

Results 302 comments of Tim Wojtulewicz

> I wonder if we should consider switching out our dict implementation with something like Google::sparse/dense_hash_map ~~I spent some time a while ago investigating other map implementations and it generally...

`btest` also functions correctly on Windows now. We should go through the failing tests on that platform and fix them.

Just for reference, https://github.com/zeek/zeek/pull/2981 adds running unit tests on CI

Would it be enough to just mark that iterator robust? I can't remember if that ends up preventing all removals as well.

`doc.zeekygen.package_index` was the one that I had seen timeout before, but I never dug into exactly what it was doing that was taking so long. Unfortunately because Cirrus kills the...

> Maybe we should add a timeout to btest itself so it can fail cleanly ahead of Cirrus' timeout. None of our tests should take more than a couple of...

Another possible timeout point is the joins on processes from `mutliprocess`.

I'm frequently seeing timeouts for the cluster-testing build: https://cirrus-ci.com/task/5393991806484480

Do you have a link to documentation for replacing these?

This looks like it's going to require some rework of how we find/add the library targets to the main Zeek target. I'm going to hold off doing these changes until...