Page MenuHomePhabricator
Feed Advanced Search

Sat, Jan 16

sandyryza requested review of D5115: don't error on solid type annotations that don't resolve to dagster types.
Sat, Jan 16, 5:16 PM
sandyryza requested review of D6036: Make type_check_fn optional on DagsterType.
Sat, Jan 16, 5:14 PM
sandyryza updated the diff for D5794: pyspark smoke test example.

up

Sat, Jan 16, 5:01 PM

Fri, Jan 15

sandyryza added reviewers for D6033: include system framing message in user errors: alangenfeld, nate, yuhan.
Fri, Jan 15, 9:59 PM
sandyryza updated the diff for D6033: include system framing message in user errors.

up

Fri, Jan 15, 9:35 PM
sandyryza requested review of D6033: include system framing message in user errors.
Fri, Jan 15, 9:32 PM
sandyryza accepted D6032: [mypy][core] execution/context.
Fri, Jan 15, 8:44 PM
sandyryza added inline comments to D6032: [mypy][core] execution/context.
Fri, Jan 15, 6:50 PM
sandyryza added a comment to D5977: update nothing docs now that we have IO managers.

shouldn't this be modeled w/ IOManagers instead of Nothings?

Fri, Jan 15, 4:36 PM
sandyryza accepted D6020: [mypy][core] execution api.py and active.py.
Fri, Jan 15, 4:34 PM
sandyryza added a comment to D6020: [mypy][core] execution api.py and active.py.

Hell yeah!

Fri, Jan 15, 4:34 PM

Thu, Jan 14

sandyryza closed D6018: update partitions and backfills doc with screenshots of new UI.
Thu, Jan 14, 11:23 PM
sandyryza committed R1:d5f4588d49b7: update partitions and backfills doc with screenshots of new UI (authored by sandyryza).
update partitions and backfills doc with screenshots of new UI
Thu, Jan 14, 11:23 PM
sandyryza updated the diff for D6018: update partitions and backfills doc with screenshots of new UI.

up

Thu, Jan 14, 11:05 PM
sandyryza accepted D6011: Remove dangling references to SystemStorageDefinition.
Thu, Jan 14, 9:25 PM
sandyryza closed D6002: mark input and output metadata experimental.
Thu, Jan 14, 7:57 PM
sandyryza committed R1:8a7a39d42b42: mark input and output metadata experimental (authored by sandyryza).
mark input and output metadata experimental
Thu, Jan 14, 7:57 PM
sandyryza updated the diff for D6002: mark input and output metadata experimental.

up

Thu, Jan 14, 7:26 PM
sandyryza updated the diff for D6002: mark input and output metadata experimental.

up

Thu, Jan 14, 7:05 PM
sandyryza accepted D5981: Deprecate FileManager and FileHandle #3535.

Looks good!

Thu, Jan 14, 4:27 PM
sandyryza accepted D5984: context.log paths in default IO manager implementations #3531.

looks good!

Thu, Jan 14, 1:56 AM
sandyryza accepted D5990: keep AssetStoreOperationType in GRAVEYARD.
Thu, Jan 14, 1:26 AM

Wed, Jan 13

sandyryza closed D5980: mark root input managers as experimental.
Wed, Jan 13, 11:57 PM
sandyryza committed R1:35638fcedeb4: mark root input managers as experimental (authored by sandyryza).
mark root input managers as experimental
Wed, Jan 13, 11:57 PM
sandyryza closed D5967: improve IO manager docstrings.
Wed, Jan 13, 11:56 PM
sandyryza committed R1:b51b3ae5d3ae: improve IO manager docstrings (authored by sandyryza).
improve IO manager docstrings
Wed, Jan 13, 11:56 PM
sandyryza closed D5942: deprecate has_configurable_inputs, has_configurable_outputs, and has_config_entry.
Wed, Jan 13, 11:47 PM
sandyryza committed R1:075ad535ecc3: deprecate has_configurable_inputs, has_configurable_outputs, and… (authored by sandyryza).
deprecate has_configurable_inputs, has_configurable_outputs, and…
Wed, Jan 13, 11:47 PM
sandyryza requested review of D5980: mark root input managers as experimental.
Wed, Jan 13, 10:43 PM
sandyryza requested review of D5977: update nothing docs now that we have IO managers.
Wed, Jan 13, 10:22 PM
sandyryza updated the diff for D5942: deprecate has_configurable_inputs, has_configurable_outputs, and has_config_entry.

up

Wed, Jan 13, 10:22 PM
sandyryza updated the diff for D5967: improve IO manager docstrings.

up

Wed, Jan 13, 10:15 PM
sandyryza updated the diff for D5967: improve IO manager docstrings.

up

Wed, Jan 13, 10:14 PM
sandyryza accepted D5970: 4/ Clarify return types and interfaces.

I love it.

Wed, Jan 13, 9:11 PM
sandyryza accepted D5971: 5/ Upgrade moto.
Wed, Jan 13, 9:07 PM
sandyryza requested review of D5967: improve IO manager docstrings.
Wed, Jan 13, 7:58 PM
sandyryza accepted D5955: 0.10.0 API removals.

noice

Wed, Jan 13, 7:42 PM
sandyryza accepted D5963: 1/ Modernize airline_demo setup.

Looks good. Left a comment.

Wed, Jan 13, 7:41 PM
sandyryza closed D5918: manager_key -> io_manager_key.
Wed, Jan 13, 6:30 PM
sandyryza committed R1:936cd12bc414: manager_key -> io_manager_key (authored by sandyryza).
manager_key -> io_manager_key
Wed, Jan 13, 6:30 PM
sandyryza updated the diff for D5918: manager_key -> io_manager_key.

up

Wed, Jan 13, 6:29 PM
sandyryza requested review of D5942: deprecate has_configurable_inputs, has_configurable_outputs, and has_config_entry.
Wed, Jan 13, 5:27 PM
sandyryza updated the diff for D5918: manager_key -> io_manager_key.

up

Wed, Jan 13, 5:10 PM
sandyryza accepted D5948: [docs] mention IO manager docs in the migration guide #3455.
Wed, Jan 13, 5:05 PM
sandyryza accepted D5947: cleanup memoization copy_required_intermediates_for_execution.

That was quick! Love the big red portion.

Wed, Jan 13, 4:56 PM
sandyryza updated the diff for D5918: manager_key -> io_manager_key.

up

Wed, Jan 13, 4:49 PM
sandyryza closed D5874: test object manager composites behavior.
Wed, Jan 13, 1:51 AM
sandyryza committed R1:a630449903cb: test object manager composites behavior (authored by sandyryza).
test object manager composites behavior
Wed, Jan 13, 1:51 AM
sandyryza closed D5925: InputManager -> RootInputManager.
Wed, Jan 13, 1:50 AM
sandyryza committed R1:6e5c16256411: InputManager -> RootInputManager (authored by sandyryza).
InputManager -> RootInputManager
Wed, Jan 13, 1:50 AM
sandyryza accepted D5937: rename s3_io_manager to s3_pickled_io_manager, adls2_io_manager to adls2_pickled_io_manager, and gcs_io_manager to gcs_pickled_io_manager..

oops meant to hit accept

Wed, Jan 13, 1:49 AM
sandyryza updated the diff for D5874: test object manager composites behavior.

up

Wed, Jan 13, 1:16 AM
sandyryza updated the diff for D5925: InputManager -> RootInputManager.

up

Wed, Jan 13, 1:10 AM
sandyryza updated the summary of D5918: manager_key -> io_manager_key.
Wed, Jan 13, 1:05 AM
sandyryza updated the diff for D5925: InputManager -> RootInputManager.

up

Wed, Jan 13, 1:01 AM
sandyryza added inline comments to D5925: InputManager -> RootInputManager.
Wed, Jan 13, 1:01 AM

Tue, Jan 12

sandyryza updated subscribers of D5937: rename s3_io_manager to s3_pickled_io_manager, adls2_io_manager to adls2_pickled_io_manager, and gcs_io_manager to gcs_pickled_io_manager..

CC @schrockn in case he wants to weigh in.

Tue, Jan 12, 10:42 PM
sandyryza accepted D5474: ObjectStoreOperation/AssetStoreOperation -> HandledOutput/LoadedInput.
Tue, Jan 12, 10:28 PM
sandyryza accepted D5922: [docs] Eliminate 'intermediates' in docs #3516.

A couple small comments - otherwise looks great!

Tue, Jan 12, 10:17 PM
sandyryza requested review of D5918: manager_key -> io_manager_key.
Tue, Jan 12, 9:41 PM
sandyryza accepted D5474: ObjectStoreOperation/AssetStoreOperation -> HandledOutput/LoadedInput.

lgtm!

Tue, Jan 12, 9:39 PM
sandyryza closed D5924: unexpose output managers.
Tue, Jan 12, 9:37 PM
sandyryza committed R1:817b31a29425: unexpose output managers (authored by sandyryza).
unexpose output managers
Tue, Jan 12, 9:37 PM
sandyryza abandoned D3588: (value-resource-3) Add value_resource.
Tue, Jan 12, 9:35 PM
sandyryza closed D5852: mypy everything (almost).
Tue, Jan 12, 9:17 PM
sandyryza committed R1:4c721ea7b5c4: mypy everything (almost) (authored by sandyryza).
mypy everything (almost)
Tue, Jan 12, 9:17 PM
sandyryza requested review of D5924: unexpose output managers.
Tue, Jan 12, 7:48 PM
sandyryza requested review of D5925: InputManager -> RootInputManager.
Tue, Jan 12, 7:47 PM
sandyryza updated the diff for D5852: mypy everything (almost).

up

Tue, Jan 12, 7:27 PM
sandyryza closed D5843: root input manager.
Tue, Jan 12, 6:03 PM
sandyryza committed R1:ae09fcdb1a03: root input manager (authored by sandyryza).
root input manager
Tue, Jan 12, 6:03 PM
sandyryza updated the diff for D5843: root input manager.

up

Tue, Jan 12, 5:18 PM
sandyryza added a reviewer for D5780: eliminate Persistable: max.
Tue, Jan 12, 4:51 PM
sandyryza updated the summary of D5780: eliminate Persistable.
Tue, Jan 12, 4:51 PM
sandyryza requested changes to D5474: ObjectStoreOperation/AssetStoreOperation -> HandledOutput/LoadedInput.
Tue, Jan 12, 4:48 PM
sandyryza updated the diff for D5843: root input manager.

up

Tue, Jan 12, 4:40 PM
sandyryza added a comment to D5474: ObjectStoreOperation/AssetStoreOperation -> HandledOutput/LoadedInput.

Looks almost there!

Tue, Jan 12, 4:37 PM
sandyryza updated the diff for D5843: root input manager.

up

Tue, Jan 12, 3:57 PM

Sun, Jan 10

sandyryza updated the diff for D5794: pyspark smoke test example.

up

Sun, Jan 10, 7:58 PM
sandyryza updated the diff for D5794: pyspark smoke test example.

up

Sun, Jan 10, 7:26 PM
sandyryza updated the diff for D5843: root input manager.

up

Sun, Jan 10, 4:38 AM

Sat, Jan 9

sandyryza requested review of D5896: build dagster graph from dbt graph.
Sat, Jan 9, 6:30 PM
sandyryza closed D5882: object manager -> IO manager.
Sat, Jan 9, 6:15 PM
sandyryza committed R1:adbc20020757: object manager -> IO manager (authored by sandyryza).
object manager -> IO manager
Sat, Jan 9, 6:15 PM
sandyryza accepted D5832: fix import dagster intermediate storage deprecation warning #3478.

If I understand correctly, this will push the deprecation warning from pipeline definition time to pipeline run time? Not a big problem, but if there's an easy way to surface a warning earlier, that could be nice.

Sat, Jan 9, 5:48 PM
sandyryza updated the diff for D5882: object manager -> IO manager.

up

Sat, Jan 9, 5:32 PM
sandyryza updated the diff for D5882: object manager -> IO manager.

up

Sat, Jan 9, 4:45 PM
sandyryza updated the diff for D5882: object manager -> IO manager.

up

Sat, Jan 9, 2:09 AM
sandyryza added inline comments to D5721: DynamicOutput DSL map approach.
Sat, Jan 9, 1:58 AM
sandyryza updated the diff for D5882: object manager -> IO manager.

up

Sat, Jan 9, 1:38 AM
sandyryza added inline comments to D5829: Modernize dagstermill file handling.
Sat, Jan 9, 1:04 AM
sandyryza added a comment to D5882: object manager -> IO manager.

@yuhan thanks for catching those ones I missed. I went through and did another pass.. I think I got everything this time.

Sat, Jan 9, 12:57 AM

Fri, Jan 8

sandyryza updated the diff for D5882: object manager -> IO manager.

up

Fri, Jan 8, 11:40 PM
sandyryza updated the diff for D5882: object manager -> IO manager.

up

Fri, Jan 8, 10:09 PM
sandyryza updated the diff for D5843: root input manager.

up

Fri, Jan 8, 9:01 PM
sandyryza requested review of D5882: object manager -> IO manager.
Fri, Jan 8, 7:44 PM
sandyryza updated the diff for D5843: root input manager.

up

Fri, Jan 8, 6:57 PM
sandyryza updated the diff for D5843: root input manager.

up

Fri, Jan 8, 6:53 PM
sandyryza retitled D5843: root input manager from RFC: root input manager to root input manager.
Fri, Jan 8, 6:36 PM
sandyryza updated the diff for D5843: root input manager.

up

Fri, Jan 8, 6:34 PM