Skip to content

Commit

Permalink
[dagster-pipes] add ThreadedPipesMessageReader with extra runtime params
Browse files Browse the repository at this point in the history
  • Loading branch information
danielgafni committed Sep 30, 2024
1 parent a993cd9 commit 7f7242b
Show file tree
Hide file tree
Showing 3 changed files with 216 additions and 64 deletions.
4 changes: 3 additions & 1 deletion python_modules/dagster/dagster/_core/pipes/context.py
Original file line number Diff line number Diff line change
Expand Up @@ -140,7 +140,9 @@ def _resolve_metadata_value(
# Type ignores because we currently validate in individual handlers
def handle_message(self, message: PipesMessage) -> None:
if self._received_closed_msg:
self._context.log.warn(f"[pipes] unexpected message received after closed: `{message}`")
self._context.log.warning(
f"[pipes] unexpected message received after closed: `{message}`"
)

method = cast(Method, message["method"])
if method == "opened":
Expand Down
Loading

0 comments on commit 7f7242b

Please sign in to comment.