fixup! add TODO entry to investigate pg promise branch

BridgeAR · BridgeAR · commit 458e04917c23 · 2025-05-18T20:37:14.000+02:00
diff --git a/packages/datadog-instrumentations/src/pg.js b/packages/datadog-instrumentations/src/pg.js
@@ -132,6 +132,9 @@ function wrapQuery (query) {
           .once(errorMonitor, finish)
           .once('end', (res) => finish(null, res))
       } else {
+        // TODO: This code is never reached in our tests.
+        // Internally, pg always uses callbacks or streams, even for promise based queries.
+        // Investigate if this code should just be removed.
         newQuery.then((res) => finish(null, res), finish)
       }
 

Original file line number	Diff line number	Diff line change
`@@ -132,6 +132,9 @@ function wrapQuery (query) {`
`132`	`132`	`.once(errorMonitor, finish)`
`133`	`133`	`.once('end', (res) => finish(null, res))`
`134`	`134`	`} else {`
	`135`	`+ // TODO: This code is never reached in our tests.`
	`136`	`+ // Internally, pg always uses callbacks or streams, even for promise based queries.`
	`137`	`+ // Investigate if this code should just be removed.`
`135`	`138`	`newQuery.then((res) => finish(null, res), finish)`
`136`	`139`	`}`
`137`	`140`