19913d85-f662-4518-a9a1-7adc1793dfd4.png){kind=small}
Channels
announcements
dagster-airbyte
dagster-airflow
dagster-bigquery
dagster-cloud
dagster-cube
dagster-dask
dagster-dbt
dagster-de
dagster-ecs
dagster-feedback
dagster-kubernetes
dagster-machine-learning
dagster-noteable
dagster-releases
dagster-serverless
dagster-showcase
dagster-snowflake
dagster-spatial
dagster-support
dagster-wandb
dagstereo
data-platform-design
events
faq-read-me-before-posting
gigs-freelance
github-discussions
introductions
jobs
random
tools
豆瓣酱帮
Title
x
Xu Zhang
08/24/2021, 9:14 PMteam, i made a module like this
from dagster import graph, op
@op
def get_name():
return "world"
@op
def hello(context, name):
<http://context.log.info|context.log.info>(f"Hello, {name}!")
@graph
def hello_graph():
hello(get_name())s
sandy
08/24/2021, 9:20 PM@Xu Zhang what version are you using? I believe @chris recently merged a fix for this
x
Xu Zhang
08/24/2021, 9:20 PM0.12.7
c
chris
08/24/2021, 9:23 PMFix didn't go out for last release - will be up in 0.12.8
A hotfix would be doing making a variable hello_graph.to_job
x
Xu Zhang
08/24/2021, 9:24 PMdo you have the PR? i can make a patch locally
@chris @sandy would you mind to point me the PR? i can make the patch locally
c
chris
08/24/2021, 9:27 PMx
Xu Zhang
08/24/2021, 9:30 PMthank you
@sandy @chris i tried to replace local code with the code from the PR, and im getting this
File "/export/content/lid/apps/meeseeks-backend-dagster-grpc-server/i001/bin/meeseeks-backend.pyz/__main__.py", line 3, in <module>
File "/export/content/lid/apps/meeseeks-backend-dagster-grpc-server/i001/bin/meeseeks-backend.pyz/_bootstrap/__init__.py", line 241, in bootstrap
File "/export/content/lid/apps/meeseeks-backend-dagster-grpc-server/i001/bin/meeseeks-backend.pyz/_bootstrap/__init__.py", line 36, in run
File "/export/content/lid/apps/meeseeks-backend-dagster-grpc-server/i001/libexec/meeseeks-backend_df1eb0fe398b05c6048ea7ba28518807d871a120d267ac6b44168e1338bb5111/site-packages/linkedin/dagster/run_dagster_grpc_server.py", line 38, in run
start_dagster_grpc_server(fixed_server_id)
File "/export/content/lid/apps/meeseeks-backend-dagster-grpc-server/i001/libexec/meeseeks-backend_df1eb0fe398b05c6048ea7ba28518807d871a120d267ac6b44168e1338bb5111/site-packages/linkedin/dagster/utils.py", line 221, in start_dagster_grpc_server
fixed_server_id=fixed_server_id,
File "/export/content/lid/apps/meeseeks-backend-dagster-grpc-server/i001/libexec/meeseeks-backend_df1eb0fe398b05c6048ea7ba28518807d871a120d267ac6b44168e1338bb5111/site-packages/dagster/grpc/server.py", line 818, in __init__
fixed_server_id=fixed_server_id,
File "/export/content/lid/apps/meeseeks-backend-dagster-grpc-server/i001/libexec/meeseeks-backend_df1eb0fe398b05c6048ea7ba28518807d871a120d267ac6b44168e1338bb5111/site-packages/dagster/grpc/server.py", line 204, in __init__
self._repository_symbols_and_code_pointers.load()
File "/export/content/lid/apps/meeseeks-backend-dagster-grpc-server/i001/libexec/meeseeks-backend_df1eb0fe398b05c6048ea7ba28518807d871a120d267ac6b44168e1338bb5111/site-packages/dagster/grpc/server.py", line 90, in load
self._loadable_target_origin
File "/export/content/lid/apps/meeseeks-backend-dagster-grpc-server/i001/libexec/meeseeks-backend_df1eb0fe398b05c6048ea7ba28518807d871a120d267ac6b44168e1338bb5111/site-packages/dagster/grpc/server.py", line 121, in load_loadable_repository_symbols
for loadable_target in loadable_targets
File "/export/content/lid/apps/meeseeks-backend-dagster-grpc-server/i001/libexec/meeseeks-backend_df1eb0fe398b05c6048ea7ba28518807d871a120d267ac6b44168e1338bb5111/site-packages/dagster/grpc/server.py", line 121, in <listcomp>
for loadable_target in loadable_targets
AttributeError: 'NoneType' object has no attribute 'name'when will 0.12.8 come out?
s
sandy
08/24/2021, 11:07 PMI'll leave @chris to comment on the error - 0.12.8 will come out on Thursday
x
Xu Zhang
08/24/2021, 11:11 PMack
i’ve reverted the patch. im adding a
@repository
def repo():
return [hello_graph]👍 1
c
chris
08/25/2021, 3:14 AMHey @Xu Zhang, I tried the example that you posted on master, and it works correctly. I suspect that the error you ran into when patching could be either some missed history between 0.12.7 and the patch, or something went wrong in applying.
I'd be happy to help you debug if it's blocking
x
Xu Zhang
08/25/2021, 4:13 AMit is ok — i reverted my patch, and use the solution mentioned inside the original issue, which is to add a repository. the thing that is blocking me is the
OSError: [Errno 30] Read-only file system: