Page MenuHomePhabricator

Recent Activity

Today

nate added a comment to D4162: fix celery-k8s code example.

yes, we should make this a literalinclude instead of a code-block - mind taking that on this diff? it'll ensure we don't end up in this state again

Wed, Aug 12, 3:21 PM
dgibson requested review of D4163: RFC: Turn on auto-formatting for string literals in black.
Wed, Aug 12, 3:00 PM
Harbormaster failed remote builds in B16782: Diff 20463 for D4021: heterogeneous compute lakehouse example!
Wed, Aug 12, 2:48 PM
johann requested review of D4162: fix celery-k8s code example.
Wed, Aug 12, 2:40 PM
cdecarolis updated the diff for D4021: heterogeneous compute lakehouse example.

Renamed example to multi_type_lakehosue to reflect its function more accurately. Added to index. Made it so that example is understandable without first reading simple_lakehouse example.

Wed, Aug 12, 2:36 PM
dgibson updated the diff for D4151: Throw from sync_get_external_execution_plan rather than relying on callsites to check error data.

up, quotes

Wed, Aug 12, 2:22 PM
Harbormaster failed remote builds in B16777: Diff 20458 for D4153: remove deprecated env param from cli pipeline commands!
Wed, Aug 12, 2:20 PM
johann added a reviewer for D4140: test subset execution in celery-k8s: yuhan.
Wed, Aug 12, 2:14 PM
dgibson updated the diff for D4156: Divest 'pipeline execute' command from workspace load path / being a host process.

up

Wed, Aug 12, 1:51 PM
dgibson updated the diff for D4153: remove deprecated env param from cli pipeline commands.

up

Wed, Aug 12, 1:48 PM
dgibson updated the diff for D4138: Replace --host CLI param with --grpc_host to avoid conflicting with dagit --host param.

up

Wed, Aug 12, 1:43 PM
dgibson updated the diff for D4137: Add cleanup thread to grpc server.

up

Wed, Aug 12, 1:41 PM
dgibson updated the diff for D4136: Re-enable CLI execute/launch/backfill commands on windows (by eliminating sources of tempdir contention).

up

Wed, Aug 12, 1:41 PM
yuhan added reviewers for D4160: Create separate Library section in API docs sidebar #2542: nate, sashank, sandyryza.
Wed, Aug 12, 4:55 AM
yuhan added reviewers for D4159: Move docs/next/src/pages/docs/deploying/docker to examples: sandyryza, sashank, nate.
Wed, Aug 12, 4:55 AM
yuhan requested review of D4160: Create separate Library section in API docs sidebar #2542.
Wed, Aug 12, 4:51 AM
yuhan requested review of D4161: set up lint for docs.
Wed, Aug 12, 4:47 AM
yuhan requested review of D4159: Move docs/next/src/pages/docs/deploying/docker to examples.
Wed, Aug 12, 4:16 AM
dgibson requested review of D4156: Divest 'pipeline execute' command from workspace load path / being a host process.
Wed, Aug 12, 3:26 AM
schrockn added inline comments to D4151: Throw from sync_get_external_execution_plan rather than relying on callsites to check error data.
Wed, Aug 12, 12:41 AM
sandyryza accepted D4151: Throw from sync_get_external_execution_plan rather than relying on callsites to check error data.

LGTM!

Wed, Aug 12, 12:04 AM

Yesterday

sandyryza committed R1:9ff2f157f3d2: nicer error on failure to load repository in external process (authored by sandyryza).
nicer error on failure to load repository in external process
Tue, Aug 11, 11:38 PM
sandyryza closed D4087: nicer error on failure to load repository in external process.
Tue, Aug 11, 11:38 PM
sandyryza updated the diff for D4087: nicer error on failure to load repository in external process.

up

Tue, Aug 11, 10:37 PM
yuhan updated subscribers of D4153: remove deprecated env param from cli pipeline commands.

oh i just realized that @sashank had the diff D4002 out for a while which does the same thing. maybe coordinate with him before landing it.

Tue, Aug 11, 10:18 PM
Harbormaster failed remote builds in B16767: Diff 20447 for D4087: nicer error on failure to load repository in external process!
Tue, Aug 11, 10:01 PM
sandyryza updated the diff for D4087: nicer error on failure to load repository in external process.

up

Tue, Aug 11, 9:38 PM
sandyryza added inline comments to D4087: nicer error on failure to load repository in external process.
Tue, Aug 11, 9:33 PM
johann committed R1:a4b391459f3c: Pass tags to PipelineSubsetDef (authored by johann).
Pass tags to PipelineSubsetDef
Tue, Aug 11, 9:15 PM
johann closed D4152: Pass tags to PipelineSubsetDef.
Tue, Aug 11, 9:14 PM
yuhan added inline comments to D4152: Pass tags to PipelineSubsetDef.
Tue, Aug 11, 9:13 PM
dgibson edited reviewers for D4136: Re-enable CLI execute/launch/backfill commands on windows (by eliminating sources of tempdir contention), added: sashank; removed: schrockn.
Tue, Aug 11, 8:50 PM
dpeng817 <cdguitar817@gmail.com> committed R1:29a9356b6232: renamed InMemoryIntermediatesManager and IntermediateStoreIntermediatesManager (authored by dpeng817 <christopherdecarolis@christokmacbook.myfiosgateway.com>).
renamed InMemoryIntermediatesManager and IntermediateStoreIntermediatesManager
Tue, Aug 11, 8:50 PM
dpeng817 <cdguitar817@gmail.com> closed D4070: renamed InMemoryIntermediatesManager and IntermediateStoreIntermediatesManager.
Tue, Aug 11, 8:50 PM
dgibson updated the diff for D4137: Add cleanup thread to grpc server.

fix default run launcher test as well

Tue, Aug 11, 8:41 PM
nate accepted D4158: updage java debian version in bk image.

cool this looks good to me. assume you're on top of this, but you'll have to also update the published integration image to "rebase" on this updated image - I think this is documented in the readme? lmk if you need a hand

Tue, Aug 11, 8:41 PM
dgibson updated the summary of D4145: Refactor CLI tests to pass in contexts for managed and unmanaged GRPC servers.
Tue, Aug 11, 8:34 PM
yuhan accepted D4152: Pass tags to PipelineSubsetDef.

passing tags looks good to me

Tue, Aug 11, 8:29 PM
sandyryza accepted D4157: better configured error messages & docs.

This looks great.

Tue, Aug 11, 8:02 PM
aj.nadel added inline comments to D4157: better configured error messages & docs.
Tue, Aug 11, 7:59 PM