Skip to content

Commit 31ee759

Browse files
committed
Deprecate pg.* singleton methods
1 parent 61921aa commit 31ee759

File tree

4 files changed

+255
-0
lines changed

4 files changed

+255
-0
lines changed

lib/index.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ var defaults = require('./defaults');
1313
var Connection = require('./connection');
1414
var ConnectionParameters = require('./connection-parameters');
1515
var poolFactory = require('./pool-factory');
16+
var deprecate = require('deprecate');
1617

1718
var PG = function(clientConstructor) {
1819
EventEmitter.call(this);
@@ -28,6 +29,7 @@ var PG = function(clientConstructor) {
2829
util.inherits(PG, EventEmitter);
2930

3031
PG.prototype.end = function() {
32+
deprecate('pg.end() is deprecated - please construct pools directly via new pg.Pool()');
3133
var self = this;
3234
var keys = Object.keys(this._pools);
3335
var count = keys.length;
@@ -50,6 +52,7 @@ PG.prototype.end = function() {
5052
};
5153

5254
PG.prototype.connect = function(config, callback) {
55+
deprecate('pg.connect() is deprecated - please construct pools directly via new pg.Pool()');
5356
if(typeof config == "function") {
5457
callback = config;
5558
config = null;
@@ -79,6 +82,7 @@ PG.prototype.connect = function(config, callback) {
7982

8083
// cancel the query running on the given client
8184
PG.prototype.cancel = function(config, client, query) {
85+
deprecate('pg.cancel() is deprecated - please create your own client instances to cancel queries');
8286
if(client.native) {
8387
return client.cancel(query);
8488
}

package.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@
1919
"main": "./lib",
2020
"dependencies": {
2121
"buffer-writer": "1.0.1",
22+
"deprecate": "1.0.0",
2223
"packet-reader": "0.3.1",
2324
"pg-connection-string": "0.1.3",
2425
"pg-pool": "1.*",

test/test-helper.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,8 @@
11
//make assert a global...
22
assert = require('assert');
33

4+
require('deprecate').silence = true;
5+
46
var EventEmitter = require('events').EventEmitter;
57
var sys = require('util');
68
var BufferList = require(__dirname+'/buffer-list')

yarn.lock

Lines changed: 248 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,248 @@
1+
# THIS IS AN AUTOGENERATED FILE. DO NOT EDIT THIS FILE DIRECTLY.
2+
# yarn lockfile v1
3+
4+
5+
ap@~0.2.0:
6+
version "0.2.0"
7+
resolved "https://registry.yarnpkg.com/ap/-/ap-0.2.0.tgz#ae0942600b29912f0d2b14ec60c45e8f330b6110"
8+
9+
async@0.9.0:
10+
version "0.9.0"
11+
resolved "https://registry.yarnpkg.com/async/-/async-0.9.0.tgz#ac3613b1da9bed1b47510bb4651b8931e47146c7"
12+
13+
buffer-writer@1.0.1:
14+
version "1.0.1"
15+
resolved "https://registry.yarnpkg.com/buffer-writer/-/buffer-writer-1.0.1.tgz#22a936901e3029afcd7547eb4487ceb697a3bf08"
16+
17+
cli@0.6.x:
18+
version "0.6.6"
19+
resolved "https://registry.yarnpkg.com/cli/-/cli-0.6.6.tgz#02ad44a380abf27adac5e6f0cdd7b043d74c53e3"
20+
dependencies:
21+
exit "0.1.2"
22+
glob "~ 3.2.1"
23+
24+
co@4.6.0:
25+
version "4.6.0"
26+
resolved "https://registry.yarnpkg.com/co/-/co-4.6.0.tgz#6ea6bdf3d853ae54ccb8e47bfa0bf3f9031fb184"
27+
28+
console-browserify@1.1.x:
29+
version "1.1.0"
30+
resolved "https://registry.yarnpkg.com/console-browserify/-/console-browserify-1.1.0.tgz#f0241c45730a9fc6323b206dbf38edc741d0bb10"
31+
dependencies:
32+
date-now "^0.1.4"
33+
34+
core-util-is@~1.0.0:
35+
version "1.0.2"
36+
resolved "https://registry.yarnpkg.com/core-util-is/-/core-util-is-1.0.2.tgz#b5fd54220aa2bc5ab57aab7140c940754503c1a7"
37+
38+
date-now@^0.1.4:
39+
version "0.1.4"
40+
resolved "https://registry.yarnpkg.com/date-now/-/date-now-0.1.4.tgz#eaf439fd4d4848ad74e5cc7dbef200672b9e345b"
41+
42+
deprecate@1.0.0:
43+
version "1.0.0"
44+
resolved "https://registry.yarnpkg.com/deprecate/-/deprecate-1.0.0.tgz#661490ed2428916a6c8883d8834e5646f4e4a4a8"
45+
46+
dom-serializer@0:
47+
version "0.1.0"
48+
resolved "https://registry.yarnpkg.com/dom-serializer/-/dom-serializer-0.1.0.tgz#073c697546ce0780ce23be4a28e293e40bc30c82"
49+
dependencies:
50+
domelementtype "~1.1.1"
51+
entities "~1.1.1"
52+
53+
domelementtype@1:
54+
version "1.3.0"
55+
resolved "https://registry.yarnpkg.com/domelementtype/-/domelementtype-1.3.0.tgz#b17aed82e8ab59e52dd9c19b1756e0fc187204c2"
56+
57+
domelementtype@~1.1.1:
58+
version "1.1.3"
59+
resolved "https://registry.yarnpkg.com/domelementtype/-/domelementtype-1.1.3.tgz#bd28773e2642881aec51544924299c5cd822185b"
60+
61+
domhandler@2.2:
62+
version "2.2.1"
63+
resolved "https://registry.yarnpkg.com/domhandler/-/domhandler-2.2.1.tgz#59df9dcd227e808b365ae73e1f6684ac3d946fc2"
64+
dependencies:
65+
domelementtype "1"
66+
67+
domutils@1.5:
68+
version "1.5.1"
69+
resolved "https://registry.yarnpkg.com/domutils/-/domutils-1.5.1.tgz#dcd8488a26f563d61079e48c9f7b7e32373682cf"
70+
dependencies:
71+
dom-serializer "0"
72+
domelementtype "1"
73+
74+
entities@1.0:
75+
version "1.0.0"
76+
resolved "https://registry.yarnpkg.com/entities/-/entities-1.0.0.tgz#b2987aa3821347fcde642b24fdfc9e4fb712bf26"
77+
78+
entities@~1.1.1:
79+
version "1.1.1"
80+
resolved "https://registry.yarnpkg.com/entities/-/entities-1.1.1.tgz#6e5c2d0a5621b5dadaecef80b90edfb5cd7772f0"
81+
82+
exit@0.1.2, exit@0.1.x:
83+
version "0.1.2"
84+
resolved "https://registry.yarnpkg.com/exit/-/exit-0.1.2.tgz#0632638f8d877cc82107d30a0fff1a17cba1cd0c"
85+
86+
generic-pool@2.4.3:
87+
version "2.4.3"
88+
resolved "https://registry.yarnpkg.com/generic-pool/-/generic-pool-2.4.3.tgz#780c36f69dfad05a5a045dd37be7adca11a4f6ff"
89+
90+
"glob@~ 3.2.1":
91+
version "3.2.11"
92+
resolved "https://registry.yarnpkg.com/glob/-/glob-3.2.11.tgz#4a973f635b9190f715d10987d5c00fd2815ebe3d"
93+
dependencies:
94+
inherits "2"
95+
minimatch "0.3"
96+
97+
htmlparser2@3.7.x:
98+
version "3.7.3"
99+
resolved "https://registry.yarnpkg.com/htmlparser2/-/htmlparser2-3.7.3.tgz#6a64c77637c08c6f30ec2a8157a53333be7cb05e"
100+
dependencies:
101+
domelementtype "1"
102+
domhandler "2.2"
103+
domutils "1.5"
104+
entities "1.0"
105+
readable-stream "1.1"
106+
107+
inherits@2, inherits@~2.0.1:
108+
version "2.0.3"
109+
resolved "https://registry.yarnpkg.com/inherits/-/inherits-2.0.3.tgz#633c2c83e3da42a502f52466022480f4208261de"
110+
111+
isarray@0.0.1:
112+
version "0.0.1"
113+
resolved "https://registry.yarnpkg.com/isarray/-/isarray-0.0.1.tgz#8a18acfca9a8f4177e09abfc6038939b05d1eedf"
114+
115+
jshint@2.5.2:
116+
version "2.5.2"
117+
resolved "https://registry.yarnpkg.com/jshint/-/jshint-2.5.2.tgz#bec223d5149e49ef6ea96dcf8b3504a27613e8be"
118+
dependencies:
119+
cli "0.6.x"
120+
console-browserify "1.1.x"
121+
exit "0.1.x"
122+
htmlparser2 "3.7.x"
123+
minimatch "0.x.x"
124+
shelljs "0.3.x"
125+
strip-json-comments "0.1.x"
126+
underscore "1.6.x"
127+
128+
lru-cache@2:
129+
version "2.7.3"
130+
resolved "https://registry.yarnpkg.com/lru-cache/-/lru-cache-2.7.3.tgz#6d4524e8b955f95d4f5b58851ce21dd72fb4e952"
131+
132+
minimatch@0.3:
133+
version "0.3.0"
134+
resolved "https://registry.yarnpkg.com/minimatch/-/minimatch-0.3.0.tgz#275d8edaac4f1bb3326472089e7949c8394699dd"
135+
dependencies:
136+
lru-cache "2"
137+
sigmund "~1.0.0"
138+
139+
minimatch@0.x.x:
140+
version "0.4.0"
141+
resolved "https://registry.yarnpkg.com/minimatch/-/minimatch-0.4.0.tgz#bd2c7d060d2c8c8fd7cde7f1f2ed2d5b270fdb1b"
142+
dependencies:
143+
lru-cache "2"
144+
sigmund "~1.0.0"
145+
146+
object-assign@4.1.0:
147+
version "4.1.0"
148+
resolved "https://registry.yarnpkg.com/object-assign/-/object-assign-4.1.0.tgz#7a3b3d0e98063d43f4c03f2e8ae6cd51a86883a0"
149+
150+
packet-reader@0.3.1:
151+
version "0.3.1"
152+
resolved "https://registry.yarnpkg.com/packet-reader/-/packet-reader-0.3.1.tgz#cd62e60af8d7fea8a705ec4ff990871c46871f27"
153+
154+
pg-connection-string@0.1.3:
155+
version "0.1.3"
156+
resolved "https://registry.yarnpkg.com/pg-connection-string/-/pg-connection-string-0.1.3.tgz#da1847b20940e42ee1492beaf65d49d91b245df7"
157+
158+
pg-copy-streams@0.3.0:
159+
version "0.3.0"
160+
resolved "https://registry.yarnpkg.com/pg-copy-streams/-/pg-copy-streams-0.3.0.tgz#a4fbc2a3b788d4e9da6f77ceb35422d8d7043b7f"
161+
162+
pg-pool@1.*:
163+
version "1.7.1"
164+
resolved "https://registry.yarnpkg.com/pg-pool/-/pg-pool-1.7.1.tgz#421105cb7469979dcc48d6fc4fe3fe4659437437"
165+
dependencies:
166+
generic-pool "2.4.3"
167+
object-assign "4.1.0"
168+
169+
pg-types@1.*:
170+
version "1.12.0"
171+
resolved "https://registry.yarnpkg.com/pg-types/-/pg-types-1.12.0.tgz#8ad3b7b897e3fd463e62de241ad5fc640b4a66f0"
172+
dependencies:
173+
ap "~0.2.0"
174+
postgres-array "~1.0.0"
175+
postgres-bytea "~1.0.0"
176+
postgres-date "~1.0.0"
177+
postgres-interval "^1.1.0"
178+
179+
pgpass@1.x:
180+
version "1.0.2"
181+
resolved "https://registry.yarnpkg.com/pgpass/-/pgpass-1.0.2.tgz#2a7bb41b6065b67907e91da1b07c1847c877b306"
182+
dependencies:
183+
split "^1.0.0"
184+
185+
postgres-array@~1.0.0:
186+
version "1.0.2"
187+
resolved "https://registry.yarnpkg.com/postgres-array/-/postgres-array-1.0.2.tgz#8e0b32eb03bf77a5c0a7851e0441c169a256a238"
188+
189+
postgres-bytea@~1.0.0:
190+
version "1.0.0"
191+
resolved "https://registry.yarnpkg.com/postgres-bytea/-/postgres-bytea-1.0.0.tgz#027b533c0aa890e26d172d47cf9ccecc521acd35"
192+
193+
postgres-date@~1.0.0:
194+
version "1.0.3"
195+
resolved "https://registry.yarnpkg.com/postgres-date/-/postgres-date-1.0.3.tgz#e2d89702efdb258ff9d9cee0fe91bd06975257a8"
196+
197+
postgres-interval@^1.1.0:
198+
version "1.1.0"
199+
resolved "https://registry.yarnpkg.com/postgres-interval/-/postgres-interval-1.1.0.tgz#1031e7bac34564132862adc9eb6c6d2f3aa75bb4"
200+
dependencies:
201+
xtend "^4.0.0"
202+
203+
readable-stream@1.1:
204+
version "1.1.13"
205+
resolved "https://registry.yarnpkg.com/readable-stream/-/readable-stream-1.1.13.tgz#f6eef764f514c89e2b9e23146a75ba106756d23e"
206+
dependencies:
207+
core-util-is "~1.0.0"
208+
inherits "~2.0.1"
209+
isarray "0.0.1"
210+
string_decoder "~0.10.x"
211+
212+
semver@4.3.2:
213+
version "4.3.2"
214+
resolved "https://registry.yarnpkg.com/semver/-/semver-4.3.2.tgz#c7a07158a80bedd052355b770d82d6640f803be7"
215+
216+
shelljs@0.3.x:
217+
version "0.3.0"
218+
resolved "https://registry.yarnpkg.com/shelljs/-/shelljs-0.3.0.tgz#3596e6307a781544f591f37da618360f31db57b1"
219+
220+
sigmund@~1.0.0:
221+
version "1.0.1"
222+
resolved "https://registry.yarnpkg.com/sigmund/-/sigmund-1.0.1.tgz#3ff21f198cad2175f9f3b781853fd94d0d19b590"
223+
224+
split@^1.0.0:
225+
version "1.0.0"
226+
resolved "https://registry.yarnpkg.com/split/-/split-1.0.0.tgz#c4395ce683abcd254bc28fe1dabb6e5c27dcffae"
227+
dependencies:
228+
through "2"
229+
230+
string_decoder@~0.10.x:
231+
version "0.10.31"
232+
resolved "https://registry.yarnpkg.com/string_decoder/-/string_decoder-0.10.31.tgz#62e203bc41766c6c28c9fc84301dab1c5310fa94"
233+
234+
strip-json-comments@0.1.x:
235+
version "0.1.3"
236+
resolved "https://registry.yarnpkg.com/strip-json-comments/-/strip-json-comments-0.1.3.tgz#164c64e370a8a3cc00c9e01b539e569823f0ee54"
237+
238+
through@2:
239+
version "2.3.8"
240+
resolved "https://registry.yarnpkg.com/through/-/through-2.3.8.tgz#0dd4c9ffaabc357960b1b724115d7e0e86a2e1f5"
241+
242+
underscore@1.6.x:
243+
version "1.6.0"
244+
resolved "https://registry.yarnpkg.com/underscore/-/underscore-1.6.0.tgz#8b38b10cacdef63337b8b24e4ff86d45aea529a8"
245+
246+
xtend@^4.0.0:
247+
version "4.0.1"
248+
resolved "https://registry.yarnpkg.com/xtend/-/xtend-4.0.1.tgz#a5c6d532be656e23db820efb943a1f04998d63af"

0 commit comments

Comments
 (0)