Daniel Suissa
12/13/2021, 3:34 PMschrockn
12/13/2021, 3:42 PMDaniel Suissa
12/13/2021, 3:57 PMschrockn
12/13/2021, 6:45 PMRoei Jacobovich
12/13/2021, 7:38 PMReloadWorkspaceMutation
.
Is there a better way we don’t know about? Without restarting the API gRPC process?
Thank you so much for your help.daniel
12/13/2021, 7:55 PMRoei Jacobovich
12/13/2021, 10:07 PMdaniel
12/13/2021, 10:08 PMRoei Jacobovich
12/13/2021, 10:14 PMtest_reload_repository_location.py
(https://github.com/dagster-io/dagster/blob/ca32cefd15849122728c72bf91421b3b234c1a2[…]_tests/host_representation_tests/test_custom_repository_data.py) and created the same objects and it worked. The only difference from my code is that the “internal” functions (like define_foo_pipeline()
on line 43) are using a member of the class, and I tried to invoke it without any changing parameter from the class itself. It’s reasonable - calling that function each time would cost a lot of computation resources. Could that be a proper explanation?daniel
12/13/2021, 10:18 PMRoei Jacobovich
12/13/2021, 10:35 PMdef define_foo_pipeline(num_calls)
must have a parameter (as num_calls) in order to actually being called again, even if the function itself doing something else each time (like calling randint() inside the function instead of getting the random number from outside) 🙂daniel
12/13/2021, 10:37 PMRoei Jacobovich
12/13/2021, 10:39 PMget_all_pipelines()
is the abstract method and not get_all_jobs()
. Maybe worth updating. Thanks a lot Daniel 🙂daniel
12/13/2021, 10:41 PMRoei Jacobovich
12/14/2021, 11:12 AMreloadRepositoryLocation
GraphQL mutation after each change of my “source” files (YAML files in your example).
2. Taking (1) and using external cronjob to execute the mutation every X seconds.
3. Patch the gRPC server to return random UUID each time. That way, the gRPC watcher thread on Dagster itself would invoke on_updated
event and call eventually the underlying function of reloadRepositoryLocation
Is there another way? Is the 3rd option a valid feature for Dagster? I could make a PR for that as the implementation is not complicated.
In that case I also need to figure out how to control the watch_interval
of that thread with proper config.
Thanks a lot again 🙂daniel
12/14/2021, 12:26 PMRoei Jacobovich
12/14/2021, 12:34 PM