Should this be bytes type?

Consider using the condensed NamedTuple syntax if users won't be instantiating this

class CapturedLogMetadata(NamedTuple):
    location: Optional[str]
    download_url: Optional[str]

Add bool return type annotation?

I'm not 100% grasping why this should go on the instance log manager. If I, for example, decide to implement an InstanceLogManager that stores the output streams to S3, how would I implement this method?

For the in-process logging proposal [1], here's the DAG I'm imagining:

In that proposal, all logs go through the Python logging, so there's no choice about how to route them aside from the set of handlers that are included in the instance-level Python logging config.

[1] https://elementl.quip.com/BtFRAY5cWKWq/Logging-Requirements-Proposal

Yep, good idea to change to bytes.

Yeah, we don't need this if we're going through the Python logging route, will remove

+ comments

not for this diff - but we should rename these for clarity

• report_and_log_from_xxx
• log_and_report_from_xxx
• some better name

these first two log directly, and this last case doesn't, should probably report directly here instead of doing it at the callsite [1]

mypy & comment

kwargs for clarity (mostly for what is None)

[1]

mypy

from the compute steps of pipeline solids

this should be changed right?