HomePhabricator

bypass apollo for compute log subscriptions

Authored by prha on Wed, Oct 2, 10:43 PM.

Description

bypass apollo for compute log subscriptions

Summary:
This diff achieves the following:

  • Strips out the Apollo client / query / subscription
  • Adds the ComputeLogProvider which handles subscription messages, merging them before passing to the presentation component
  • Moves ComputeLogContent and other presentation only components to a separate file
  • Adds graphql fragments to the presentation layer
  • Uses generated fragment types instead of bespoke hand-written types for props

Test Plan: bk, ran stdout_spew_pipeline, saw streaming logs

Reviewers: #ft, bengotow, alangenfeld

Reviewed By: #ft, bengotow, alangenfeld

Subscribers: alangenfeld

Differential Revision: https://dagster.phacility.com/D1172

Details

Committed
prhaThu, Oct 3, 6:01 PM
Reviewer
Restricted Project
Differential Revision
D1172: bypass apollo for compute log subscriptions
Parents
R1:59be04d422ab: [cli] dagster instance info
Branches
Unknown
Tags
Unknown