Clean up `._transport` error-case comment
Expand and clarify the comment for the default `case _` block in the `.send()` error matcher, noting that we console-error and raise-thru for unexpected disconnect conditions. (this patch was suggested by copilot in, https://github.com/goodboy/tractor/pull/411) (this commit msg was generated in some part by [`claude-code`][claude-code-gh]) [claude-code-gh]: https://github.com/anthropics/claude-codetpt_tolerance
parent
9470815f5a
commit
d0b92bbeba
|
|
@ -485,9 +485,11 @@ class MsgpackTransport(MsgTransport):
|
||||||
# await tractor.devx._trace.maybe_pause_bp()
|
# await tractor.devx._trace.maybe_pause_bp()
|
||||||
raise tpt_closed from trans_err
|
raise tpt_closed from trans_err
|
||||||
|
|
||||||
# unless the disconnect condition falls under "a
|
# XXX, unless the disconnect condition falls
|
||||||
# normal operation breakage" we usualy console warn
|
# under "a normal/expected operating breakage"
|
||||||
# about it.
|
# (per the `trans_err_msg` guards in the cases
|
||||||
|
# above) we usualy console-error about it and
|
||||||
|
# raise-thru. about it.
|
||||||
case _:
|
case _:
|
||||||
log.exception(
|
log.exception(
|
||||||
f'{tpt_name} layer failed pre-send ??\n'
|
f'{tpt_name} layer failed pre-send ??\n'
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue