concurrent.futures.Executor compliance

[jeff-dh]

@evnchn

TODO(?):

• not thread safe
• map: timeout & chunksize are not implemented
• ....?

[evnchn]

• submit
• map
• shutdown
• _enter_
• _exit_

So yeah, basically context manager support is missing. Other than that, nothing more is expected from the concurrent.futures.Executor

[jeff-dh]

aio_process_pool/tests/pool_test.py

Lines 43 to 50 in 64fe1b2

	def test_context_manager():
	with Executor() as exe:
	assert exe.map(fib, range(30)) == first_30_fib_numbers
	@pytest.mark.asyncio
	async def test_async_context_manager():
	with Executor() as exe:
	assert await exe.map_async(fib, range(30)) == first_30_fib_numbers

would you try to run the test on windows?

[jeff-dh]

aio_process_pool/src/aio_process_pool/executor.py

Line 9 in 64fe1b2

class Executor(concurrent.futures.Executor):

does the trick, it has a default implementation that works

[evnchn]

Later. I'm sleeping soon.

I like the one-liner fix.

[evnchn]

@jeff-dh works on Windows

Remind me when I back at home, that I setup like a RDP server explicitly for you and any dev in NiceGUI who want to online-borrow an (admittedly slow) Windows machine