Skip to content
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

Python Dataflow not gracefully stopping #625

Open
haixuanTao opened this issue Aug 13, 2024 · 4 comments
Open

Python Dataflow not gracefully stopping #625

haixuanTao opened this issue Aug 13, 2024 · 4 comments
Labels
python Python API

Comments

@haixuanTao
Copy link
Collaborator

After trying to add a new feature in Python, I have notived that python dataflow does not gracefully end anymore.

Screencast.from.2024-08-13.07-05-38.webm

To reproduce

cd examples/python-dataflow
dora build dataflow.yml
dora start dataflow.yml
# Ctrl - C
@github-actions github-actions bot added the python Python API label Aug 13, 2024
@phil-opp
Copy link
Collaborator

I just tried it a few times with the main branch but I couldn't reproduce this issue.

But I noticed that the example is now combining pip-installed nodes and with the dora-cli in git since the node hub changes. This will lead to compatibiliy issues whenever we change the message format or do some other breaking change. The version check doesn't catch this because we only bump the version number on release. So there will be some deserialization issues or weird behavior in the future and we will probably experience CI failures because of this.

@haixuanTao
Copy link
Collaborator Author

pip always reinstall the latest git version of the nodes used as we're passing the source code in the graph.

Also, as we're preinstalling dora-rs within run.rs, it should also always follow the latest dora-rs pip version.

I'll try to make another example maybe easier to reproduce.

@haixuanTao
Copy link
Collaborator Author

FYI, I have changed the CI to use the cli and is now blocked by this: https://github.com/dora-rs/dora/actions/runs/11084418707/job/30799710937#step:9:1

it happens within the CLI but not when using the example runner

@haixuanTao
Copy link
Collaborator Author

haixuanTao commented Oct 2, 2024

Actually now that I think about it, the daemon stop gracefully because it doesn't react to a stop command.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
python Python API
Projects
None yet
Development

No branches or pull requests

2 participants