-
Notifications
You must be signed in to change notification settings - Fork 910
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Can we remove ParallelRunner
?
#4291
Comments
We discussed that there is indeed evidence that some people at least try to use this runner, but because it's so broken, they're rarely successful or they have to battle workarounds. We agreed to do some research to understand the underlying issues and see if the |
I'm using |
Thanks @yury-fedotov, that's good to know. Does it work well for your needs? Have you been hit by any issues? |
Even I have used it in my previous organisation and it is most likely still being used. If I remember correctly, the telemetry data won't show up from that organisation. |
Unsure if this is being recorded in the telemetry at all. The prompt for this issue was a Slack thread that isn't necessarily representative. I'm mostly ready to say that we should fix the issues instead of deprecating the runner. |
Yes, works well, and I haven't encountered issues |
Description
Check telemetry to find out how much
ParallelRunner
is used and whether it would be possible to deprecate and remove it.Context
ParellelRunner
is the most problematic among our runners:ParallelRunner
#4078ParallelRunner
raisesAttributeError: The following data sets cannot be used by multiprocessing...
on datasets not involved in--pipeline
being run #3804ParallelRunner
#3674And there's already a bit of evidence it's not used that much: https://linen-slack.kedro.org/t/16663577/do-you-use-kedro-run-runner-parallerunner-to-speed-up-your-p#99abccb0-7970-4a65-8fad-85fd22681beb
The text was updated successfully, but these errors were encountered: