added param to default fxn
can we add type checking here? i don't see these two params being checked in the stack.
do we assume versions will always be provided in memoized run? if so, it may be better to error out instead of a silent None. if not, can we comment some cases which will hit this path?
would it be worth making this attribute public? i havent read through D4407 tho. it's your call.
good call, will do.
I don't think we should always assume that versions will be provided. The way I was thinking about it was that None provides a mechanism for specifying that a specific portion of a pipeline should re-run every time. The reason I think this would be okay is that even though the None is technically silent, the version information would presumably be populated in dagit/cli, and the user would still be able to see that this loading code is running every time, for example.
it's not used outside of the computation of version for DagsterTypeLoader, but I could see it being used for populating version information later, so I think it might be worth making it public.