HomeElementl

[demo pipelines] remove slack on success hook

Description

[demo pipelines] remove slack on success hook

Summary:
Open to pushback on this. A few reasons here:

  • The implementation includes hardcoded values from the Elementl dogfooding setup.
  • This is a weakly-held opinion, but I think success hooks are kind of an anti-pattern. Any success hook can be written as a regular solid, and having two recommended ways of doing the same thing is awkward.
  • What the hook is doing in this case is unrealistic. I don't think I would want a slack message every hour reporting that I pulled some data into an S3 table.

Test Plan: bk

Reviewers: owen

Reviewed By: owen

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

Details

Provenance
sandyryzaAuthored on Aug 4 2021, 3:26 PM
Reviewer
owen
Differential Revision
D9205: [demo pipelines] remove slack on success hook
Parents
R1:fc435395a8c6: [dagit] Lazy-load partition set info for Schedules
Branches
Unknown
Tags
Unknown