HomeElementl

[dagster-postgres] [easy] Quick fix in listen/notify logic

Description

[dagster-postgres] [easy] Quick fix in listen/notify logic

Summary:
There was an unclear isinstance call on the notifications yielded from await_pg_notifications as a NoneType check - a better pattern is the is not None check.

I think the reason this was there in the first place is that older versions of the await_pg_notifications code would occasionally yield a integer signal.

Test Plan: bk

Reviewers: rexledesma

Reviewed By: rexledesma

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

Details

Provenance
Sidharth Menon <menon.sid.k@gmail.com>Authored on Apr 26 2021, 6:56 PM
Reviewer
rexledesma
Differential Revision
D7602: [dagster-postgres] [easy] Quick fix in listen/notify logic
Parents
R1:883df03812d3: Remove get_test_helm_namespace
Branches
Unknown
Tags
Unknown