Skip to content

Conversation

@Jannis
Copy link
Contributor

@Jannis Jannis commented Apr 24, 2019

This fixes #889 by using a single Postgres connection for the chain
head update streams of all subgraph deployments.

@Jannis Jannis added bug Something isn't working chains/ethereum area/store labels Apr 24, 2019
@Jannis Jannis added this to the Q2 2019 milestone Apr 24, 2019
@Jannis Jannis requested a review from a team April 24, 2019 12:47
@Jannis Jannis self-assigned this Apr 24, 2019
@ghost ghost added the pending review label Apr 24, 2019
@Jannis Jannis force-pushed the jannis/shared-chain-head-update-listener branch 3 times, most recently from 9e8790e to 43dc61b Compare April 24, 2019 12:54
Copy link
Collaborator

@leoyvens leoyvens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, just a typo

This fixes #889 by using a single Postgres connection for the chain
head update streams of all subgraph deployments.
@Jannis Jannis force-pushed the jannis/shared-chain-head-update-listener branch from 43dc61b to 9be1279 Compare April 24, 2019 15:15
@Jannis Jannis merged commit 6ec3157 into master Apr 24, 2019
@ghost ghost removed the pending review label Apr 24, 2019
@Jannis Jannis deleted the jannis/shared-chain-head-update-listener branch April 24, 2019 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Dynamic data sources occasionally cause too many Postgres connections

3 participants