Skip to content

Add pgPool 'error' handler#26

Merged
benjie merged 1 commit intographile:masterfrom
madflow:pg-error-handler
Sep 13, 2019
Merged

Add pgPool 'error' handler#26
benjie merged 1 commit intographile:masterfrom
madflow:pg-error-handler

Conversation

@madflow
Copy link
Copy Markdown
Contributor

@madflow madflow commented Sep 13, 2019

1|worker  | (node:64357) UnhandledPromiseRejectionWarning: Error: Client has encountered a connection error and is not queryable
1|worker  |     at process.nextTick (.../node_modules/pg/lib/client.js:521:25)
1|worker  |     at process._tickCallback (internal/process/next_tick.js:61:11)
1|worker  | (node:64357) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
1|worker  | (node:64357) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code
  • Maybe there is more work to do here...

Comment thread src/runner.ts
@madflow madflow force-pushed the pg-error-handler branch 2 times, most recently from 938628a to 4cda806 Compare September 13, 2019 14:12
Copy link
Copy Markdown
Member

@benjie benjie left a comment

Choose a reason for hiding this comment

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

Great; thanks for this! 🙏

@benjie benjie changed the title pgPool error handler #22 Add pgPool 'error' handler Sep 13, 2019
@benjie benjie merged commit c3dfaf4 into graphile:master Sep 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants