Page MenuHomePhabricator

Remove lambda_solid from tutorial and examples

Authored by schrockn on Aug 22 2019, 6:53 PM.


Group Reviewers
Restricted Project
R1:bb85c68508b4: Remove lambda_solid from tutorial and examples

This has always been a bit problematic and now we have evidence that it
is actively causing confusion for folks getting up to speed. Post py3 type
annotations this is actually more reasonable to deal with, and it only
requires having a placeholder variable for the context variable.

Test Plan

Look at docs

Diff Detail

R1 dagster
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

schrockn created this revision.Aug 22 2019, 6:53 PM

Yeah agree that it's not a slam dunk

I'm leaning yes. One anecdote is from a recent call watching some one want to log something and having to import a new thing, change the call sig, then add the context.log to do it.


  • less concepts to learn


  • unused context arg

Curious to hear opinions from others.

prha added a subscriber: prha.Aug 26 2019, 10:47 PM

I prefer removing lambda_solid... I don't think the extra parameter is that crazy / unusual.

schrockn updated this revision to Diff 4043.Aug 27 2019, 9:39 PM
schrockn retitled this revision from RFC: Remove lambda_solid from tutorial to Remove lambda_solid from tutorial and examples.


alangenfeld accepted this revision.Aug 28 2019, 8:28 PM

fire at will

This revision is now accepted and ready to land.Aug 28 2019, 8:28 PM
This revision was automatically updated to reflect the committed changes.