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 Oct 2, 2024
1 parent 4521b39 commit 681425a
Show file tree
Hide file tree
Showing 3 changed files with 215 additions and 66 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 @@ -161,7 +161,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 681425a

Please sign in to comment.