aleph

dm3 2017-06-06T16:11:45.306897Z

hmm, interesting

dm3 2017-06-06T16:12:05.314833Z

somehow fixed-thread-executor is not working as expected

dm3 2017-06-06T16:12:19.320438Z

i.e. as java.util.concurrent.Executors/newFixedThreadPool

dm3 2017-06-06T16:13:33.349736Z

@hansen-pansen try (e/fixed-thread-executor num {:control-period 100})

dm3 2017-06-06T16:13:57.359252Z

your test duration coincided with the default control period of the executor

dm3 2017-06-06T16:14:15.366033Z

that’s how often the instrumented executor will adjust the number of workers in the pool

dm3 2017-06-06T16:14:54.381486Z

there’s also :initial-thread-count parameter

hansen-pansen 2017-06-06T16:22:30.556996Z

Thank you very much! I'll give it a try and will also check with a snapshot build of manifold, in case this has been fixed.

hansen-pansen 2017-06-06T17:31:36.093831Z

@dm3 Again, thank you very much! Both :control-period and :initial-thread-count showed the expected behavior. I settled for :initial-thread-count, as this is more consistent of what I would expect from a fixed thread pool. As a side note, [manifold "0.1.7-alpha5"] behaves like 0.1.6.

dm3 2017-06-06T17:42:46.337979Z

I don’t think it should be fixed

dm3 2017-06-06T17:43:01.343478Z

you might create a PR with enhanced docs though

dm3 2017-06-06T17:43:21.350594Z

although

dm3 2017-06-06T17:43:45.359431Z

maybe fixed executor should have initial thread = num-threads from the start

2017-06-06T20:25:15.234738Z

hia

2017-06-06T20:25:59.236677Z

Im having trouble using d/zip with `d/chain