Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Commite955bd4

Browse files
committed
Move some code from src/bin/scripts to src/fe_utils to permit reuse.
The parallel slots infrastructure (which implements client-sidemultiplexing of server connections doing similar things, notthreading or multiple processes or anything like that) are moved fromsrc/bin/scripts/scripts_parallel.c to src/fe_utils/parallel_slot.c.The functions consumeQueryResult() and processQueryResult() which werepreviously part of src/bin/scripts/common.c are now moved into thatfile as well, becoming static helper functions. This might need to bechanged in the future, but currently they're not used for anythingelse.Some other functions from src/bin/scripts/common.c are moved to tosrc/fe_utils and are split up among several files. connectDatabase(),connectMaintenanceDatabase(), and disconnectDatabase() are moved toconnect_utils.c. executeQuery(), executeCommand(), andexecuteMaintenanceCommand() are move to query_utils.c.handle_help_version_opts() is moved to option_utils.c.Mark Dilger, reviewed by me. The larger patch series of which this isa part has also had review from Peter Geoghegan, Andres Freund, ÁlvaroHerrera, Michael Paquier, and Amul Sul, but I don't know whether anyof them have reviewed this bit specifically.Discussion:http://postgr.es/m/12ED3DA8-25F0-4B68-937D-D907CFBF08E7@enterprisedb.comDiscussion:http://postgr.es/m/5F743835-3399-419C-8324-2D424237E999@enterprisedb.comDiscussion:http://postgr.es/m/70655DF3-33CE-4527-9A4D-DDEB582B6BA0@enterprisedb.com
1 parentc444472 commite955bd4

22 files changed

+498
-385
lines changed

‎src/bin/scripts/Makefile

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -28,8 +28,8 @@ createuser: createuser.o common.o $(WIN32RES) | submake-libpq submake-libpgport
2828
dropdb: dropdb.o common.o$(WIN32RES) | submake-libpq submake-libpgport submake-libpgfeutils
2929
dropuser: dropuser.o common.o$(WIN32RES) | submake-libpq submake-libpgport submake-libpgfeutils
3030
clusterdb: clusterdb.o common.o$(WIN32RES) | submake-libpq submake-libpgport submake-libpgfeutils
31-
vacuumdb: vacuumdb.o common.oscripts_parallel.o$(WIN32RES) | submake-libpq submake-libpgport submake-libpgfeutils
32-
reindexdb: reindexdb.o common.oscripts_parallel.o$(WIN32RES) | submake-libpq submake-libpgport submake-libpgfeutils
31+
vacuumdb: vacuumdb.o common.o$(WIN32RES) | submake-libpq submake-libpgport submake-libpgfeutils
32+
reindexdb: reindexdb.o common.o$(WIN32RES) | submake-libpq submake-libpgport submake-libpgfeutils
3333
pg_isready: pg_isready.o common.o$(WIN32RES) | submake-libpq submake-libpgport submake-libpgfeutils
3434

3535
install: all installdirs
@@ -50,7 +50,7 @@ uninstall:
5050

5151
cleandistcleanmaintainer-clean:
5252
rm -f$(addsuffix$(X),$(PROGRAMS))$(addsuffix .o,$(PROGRAMS))
53-
rm -f common.oscripts_parallel.o$(WIN32RES)
53+
rm -f common.o$(WIN32RES)
5454
rm -rf tmp_check
5555

5656
check:

‎src/bin/scripts/clusterdb.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,8 @@
1313
#include"common.h"
1414
#include"common/logging.h"
1515
#include"fe_utils/cancel.h"
16+
#include"fe_utils/option_utils.h"
17+
#include"fe_utils/query_utils.h"
1618
#include"fe_utils/simple_list.h"
1719
#include"fe_utils/string_utils.h"
1820

‎src/bin/scripts/common.c

Lines changed: 1 addition & 317 deletions
Original file line numberDiff line numberDiff line change
@@ -22,325 +22,9 @@
2222
#include"common/logging.h"
2323
#include"common/string.h"
2424
#include"fe_utils/cancel.h"
25+
#include"fe_utils/query_utils.h"
2526
#include"fe_utils/string_utils.h"
2627

27-
#defineERRCODE_UNDEFINED_TABLE "42P01"
28-
29-
/*
30-
* Provide strictly harmonized handling of --help and --version
31-
* options.
32-
*/
33-
void
34-
handle_help_version_opts(intargc,char*argv[],
35-
constchar*fixed_progname,help_handlerhlp)
36-
{
37-
if (argc>1)
38-
{
39-
if (strcmp(argv[1],"--help")==0||strcmp(argv[1],"-?")==0)
40-
{
41-
hlp(get_progname(argv[0]));
42-
exit(0);
43-
}
44-
if (strcmp(argv[1],"--version")==0||strcmp(argv[1],"-V")==0)
45-
{
46-
printf("%s (PostgreSQL) "PG_VERSION"\n",fixed_progname);
47-
exit(0);
48-
}
49-
}
50-
}
51-
52-
53-
/*
54-
* Make a database connection with the given parameters.
55-
*
56-
* An interactive password prompt is automatically issued if needed and
57-
* allowed by cparams->prompt_password.
58-
*
59-
* If allow_password_reuse is true, we will try to re-use any password
60-
* given during previous calls to this routine. (Callers should not pass
61-
* allow_password_reuse=true unless reconnecting to the same database+user
62-
* as before, else we might create password exposure hazards.)
63-
*/
64-
PGconn*
65-
connectDatabase(constConnParams*cparams,constchar*progname,
66-
boolecho,boolfail_ok,boolallow_password_reuse)
67-
{
68-
PGconn*conn;
69-
boolnew_pass;
70-
staticchar*password=NULL;
71-
72-
/* Callers must supply at least dbname; other params can be NULL */
73-
Assert(cparams->dbname);
74-
75-
if (!allow_password_reuse&&password)
76-
{
77-
free(password);
78-
password=NULL;
79-
}
80-
81-
if (cparams->prompt_password==TRI_YES&&password==NULL)
82-
password=simple_prompt("Password: ", false);
83-
84-
/*
85-
* Start the connection. Loop until we have a password if requested by
86-
* backend.
87-
*/
88-
do
89-
{
90-
constchar*keywords[8];
91-
constchar*values[8];
92-
inti=0;
93-
94-
/*
95-
* If dbname is a connstring, its entries can override the other
96-
* values obtained from cparams; but in turn, override_dbname can
97-
* override the dbname component of it.
98-
*/
99-
keywords[i]="host";
100-
values[i++]=cparams->pghost;
101-
keywords[i]="port";
102-
values[i++]=cparams->pgport;
103-
keywords[i]="user";
104-
values[i++]=cparams->pguser;
105-
keywords[i]="password";
106-
values[i++]=password;
107-
keywords[i]="dbname";
108-
values[i++]=cparams->dbname;
109-
if (cparams->override_dbname)
110-
{
111-
keywords[i]="dbname";
112-
values[i++]=cparams->override_dbname;
113-
}
114-
keywords[i]="fallback_application_name";
115-
values[i++]=progname;
116-
keywords[i]=NULL;
117-
values[i++]=NULL;
118-
Assert(i <=lengthof(keywords));
119-
120-
new_pass= false;
121-
conn=PQconnectdbParams(keywords,values, true);
122-
123-
if (!conn)
124-
{
125-
pg_log_error("could not connect to database %s: out of memory",
126-
cparams->dbname);
127-
exit(1);
128-
}
129-
130-
/*
131-
* No luck? Trying asking (again) for a password.
132-
*/
133-
if (PQstatus(conn)==CONNECTION_BAD&&
134-
PQconnectionNeedsPassword(conn)&&
135-
cparams->prompt_password!=TRI_NO)
136-
{
137-
PQfinish(conn);
138-
if (password)
139-
free(password);
140-
password=simple_prompt("Password: ", false);
141-
new_pass= true;
142-
}
143-
}while (new_pass);
144-
145-
/* check to see that the backend connection was successfully made */
146-
if (PQstatus(conn)==CONNECTION_BAD)
147-
{
148-
if (fail_ok)
149-
{
150-
PQfinish(conn);
151-
returnNULL;
152-
}
153-
pg_log_error("%s",PQerrorMessage(conn));
154-
exit(1);
155-
}
156-
157-
/* Start strict; callers may override this. */
158-
PQclear(executeQuery(conn,ALWAYS_SECURE_SEARCH_PATH_SQL,echo));
159-
160-
returnconn;
161-
}
162-
163-
/*
164-
* Try to connect to the appropriate maintenance database.
165-
*
166-
* This differs from connectDatabase only in that it has a rule for
167-
* inserting a default "dbname" if none was given (which is why cparams
168-
* is not const). Note that cparams->dbname should typically come from
169-
* a --maintenance-db command line parameter.
170-
*/
171-
PGconn*
172-
connectMaintenanceDatabase(ConnParams*cparams,
173-
constchar*progname,boolecho)
174-
{
175-
PGconn*conn;
176-
177-
/* If a maintenance database name was specified, just connect to it. */
178-
if (cparams->dbname)
179-
returnconnectDatabase(cparams,progname,echo, false, false);
180-
181-
/* Otherwise, try postgres first and then template1. */
182-
cparams->dbname="postgres";
183-
conn=connectDatabase(cparams,progname,echo, true, false);
184-
if (!conn)
185-
{
186-
cparams->dbname="template1";
187-
conn=connectDatabase(cparams,progname,echo, false, false);
188-
}
189-
returnconn;
190-
}
191-
192-
/*
193-
* Disconnect the given connection, canceling any statement if one is active.
194-
*/
195-
void
196-
disconnectDatabase(PGconn*conn)
197-
{
198-
charerrbuf[256];
199-
200-
Assert(conn!=NULL);
201-
202-
if (PQtransactionStatus(conn)==PQTRANS_ACTIVE)
203-
{
204-
PGcancel*cancel;
205-
206-
if ((cancel=PQgetCancel(conn)))
207-
{
208-
(void)PQcancel(cancel,errbuf,sizeof(errbuf));
209-
PQfreeCancel(cancel);
210-
}
211-
}
212-
213-
PQfinish(conn);
214-
}
215-
216-
/*
217-
* Run a query, return the results, exit program on failure.
218-
*/
219-
PGresult*
220-
executeQuery(PGconn*conn,constchar*query,boolecho)
221-
{
222-
PGresult*res;
223-
224-
if (echo)
225-
printf("%s\n",query);
226-
227-
res=PQexec(conn,query);
228-
if (!res||
229-
PQresultStatus(res)!=PGRES_TUPLES_OK)
230-
{
231-
pg_log_error("query failed: %s",PQerrorMessage(conn));
232-
pg_log_info("query was: %s",query);
233-
PQfinish(conn);
234-
exit(1);
235-
}
236-
237-
returnres;
238-
}
239-
240-
241-
/*
242-
* As above for a SQL command (which returns nothing).
243-
*/
244-
void
245-
executeCommand(PGconn*conn,constchar*query,boolecho)
246-
{
247-
PGresult*res;
248-
249-
if (echo)
250-
printf("%s\n",query);
251-
252-
res=PQexec(conn,query);
253-
if (!res||
254-
PQresultStatus(res)!=PGRES_COMMAND_OK)
255-
{
256-
pg_log_error("query failed: %s",PQerrorMessage(conn));
257-
pg_log_info("query was: %s",query);
258-
PQfinish(conn);
259-
exit(1);
260-
}
261-
262-
PQclear(res);
263-
}
264-
265-
266-
/*
267-
* As above for a SQL maintenance command (returns command success).
268-
* Command is executed with a cancel handler set, so Ctrl-C can
269-
* interrupt it.
270-
*/
271-
bool
272-
executeMaintenanceCommand(PGconn*conn,constchar*query,boolecho)
273-
{
274-
PGresult*res;
275-
boolr;
276-
277-
if (echo)
278-
printf("%s\n",query);
279-
280-
SetCancelConn(conn);
281-
res=PQexec(conn,query);
282-
ResetCancelConn();
283-
284-
r= (res&&PQresultStatus(res)==PGRES_COMMAND_OK);
285-
286-
if (res)
287-
PQclear(res);
288-
289-
returnr;
290-
}
291-
292-
/*
293-
* Consume all the results generated for the given connection until
294-
* nothing remains. If at least one error is encountered, return false.
295-
* Note that this will block if the connection is busy.
296-
*/
297-
bool
298-
consumeQueryResult(PGconn*conn)
299-
{
300-
boolok= true;
301-
PGresult*result;
302-
303-
SetCancelConn(conn);
304-
while ((result=PQgetResult(conn))!=NULL)
305-
{
306-
if (!processQueryResult(conn,result))
307-
ok= false;
308-
}
309-
ResetCancelConn();
310-
returnok;
311-
}
312-
313-
/*
314-
* Process (and delete) a query result. Returns true if there's no error,
315-
* false otherwise -- but errors about trying to work on a missing relation
316-
* are reported and subsequently ignored.
317-
*/
318-
bool
319-
processQueryResult(PGconn*conn,PGresult*result)
320-
{
321-
/*
322-
* If it's an error, report it. Errors about a missing table are harmless
323-
* so we continue processing; but die for other errors.
324-
*/
325-
if (PQresultStatus(result)!=PGRES_COMMAND_OK)
326-
{
327-
char*sqlState=PQresultErrorField(result,PG_DIAG_SQLSTATE);
328-
329-
pg_log_error("processing of database \"%s\" failed: %s",
330-
PQdb(conn),PQerrorMessage(conn));
331-
332-
if (sqlState&&strcmp(sqlState,ERRCODE_UNDEFINED_TABLE)!=0)
333-
{
334-
PQclear(result);
335-
return false;
336-
}
337-
}
338-
339-
PQclear(result);
340-
return true;
341-
}
342-
343-
34428
/*
34529
* Split TABLE[(COLUMNS)] into TABLE and [(COLUMNS)] portions. When you
34630
* finish using them, pg_free(*table). *columns is a pointer into "spec",

0 commit comments

Comments
 (0)

[8]ページ先頭

©2009-2025 Movatter.jp