Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Fri, Dec 6, 2013 at 04:04:36PM +0100, Andres Freund wrote: > On 2013-12-05 23:01:28 +0200, Heikki Linnakangas wrote: > > On 12/05/2013 10:37 PM, Robert Haas wrote: > > >On Thu, Dec 5, 2013 at 3:05 PM, Tom Lane wrote: > > >>It might be unpleasant to use in some cases, though. > > > > > >Why would there be more than a few cases in the first place? Who is > > >going to use this beyond psql, pg_dump(all), and pg_upgrade, and why? > > > > Well, you might want to use pgAdmin, or your other favorite admin tool. I'm > > not sure how well it would work, and I think it's OK if we say "sorry, can't > > do that", but it's not a crazy thing to want. > > Pgadmin wouldn't work, it uses multiple connections for anything but the > most trivial tasks. You can't even send a manual sql query using only > one connection. > I think that's true for most of the non-trivial tools. FYI, pg_upgrade in parallel mode needs multiple database connections too. -- Bruce Momjian http://momjian.us EnterpriseDB http://enterprisedb.com + Everyone has their own god. + -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
Andres Freund writes: > On 2013-12-06 11:02:48 -0500, Tom Lane wrote: >> I think the special-purpose command line switches you mention can be >> passed through PGOPTIONS, rather than inventing a new parameter -- do you >> have an objection to that? > I am not sure if they currently will get recognized early enough and > whether permission checking will interferes, but if so, that's probably > fixable. Shouldn't be a problem --- the single-user mode will just concatenate the options parameter onto the command line it builds. > There's the question what we're going to end up doing with the current > single user mode? There's some somewhat ugly code around for it... Nothing, in the short term. In a release or two we can get rid of it, probably, but I'd hesitate to provide no overlap at all of these usage modes. regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On 2013-12-06 11:02:48 -0500, Tom Lane wrote: > Andres Freund writes: > My feeling is that we should just treat the executable name and data > directory path as new connection parameters, which'd be ignored in > normal-connection mode, just as some other parameters will be ignored in > single-user mode. Otherwise we'll find ourselves building parameter > setting infrastructure that pretty much duplicates what's there for the > existing connection parameters. Right. > I think the special-purpose command line switches you mention can be > passed through PGOPTIONS, rather than inventing a new parameter -- do you > have an objection to that? I am not sure if they currently will get recognized early enough and whether permission checking will interferes, but if so, that's probably fixable. > > Not sure if we need anything but the pid of the postmaster be returned? > > The new PQconnect routine would certainly hand back a PGconn. I think > we'd need a new query function PQchildPid(PGconn *) or some such to > provide access to the child process PID. I was thinking of a pid_t* argument to the new routine, but it's likely unneccessary as we're probably going to end up storing it in PGconn anyway. There's the question what we're going to end up doing with the current single user mode? There's some somewhat ugly code around for it... Greetings, Andres Freund -- Andres Freund http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
Andres Freund writes: > On 2013-12-05 23:01:28 +0200, Heikki Linnakangas wrote: >> Right. Not all of the parameters will make sense for a stand-alone backend >> though, like the hostname and port number. And I think you need need a new >> parameter to pass the path to the 'postgres' executable, unless we re-use >> the host parameter for that. > Hm. I'd guessed that we wouldn't use the connection string to pass down > the executable name and the datadir now that we're inventing a separate > function. But maybe that's unneccessary. > What parameters do we require to be set for that mode: > * path to postgres > * data directory > * database name (single mode after all) > * port, because of the shmem key? I'd say that's not important enough > I think we also need to be able to pass some additional parameters to > postgres: > - config_file, hba_file, ... might be required to start pg in some > environments > - -P, -O , are sometimes required in cases single user mode is > neccessary for data recovery. Right, so by the time we're done, we'd still need a connection string or the moral equivalent. My feeling is that we should just treat the executable name and data directory path as new connection parameters, which'd be ignored in normal-connection mode, just as some other parameters will be ignored in single-user mode. Otherwise we'll find ourselves building parameter setting infrastructure that pretty much duplicates what's there for the existing connection parameters. I think the special-purpose command line switches you mention can be passed through PGOPTIONS, rather than inventing a new parameter -- do you have an objection to that? > Not sure if we need anything but the pid of the postmaster be returned? The new PQconnect routine would certainly hand back a PGconn. I think we'd need a new query function PQchildPid(PGconn *) or some such to provide access to the child process PID. regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On 2013-12-05 23:01:28 +0200, Heikki Linnakangas wrote: > On 12/05/2013 10:37 PM, Robert Haas wrote: > >On Thu, Dec 5, 2013 at 3:05 PM, Tom Lane wrote: > >>It might be unpleasant to use in some cases, though. > > > >Why would there be more than a few cases in the first place? Who is > >going to use this beyond psql, pg_dump(all), and pg_upgrade, and why? > > Well, you might want to use pgAdmin, or your other favorite admin tool. I'm > not sure how well it would work, and I think it's OK if we say "sorry, can't > do that", but it's not a crazy thing to want. Pgadmin wouldn't work, it uses multiple connections for anything but the most trivial tasks. You can't even send a manual sql query using only one connection. I think that's true for most of the non-trivial tools. > >>Another issue is that we have too many variants of PQconnect > >>already; which of them are we prepared to clone for this > >>hypothetical new connection method? > > > >PQconnectdbParams, I assume. Isn't that the one to rule them all, > >modulo async connect which I can't think is relevant here? > Right. Not all of the parameters will make sense for a stand-alone backend > though, like the hostname and port number. And I think you need need a new > parameter to pass the path to the 'postgres' executable, unless we re-use > the host parameter for that. Hm. I'd guessed that we wouldn't use the connection string to pass down the executable name and the datadir now that we're inventing a separate function. But maybe that's unneccessary. What parameters do we require to be set for that mode: * path to postgres * data directory * database name (single mode after all) * port, because of the shmem key? I'd say that's not important enough I think we also need to be able to pass some additional parameters to postgres: - config_file, hba_file, ... might be required to start pg in some environments - -P, -O , are sometimes required in cases single user mode is neccessary for data recovery. So I think we should just allow passing through arguments to postgres. Not sure if we need anything but the pid of the postmaster be returned? > >Or don't clone that one but instead have > >PQnextConnectionShouldForkThisBinary('...') and let the psql/pg_dump > >switch be --standalone=full-path-to-the-postgres-binary. Yuck, that's ugly. Greetings, Andres Freund -- Andres Freund http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On 12/05/2013 10:37 PM, Robert Haas wrote: On Thu, Dec 5, 2013 at 3:05 PM, Tom Lane wrote: It might be unpleasant to use in some cases, though. Why would there be more than a few cases in the first place? Who is going to use this beyond psql, pg_dump(all), and pg_upgrade, and why? Well, you might want to use pgAdmin, or your other favorite admin tool. I'm not sure how well it would work, and I think it's OK if we say "sorry, can't do that", but it's not a crazy thing to want. Another issue is that we have too many variants of PQconnect already; which of them are we prepared to clone for this hypothetical new connection method? PQconnectdbParams, I assume. Isn't that the one to rule them all, modulo async connect which I can't think is relevant here? Right. Not all of the parameters will make sense for a stand-alone backend though, like the hostname and port number. And I think you need need a new parameter to pass the path to the 'postgres' executable, unless we re-use the host parameter for that. Or don't clone that one but instead have PQnextConnectionShouldForkThisBinary('...') and let the psql/pg_dump switch be --standalone=full-path-to-the-postgres-binary. I think a separate function makes more sense. - Heikki -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Thu, Dec 5, 2013 at 3:05 PM, Tom Lane wrote: > I'm pretty much persuaded by Andres' point that we should not allow a > child process to be launched under a client app without clear permission > from the code of the app (and *not* just some environment variable that > might have been set far away, perhaps by someone who doesn't know what the > app assumes about SIGCHLD etc). So a separate connection call seems like > not a bad idea. In the case of psql and pg_dump it'd be reasonable to > invent a separate command line switch that drives use of this call instead > of normal PQconnect. Doing that, and *not* allowing the text of the > connection string to determine it, seems like it pretty well solves any > security objections. Yep. > It might be unpleasant to use in some cases, though. Why would there be more than a few cases in the first place? Who is going to use this beyond psql, pg_dump(all), and pg_upgrade, and why? > Another issue is that we have too many variants of PQconnect already; > which of them are we prepared to clone for this hypothetical new > connection method? PQconnectdbParams, I assume. Isn't that the one to rule them all, modulo async connect which I can't think is relevant here? Or don't clone that one but instead have PQnextConnectionShouldForkThisBinary('...') and let the psql/pg_dump switch be --standalone=full-path-to-the-postgres-binary. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
Robert Haas writes: > Yeah, seriously. I don't understand what the big deal is here. The > right design here is 99.44% clear here, and the committer (presumably > Tom) can handle the other 0.56% however he'd like. Let's do this and > move on. Yeah, but the remaining 0.56% is an important decision, not least because it's got security implications. I think we need some consensus not just a unilateral committer decision. I'm pretty much persuaded by Andres' point that we should not allow a child process to be launched under a client app without clear permission from the code of the app (and *not* just some environment variable that might have been set far away, perhaps by someone who doesn't know what the app assumes about SIGCHLD etc). So a separate connection call seems like not a bad idea. In the case of psql and pg_dump it'd be reasonable to invent a separate command line switch that drives use of this call instead of normal PQconnect. Doing that, and *not* allowing the text of the connection string to determine it, seems like it pretty well solves any security objections. It might be unpleasant to use in some cases, though. Another issue is that we have too many variants of PQconnect already; which of them are we prepared to clone for this hypothetical new connection method? regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Thu, Dec 5, 2013 at 11:52 AM, Andres Freund wrote: > On 2013-12-05 11:39:29 -0500, Peter Eisentraut wrote: >> I think this proposal is a bit deadlocked now. >> >> - There are technical concerns about launching a server executable from >> within a client. >> >> - There are conceptual concerns about promoting an embedded database mode. >> >> On the other hand: >> >> - Everyone would like to have a way to use psql (and other basic >> clients) in stand-alone mode. >> The only solutions I see are: >> >> 1. do nothing >> >> 2. do everything (i.e., existing terminal mode plus socket mode plus >> embedded mode), letting the user work out the differences >> >> Pick one. ;-) > > 3) make it an explicit parameter, outside the database DSN, and let the >clients contain a tiny bit of explict code about it. There really >aren't that many clients that can use such a mode sensibly. > > If we ever want to support a real embedded mode, much, much more than > this is needed. I don't think we should let that stop us from improving > single user mode. Yeah, seriously. I don't understand what the big deal is here. The right design here is 99.44% clear here, and the committer (presumably Tom) can handle the other 0.56% however he'd like. Let's do this and move on. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On 2013-12-05 11:39:29 -0500, Peter Eisentraut wrote: > I think this proposal is a bit deadlocked now. > > - There are technical concerns about launching a server executable from > within a client. > > - There are conceptual concerns about promoting an embedded database mode. > > On the other hand: > > - Everyone would like to have a way to use psql (and other basic > clients) in stand-alone mode. > The only solutions I see are: > > 1. do nothing > > 2. do everything (i.e., existing terminal mode plus socket mode plus > embedded mode), letting the user work out the differences > > Pick one. ;-) 3) make it an explicit parameter, outside the database DSN, and let the clients contain a tiny bit of explict code about it. There really aren't that many clients that can use such a mode sensibly. If we ever want to support a real embedded mode, much, much more than this is needed. I don't think we should let that stop us from improving single user mode. Greetings, Andres Freund -- Andres Freund http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
I think this proposal is a bit deadlocked now. - There are technical concerns about launching a server executable from within a client. - There are conceptual concerns about promoting an embedded database mode. On the other hand: - Everyone would like to have a way to use psql (and other basic clients) in stand-alone mode. The compromise would be to not launch the server from within the client, but have client and server communicate over external mechanisms (e.g., Unix-domain socket). The concern about that was that it would open up standalone mode to accidental third-party connections. While there are some ways around that (socket in private directory), they are not easy and not portable. So standalone mode would became less robust and reliable overall. The only solutions I see are: 1. do nothing 2. do everything (i.e., existing terminal mode plus socket mode plus embedded mode), letting the user work out the differences Pick one. ;-) -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On 12/5/13, 6:07 AM, Simon Riggs wrote: > On 5 December 2013 01:55, Peter Eisentraut wrote: >> On Thu, 2013-11-14 at 12:11 +0530, Amit Kapila wrote: >>>If an application wants to allow these connection parameters to be >>> used, it would need to do PQenableStartServer() first. If it doesn't, >>> those connection parameters will be rejected. >> >> Stupid idea: Would it work that we require an environment variable to be >> set before we allow the standalone_backend connection parameter? That's >> easy to do, easy to audit, and doesn't require any extra code in the >> individual clients. > > I like the idea... should it be in pg_hba.conf ? > Or should it be next to listen_addresses in postgresql.conf? No, it's an environment variable. -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On 2013-12-04 20:55:08 -0500, Peter Eisentraut wrote: > On Thu, 2013-11-14 at 12:11 +0530, Amit Kapila wrote: > >If an application wants to allow these connection parameters to be > > used, it would need to do PQenableStartServer() first. If it doesn't, > > those connection parameters will be rejected. > > Stupid idea: Would it work that we require an environment variable to be > set before we allow the standalone_backend connection parameter? That's > easy to do, easy to audit, and doesn't require any extra code in the > individual clients. I still don't think it's ok to start forking in arbitrary applications without their knowledge. So I don't think that buys us enough. Greetings, Andres Freund -- Andres Freund http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On 5 December 2013 01:55, Peter Eisentraut wrote: > On Thu, 2013-11-14 at 12:11 +0530, Amit Kapila wrote: >>If an application wants to allow these connection parameters to be >> used, it would need to do PQenableStartServer() first. If it doesn't, >> those connection parameters will be rejected. > > Stupid idea: Would it work that we require an environment variable to be > set before we allow the standalone_backend connection parameter? That's > easy to do, easy to audit, and doesn't require any extra code in the > individual clients. I like the idea... should it be in pg_hba.conf ? Or should it be next to listen_addresses in postgresql.conf? hba might be less convenient but seems like the correct place -- Simon Riggs http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Thu, 2013-12-05 at 09:02 +0530, Amit Kapila wrote: > This is certainly not a stupid idea, rather something on similar lines > has been discussed previously in this thread. > Tom has suggested something similar, but I am not sure if there was a > conclusion on that point. Please see the > relavant discussion at below link: > http://www.postgresql.org/message-id/17384.1346645...@sss.pgh.pa.us Yeah, I think the environment variable idea wasn't actually refuted there. -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Thu, Dec 5, 2013 at 7:25 AM, Peter Eisentraut wrote: > On Thu, 2013-11-14 at 12:11 +0530, Amit Kapila wrote: >>If an application wants to allow these connection parameters to be >> used, it would need to do PQenableStartServer() first. If it doesn't, >> those connection parameters will be rejected. > > Stupid idea: Would it work that we require an environment variable to be > set before we allow the standalone_backend connection parameter? That's > easy to do, easy to audit, and doesn't require any extra code in the > individual clients. This is certainly not a stupid idea, rather something on similar lines has been discussed previously in this thread. Tom has suggested something similar, but I am not sure if there was a conclusion on that point. Please see the relavant discussion at below link: http://www.postgresql.org/message-id/17384.1346645...@sss.pgh.pa.us I think the basic question at that time was why should we consider an environment variable more safe. With Regards, Amit Kapila. EnterpriseDB: http://www.enterprisedb.com -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Thu, 2013-11-14 at 12:11 +0530, Amit Kapila wrote: >If an application wants to allow these connection parameters to be > used, it would need to do PQenableStartServer() first. If it doesn't, > those connection parameters will be rejected. Stupid idea: Would it work that we require an environment variable to be set before we allow the standalone_backend connection parameter? That's easy to do, easy to audit, and doesn't require any extra code in the individual clients. -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Thu, Nov 21, 2013 at 9:54 PM, Amit Kapila wrote: > On Thu, Nov 21, 2013 at 8:14 PM, Tom Lane wrote: >> Amit Kapila writes: >>> Here what I have in mind is that: >> >> Why would you make psql behave differently from our other command-line >> clients? > >No, psql should not behave different from other clients. Sorry, I > was under assumption that for other programs we will not take backend > executable >path. >One other thing which is not clear to me is that how by calling > some special/new API we can ensure that the path provided by user is >a valid path, are we going to validate file given in > 'standalone_backend' switch in some way? Here, do we mean that if user specifies special switch, then psql/pg_dump will call new API (eg. PQstartSingleUser(dsn)) which will use postgres from same bin directory they are in, rather than using from standalone_backend. Can we consider this as a way to proceed for this patch? On a side note, today while reading what other software's does to protect them from such a security threat, I came across this link (http://osxbook.com/book/bonus/chapter7/binaryprotection/index.html) which suggests to have encrypted binaries. The use for encrypted binaries is somewhat similar to what we discussed as a security threat in this mail thread. Some text from link which made me think that this is relevant. For example, "one could turn the requirement around and say that a given system must not run any binaries unless they are from a certain source (or set of sources). This could be used to create an admission-control mechanism for executables, which in turn could be used in defending against malware. In a draconian managed environment, it might be desired to limit program execution on managed systems to a predefined set of programs—nothing else will execute." With Regards, Amit Kapila. EnterpriseDB: http://www.enterprisedb.com -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Thu, Nov 21, 2013 at 8:14 PM, Tom Lane wrote: > Amit Kapila writes: >> Here what I have in mind is that: >> a. In pg_dump or other internal utilities where we want to use this >> feature, they should call PQenableStart() or some other API before >> calling PQConnect() which will indicate that it wants to operate >> as a standalone mode. >> b. In psql, if user specifies this special switch ( >> 'standalone_datadir'), then internally we will call PQenableStart() >> and use postgres from same >> directory. > > Why would you make psql behave differently from our other command-line > clients? No, psql should not behave different from other clients. Sorry, I was under assumption that for other programs we will not take backend executable path. One other thing which is not clear to me is that how by calling some special/new API we can ensure that the path provided by user is a valid path, are we going to validate file given in 'standalone_backend' switch in some way? With Regards, Amit Kapila. EnterpriseDB: http://www.enterprisedb.com -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Thu, Nov 21, 2013 at 9:11 AM, Amit Kapila wrote: > On Thu, Nov 21, 2013 at 2:14 AM, Tom Lane wrote: >> Peter Eisentraut writes: >>> The argument elsewhere in this thread was that the reason for putting >>> this in the connection options was so that you do *not* have to patch up >>> every client to be able to use this functionality. If you have to add >>> separate options everywhere, then you might as well just have a separate >>> libpq function to initiate the session. >> >> Right, Andres was saying that we had to do both (special switches that >> lead to calling a special connection function). > >Doesn't the new option 'standalone_datadir' (which is already in > patch) a good candidate for special switch? >How does having one more new switch helps better? Here what I have in mind is that: a. In pg_dump or other internal utilities where we want to use this feature, they should call PQenableStart() or some other API before calling PQConnect() which will indicate that it wants to operate as a standalone mode. b. In psql, if user specifies this special switch ( 'standalone_datadir'), then internally we will call PQenableStart() and use postgres from same directory. So standalone_backend option will not be exposed through psql, but other internal tools can use it. With Regards, Amit Kapila. EnterpriseDB: http://www.enterprisedb.com -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
Amit Kapila writes: > Here what I have in mind is that: > a. In pg_dump or other internal utilities where we want to use this > feature, they should call PQenableStart() or some other API before > calling PQConnect() which will indicate that it wants to operate > as a standalone mode. > b. In psql, if user specifies this special switch ( > 'standalone_datadir'), then internally we will call PQenableStart() > and use postgres from same > directory. Why would you make psql behave differently from our other command-line clients? That seems bizarre. If we're going to use a special switch to enable standalone mode, it should be the same on every program that supports it. regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Thu, Nov 21, 2013 at 2:14 AM, Tom Lane wrote: > Peter Eisentraut writes: >> The argument elsewhere in this thread was that the reason for putting >> this in the connection options was so that you do *not* have to patch up >> every client to be able to use this functionality. If you have to add >> separate options everywhere, then you might as well just have a separate >> libpq function to initiate the session. > > Right, Andres was saying that we had to do both (special switches that > lead to calling a special connection function). Doesn't the new option 'standalone_datadir' (which is already in patch) a good candidate for special switch? How does having one more new switch helps better? With Regards, Amit Kapila. EnterpriseDB: http://www.enterprisedb.com -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Wed, Nov 20, 2013 at 05:38:14PM -0500, Gurjeet Singh wrote: > On Wed, Nov 20, 2013 at 3:44 PM, Tom Lane wrote: > > > To my mind, the "create a socket and hope nobody else can get to it" > approach is exactly one of the main things we're trying to avoid here. > If you'll recall, awhile back we had a big discussion about how pg_upgrade > could positively guarantee that nobody messed with the source database > while it was working, and we still don't have a bulletproof guarantee > there. I would like to fix that by making pg_upgrade use only standalone > backends to talk to the source database, never starting a real postmaster > at all. But if the standalone-pg_dump mode goes through a socket, we're > back to square one on that concern. > > > (I couldn't find the pg_upgrade-related thread mentioned above). > > I am not sure of the mechanics of this, but can we not launch the postmaster > with a random magic-cookie, and use that cookie while initiating the > connection > from libpq. The postmaster will then reject any connections that don't provide > the cookie. > > We do something similar to enable applications to send cancellation signals > (postmaster.c:Backend.cancel_key), just that it's establishing trust in the > opposite direction. The magic cookie can be tha application_name. I had pg_upgrade code to prevent anyone from connecting unless their application_name was "pg_upgrade", but the idea was rejected. -- Bruce Momjian http://momjian.us EnterpriseDB http://enterprisedb.com + Everyone has their own god. + -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On 2013-11-20 15:44:03 -0500, Tom Lane wrote: > In practice, as long as psql and pg_dump and pg_upgrade can do it, I > think we've covered most of the interesting bases. I'd say vacuumdb/reindexdb should be added to that list. In my experience xid wraparound and corrupted system indexes are the most frequent use-case of single user mode. Greetings, Andres Freund -- Andres Freund http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Wed, Nov 20, 2013 at 3:44 PM, Tom Lane wrote: > > To my mind, the "create a socket and hope nobody else can get to it" > approach is exactly one of the main things we're trying to avoid here. > If you'll recall, awhile back we had a big discussion about how pg_upgrade > could positively guarantee that nobody messed with the source database > while it was working, and we still don't have a bulletproof guarantee > there. I would like to fix that by making pg_upgrade use only standalone > backends to talk to the source database, never starting a real postmaster > at all. But if the standalone-pg_dump mode goes through a socket, we're > back to square one on that concern. > (I couldn't find the pg_upgrade-related thread mentioned above). I am not sure of the mechanics of this, but can we not launch the postmaster with a random magic-cookie, and use that cookie while initiating the connection from libpq. The postmaster will then reject any connections that don't provide the cookie. We do something similar to enable applications to send cancellation signals (postmaster.c:Backend.cancel_key), just that it's establishing trust in the opposite direction. Best regards, -- Gurjeet Singh http://gurjeet.singh.im/ EnterprsieDB Inc. www.enterprisedb.com
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
Peter Eisentraut writes: > The argument elsewhere in this thread was that the reason for putting > this in the connection options was so that you do *not* have to patch up > every client to be able to use this functionality. If you have to add > separate options everywhere, then you might as well just have a separate > libpq function to initiate the session. Right, Andres was saying that we had to do both (special switches that lead to calling a special connection function). I'm not terribly happy about that, because it will greatly constrain the set of programs that are able to connect to standalone backends --- but I think that there are some in this discussion who want that, anyway. In practice, as long as psql and pg_dump and pg_upgrade can do it, I think we've covered most of the interesting bases. To my mind, the "create a socket and hope nobody else can get to it" approach is exactly one of the main things we're trying to avoid here. If you'll recall, awhile back we had a big discussion about how pg_upgrade could positively guarantee that nobody messed with the source database while it was working, and we still don't have a bulletproof guarantee there. I would like to fix that by making pg_upgrade use only standalone backends to talk to the source database, never starting a real postmaster at all. But if the standalone-pg_dump mode goes through a socket, we're back to square one on that concern. regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Wed, Nov 20, 2013 at 3:32 PM, Peter Eisentraut wrote: > On 11/20/13, 3:24 PM, Robert Haas wrote: >> The point is that client applications should expose whether or not to >> set this function as a command-line switch separate from whatever they >> accept in terms of connection strings. So pg_dump should have a flag >> called --standalone-server or something like, and it should all >> PQenableStartServer() only when that flag is used. > > The argument elsewhere in this thread was that the reason for putting > this in the connection options was so that you do *not* have to patch up > every client to be able to use this functionality. If you have to add > separate options everywhere, then you might as well just have a separate > libpq function to initiate the session. Well, that's fair enough. I don't care much what the syntax is for invoking the postmaster this way, as long as it's reasonably convenient. I just want there to be one. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On 11/20/13, 3:24 PM, Robert Haas wrote: > The point is that client applications should expose whether or not to > set this function as a command-line switch separate from whatever they > accept in terms of connection strings. So pg_dump should have a flag > called --standalone-server or something like, and it should all > PQenableStartServer() only when that flag is used. The argument elsewhere in this thread was that the reason for putting this in the connection options was so that you do *not* have to patch up every client to be able to use this functionality. If you have to add separate options everywhere, then you might as well just have a separate libpq function to initiate the session. -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Wed, Nov 20, 2013 at 10:13 AM, Peter Eisentraut wrote: > On 11/14/13, 1:41 AM, Amit Kapila wrote: >>Security Concern >>- >>If a user can specify libpq connection options, he can now execute >> any file he wants by passing it as standalone_backend. >> >>Method to resolve Security concern >> >>If an application wants to allow these connection parameters to be >> used, it would need to do PQenableStartServer() first. If it doesn't, >> those connection parameters will be rejected. > > I don't think this really helps. You can't tell with reasonable effort > or certainty whether a given program is calling PQenableStartServer(), > so you cannot audit this from the outside. Also, someone could, > depending on circumstances, dynamically load a module that calls > PQenableStartServer(), thus circumventing this check. What?! The complaint is that somebody who only has access to set connection parameters could cause a server to be started. There's a tremendous gulf between "I can set the connection string" and "I can set LD_PRELOAD". If you can set LD_PRELOAD to a value of your choice, I'm pretty sure you can do things that are far more entertaining than calling a hypothetical PQenableStartServer() function. > And maybe before > long someone will patch up all drivers to call PQenableStartServer() > automatically, because why shouldn't I be able to run a standalone > backend from PHP or Ruby? Also, at some point at least, something like > phpPgAdmin called pg_dump internally, so you could imagine that in > situations like that, assuming that pg_dump called > PQenableStartServer(), with a little bit craftiness, you could expose > the execute-any-file hole through a web server. The point is that client applications should expose whether or not to set this function as a command-line switch separate from whatever they accept in terms of connection strings. So pg_dump should have a flag called --standalone-server or something like, and it should all PQenableStartServer() only when that flag is used. So if the user has a shell script that invokes pg_dump -d "$1", the user cannot contrive a server. If they write the script as pg_dump --standalone-server -d "$1", then they can, but by putting that option in there you pretty much bought the farm. Any program that calls that function unconditionally while at the same time accepting untrusted user input will be insecure, but chmod -R u+s /bin is insecure, too. That's why we don't do that. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On 11/20/13, 10:48 AM, Tom Lane wrote: > Perhaps more to the point, I think this approach actually breaks one of > the principal good-thing-in-emergencies attributes of standalone mode, > namely being sure that nobody but you can connect. With this, you're > right back to having a race condition as to whether your psql command > gets to the socket before somebody else. I don't disagree, except maybe about the relative gravity of the various competing concerns. But I want to see if we can split the proposed patch into smaller, more acceptable parts. There is elegance in being able to start a standalone backend from libpq connection parameters. But there are also security concerns and some general concerns about promoting an embedded database mode. If we allow single-user backends to speak protocol over sockets, then we have at least solved the problem of being able to use standard tools in emergency mode. And I don't think it precludes adding some of the other functionality later. -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On 11/20/13, 11:31 AM, Stephen Frost wrote: > Couldn't that be an issue for people who have multiple major versions of > binaries installed? In particular, the "default" on the system for psql > might be 9.3 while the cluster you're trying to recover may be 9.2. Of > course, in that case you might say to use the 9.2 psql, which would be > fair, but what if you're looking to get the data out of the 9.2 DB and > into the 9.3? In that case, we'd recommend using the 9.3 pg_dump. Right. And also, in emergency situations you might have a custom built postgres binary lying around in a separate path that includes a patch from a mailing list you're supposed to test or something. Best not to make that even more difficult. -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
Andres Freund writes: > On 2013-11-20 11:08:33 -0500, Tom Lane wrote: >> Andres Freund writes: >>> Something like PQstartSingleUser(dsn) returning a established connection >>> seems better to me. >> That just pushes the problem up a level --- how are you going to tell >> psql, pg_dump, or other programs that they should do that? > An explicit parameter. A program imo explicitly needs to be aware that a > PQconnect() suddenly starts forking and such. What if it is using > threads? What if it has it's own SIGCHLD handler for other business it's > doing? Hm. That's a fair point. I don't especially buy your other argument about additional connections --- if the program tries such, they'll just fail, which can hardly be said to be unexpected. But it's reasonable to worry that programs might need to be aware that they now have a child process. (It occurs to me that we'll need to provide a way to get the PID of the child, too.) regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On 2013-11-20 17:19:42 +0100, Andres Freund wrote: > > That just pushes the problem up a level --- how are you going to tell > > psql, pg_dump, or other programs that they should do that? > > An explicit parameter. A program imo explicitly needs to be aware that a > PQconnect() suddenly starts forking and such. What if it is using > threads? What if it has it's own SIGCHLD handler for other business it's > doing? Just as an example, consider what happens if somebody does pg_dump -j? Or somebody specifies such a connection for primary_conninfo? I am also not sure whether vacuumdb -a/reindexdb -a (both not unlikely commands to use for single user mode) are careful enough not to have parallel connections open? Greetings, Andres Freund -- Andres Freund http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
* Tom Lane (t...@sss.pgh.pa.us) wrote: > I think we'd be better off trying to fix the security issue by > constraining what can be executed as a "standalone backend". Would > it work to insist that psql/pg_dump launch the program named postgres > from the same bin directory they're in, rather than accepting a path > from the connection string? Couldn't that be an issue for people who have multiple major versions of binaries installed? In particular, the "default" on the system for psql might be 9.3 while the cluster you're trying to recover may be 9.2. Of course, in that case you might say to use the 9.2 psql, which would be fair, but what if you're looking to get the data out of the 9.2 DB and into the 9.3? In that case, we'd recommend using the 9.3 pg_dump. Basically, I'd suggest that we try and avoid things like "the binaries have to be in the same directory".. With regard to access to the socket, perhaps we create our own socket w/ 0600 and use that? Seems like it'd be sufficient to prevent the 'normal' users from getting into the DB while we're working on it. If there's two different individuals gettings into the same system and trying to start the same cluster as the same unix user, well.. I'm not convinced we'd be able to come up with a perfect solution to that anyway. Thanks, Stephen signature.asc Description: Digital signature
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On 2013-11-20 11:08:33 -0500, Tom Lane wrote: > Andres Freund writes: > > On 2013-11-20 10:48:20 -0500, Tom Lane wrote: > >> constraining what can be executed as a "standalone backend". Would > >> it work to insist that psql/pg_dump launch the program named postgres > >> from the same bin directory they're in, rather than accepting a path > >> from the connection string? > > > But why do we want to start the server through the connection string > > using PQconnectb() in the first place? That doesn't really seem right to > > me. > > Something like PQstartSingleUser(dsn) returning a established connection > > seems better to me. > > That just pushes the problem up a level --- how are you going to tell > psql, pg_dump, or other programs that they should do that? An explicit parameter. A program imo explicitly needs to be aware that a PQconnect() suddenly starts forking and such. What if it is using threads? What if it has it's own SIGCHLD handler for other business it's doing? Greetings, Andres Freund -- Andres Freund http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
Andres Freund writes: > On 2013-11-20 10:48:20 -0500, Tom Lane wrote: >> constraining what can be executed as a "standalone backend". Would >> it work to insist that psql/pg_dump launch the program named postgres >> from the same bin directory they're in, rather than accepting a path >> from the connection string? > But why do we want to start the server through the connection string > using PQconnectb() in the first place? That doesn't really seem right to > me. > Something like PQstartSingleUser(dsn) returning a established connection > seems better to me. That just pushes the problem up a level --- how are you going to tell psql, pg_dump, or other programs that they should do that? regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On 2013-11-20 10:48:20 -0500, Tom Lane wrote: > constraining what can be executed as a "standalone backend". Would > it work to insist that psql/pg_dump launch the program named postgres > from the same bin directory they're in, rather than accepting a path > from the connection string? But why do we want to start the server through the connection string using PQconnectb() in the first place? That doesn't really seem right to me. Something like PQstartSingleUser(dsn) returning a established connection seems better to me. Greetings, Andres Freund -- Andres Freund http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
Peter Eisentraut writes: > I would consider sidestepping this entire issue by having the > stand-alone backend create a Unix-domain socket and have a client > connect to that in the normal way. Hmm. But that requires the "stand-alone backend" to take on at least some properties of a postmaster; at the very least, it would need to accept some form of shutdown signal (not just EOF on its stdin). Perhaps more to the point, I think this approach actually breaks one of the principal good-thing-in-emergencies attributes of standalone mode, namely being sure that nobody but you can connect. With this, you're right back to having a race condition as to whether your psql command gets to the socket before somebody else. I think we'd be better off trying to fix the security issue by constraining what can be executed as a "standalone backend". Would it work to insist that psql/pg_dump launch the program named postgres from the same bin directory they're in, rather than accepting a path from the connection string? regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On 11/14/13, 1:41 AM, Amit Kapila wrote: >Security Concern >- >If a user can specify libpq connection options, he can now execute > any file he wants by passing it as standalone_backend. > >Method to resolve Security concern > >If an application wants to allow these connection parameters to be > used, it would need to do PQenableStartServer() first. If it doesn't, > those connection parameters will be rejected. I don't think this really helps. You can't tell with reasonable effort or certainty whether a given program is calling PQenableStartServer(), so you cannot audit this from the outside. Also, someone could, depending on circumstances, dynamically load a module that calls PQenableStartServer(), thus circumventing this check. And maybe before long someone will patch up all drivers to call PQenableStartServer() automatically, because why shouldn't I be able to run a standalone backend from PHP or Ruby? Also, at some point at least, something like phpPgAdmin called pg_dump internally, so you could imagine that in situations like that, assuming that pg_dump called PQenableStartServer(), with a little bit craftiness, you could expose the execute-any-file hole through a web server. I don't have a better idea right now how to set up these connection parameters in a way that you can only set them in certain "safe" circumstances. I would consider sidestepping this entire issue by having the stand-alone backend create a Unix-domain socket and have a client connect to that in the normal way. At least if you split the patch that way, you might alleviate some concerns of others about whether this patch is about fixing standalone mode vs. allowing using standalone mode with psql vs. making a fully embedded database. -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
Robert Haas writes: > On Fri, Nov 15, 2013 at 6:51 AM, Simon Riggs wrote: >> Not enough. This feature is clearly being suggested as a way to offer >> Postgres in embedded mode for users by a back door. Doing that forces >> us to turn off many of the server's features and we will take a huge >> step backwards in features, testing, maintainability of code and >> wasted community time. > That's not clear to me at all. IIRC, the original idea was Tom's, and > the idea is to make it possible to have, for example, a psql session > connected to a standalone database, which can't be done right now. I > don't use standalone mode much, but when I do, I'd sure like to have > the psql interface rather than the existing standalone mode interface. pg_dump from a standalone backend seems like another core use case. That's not just more convenience, it's functionality you just don't have right now. I think that it might someday be interesting to allow a full server to be started on-demand in this way. But the patch as proposed is not that, it's just a nicer interface to the existing standalone mode. regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Fri, Nov 15, 2013 at 6:51 AM, Simon Riggs wrote: > Not enough. This feature is clearly being suggested as a way to offer > Postgres in embedded mode for users by a back door. Doing that forces > us to turn off many of the server's features and we will take a huge > step backwards in features, testing, maintainability of code and > wasted community time. That's not clear to me at all. IIRC, the original idea was Tom's, and the idea is to make it possible to have, for example, a psql session connected to a standalone database, which can't be done right now. I don't use standalone mode much, but when I do, I'd sure like to have the psql interface rather than the existing standalone mode interface. I'm not aware that there's anything in this patch which targets any other use case; if there is, sure, rip it out. But let's not assume this is going in a bad direction, especially considering who it was that suggested the idea originally. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Fri, Nov 15, 2013 at 5:21 PM, Simon Riggs wrote: > On 14 November 2013 03:41, Amit Kapila wrote: > >> I have gone through the mail chain of this thread and tried to find >> the different concerns or open ends for this patch. > > Not enough. This feature is clearly being suggested as a way to offer > Postgres in embedded mode for users by a back door. Current patch doesn't have such facility and I don't think somebody can use it as an embedded database. > Doing that forces > us to turn off many of the server's features and we will take a huge > step backwards in features, testing, maintainability of code and > wasted community time. > > "No administrative hassles" is just a complete fiction. Admin will > become a huge burden for any user in this mode, which will bite the > community and cause us to waste much time redesigning the server to > operate on a single session. > > -1 from me What I could understand from your objection is that you don't want users to get the impression of this feature as an embedded database. I think as the patch stands, it doesn't have such facility, so advertising it as an substitute for embedded database would be anyway inappropriate. The use case is to provide a standalone mode which will be useful for cases where today --single mode is required/used and I think documenting the feature that way is the right way to proceed. If this addresses your concern, then we can proceed to discuss solutions for other concerns like security? With Regards, Amit Kapila. EnterpriseDB: http://www.enterprisedb.com -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Fri, Nov 15, 2013 at 6:06 AM, Dimitri Fontaine wrote: >> But: I very, very much agree with the other concerns around this. This >> should be a patch to fix single user mode, not one to make postgres into >> a single process database. It's not, and trying to make it by using >> single user mode for it will start to hinder development of normal >> postgres because we suddenly need to be concerned about performance and >> features in situations where we previously weren't. > > +1 > > Maybe what needs to happen to this patch is away to restrict its usage > to --single. I'm thinking that postgres --single maybe could be made to > fork the server process underneath the psql controler client process > transparently. I personally would prefer not to do that. My main non-administrative interest in this mode is doing things like benchmarking protocol overhead. I'm OK with not supporting (and optimizing) for single user code paths but I don't like the idea of building walls that serve no purpose other than to make it difficult for other people mess around. Just document strenuously that this mode is not intended for application bundling and move on... merlin -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On 15 November 2013 09:00, Andres Freund wrote: > This > should be a patch to fix single user mode, not one to make postgres into > a single process database. +1 -- Simon Riggs http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
Andres Freund writes: > I think fixing single user mode to work halfway reasonable is enough > justification for the feature. Having to deal with that when solving > critical issues is just embarassing. +1 > But: I very, very much agree with the other concerns around this. This > should be a patch to fix single user mode, not one to make postgres into > a single process database. It's not, and trying to make it by using > single user mode for it will start to hinder development of normal > postgres because we suddenly need to be concerned about performance and > features in situations where we previously weren't. +1 Maybe what needs to happen to this patch is away to restrict its usage to --single. I'm thinking that postgres --single maybe could be made to fork the server process underneath the psql controler client process transparently. Regards, -- Dimitri Fontaine http://2ndQuadrant.fr PostgreSQL : Expertise, Formation et Support -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On 2013-11-15 09:51:28 -0200, Simon Riggs wrote: > On 14 November 2013 03:41, Amit Kapila wrote: > > > I have gone through the mail chain of this thread and tried to find > > the different concerns or open ends for this patch. > > Not enough. This feature is clearly being suggested as a way to offer > Postgres in embedded mode for users by a back door. I think fixing single user mode to work halfway reasonable is enough justification for the feature. Having to deal with that when solving critical issues is just embarassing. > Doing that forces > us to turn off many of the server's features and we will take a huge > step backwards in features, testing, maintainability of code and > wasted community time. I think the patch as proposed actually reduces maintenance overhead since we don't have to deal with the strange separate codepaths for single user mode. But: I very, very much agree with the other concerns around this. This should be a patch to fix single user mode, not one to make postgres into a single process database. It's not, and trying to make it by using single user mode for it will start to hinder development of normal postgres because we suddenly need to be concerned about performance and features in situations where we previously weren't. Greetings, Andres Freund -- Andres Freund http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On 14 November 2013 03:41, Amit Kapila wrote: > I have gone through the mail chain of this thread and tried to find > the different concerns or open ends for this patch. Not enough. This feature is clearly being suggested as a way to offer Postgres in embedded mode for users by a back door. Doing that forces us to turn off many of the server's features and we will take a huge step backwards in features, testing, maintainability of code and wasted community time. "No administrative hassles" is just a complete fiction. Admin will become a huge burden for any user in this mode, which will bite the community and cause us to waste much time redesigning the server to operate on a single session. -1 from me > 4. Secondary connections for data access > >Proposal >--- >A single-user connection database with "no administrative hassles" > >Concerns >- >As this proposal will not allow any data it stores to be accessed > by another connection, so all forms of replication are excluded and > all maintenance actions force the database to be >unavailable for a period of time. Those two things are barriers of > the most major kind to anybody working in an enterprise with connected > data and devices. > >Suggestions for it's use or make it usable > >a. a usable & scriptable --single mode is justification enough. > Having to wait for hours just enter one more command because --single > doesn't support any scripts sucks. Especially in >recovery situations. >b. it's worth having this particular thing because it makes > pg_upgrade more robust. >c. some competing solutions already provide similar solution > (http://www.firebirdsql.org/manual/fbmetasecur-embedded.html). >d. we need to make sure that this isn't foreclosing the option of > having a multi-process environment with a single user connection. I > don't see that it is, but it might be wise to sketch >exactly how that case would work before accepting this. Why is not feasible to run a normal server with 1 connection. Are we really following what Firebird is doing? Why? > 6. Restricting operation's in single backend mode > >Serializable transactions could skip all the SSI predicate locking > and conflict checking when in single-connection mode. With only one > connection the transactions could never overlap, so >there would be no chance of serialization anomalies when running > snapshot isolation. > >It could be of use if someone had code they wanted to run under > both normal and single-connection modes. For single-connection only, > they could just choose REPEATABLE READ to >get exactly the same semantics. This is an example of my concern that we would begin optimising for the case of single user mode and encourage its use by users. This shows that the feature is not being suggested just for recovery. PostgreSQL has been designed from the ground up to support concurrency. If we begin optimising for single user mode it will take years to unpick our work and eventually we'll have a conflict and someone will say "we can't do that because it will be a problem in single user mode". > 7. Proposal related to maintainence activities > >For maintainence activities, in longer run, we can have a > postmaster process that isn't listening on any ports, but is managing > background processes in addition to a single child backend. > > As per my understanding, to complete this patch we need to > a. complete the work for #1, #2, #5 > b. #6 and #7 can be done as enhancements after the initial feature is > committed > c. need to decide what should we do for #3 and #4. Huh? Multi-process mode already works. Why would do we need "a solution" for the problem that single process mode uses only one process? Don't use it. -- Simon Riggs http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
Amit Kapila writes: > Any objections for adding this idea/patch to CF? You should certainly add it to the CF. You've listed lots of matters for review, but that's no reason to not get it in the queue to be reviewed. regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
I have gone through the mail chain of this thread and tried to find the different concerns or open ends for this patch. Summarisation of the discussion and concerns for this patch: 1. Security concern in interface 2. Security concern in Windows implementation 3. Handling of Ctrl-C/SIGTERM 4. Secondary connections for maintenance activities, replication 5. Windows Implementation - what should be behaviour for admin users? 6. Restricting operation's in single backend mode 7. Proposal related to maintenance activities Description of each concern -- 1. Security concern in interface - Interface --- $ psql "standalone_datadir = $PGDATA dbname = regression" There is another option "standalone_backend", which can be set to specify which postgres executable to launch. If the latter isn't specified, libpq defaults to trying the installation PGBINDIR that was selected by configure. Security Concern - If a user can specify libpq connection options, he can now execute any file he wants by passing it as standalone_backend. Method to resolve Security concern If an application wants to allow these connection parameters to be used, it would need to do PQenableStartServer() first. If it doesn't, those connection parameters will be rejected. 2. Security concern in Windows implementation - Interface --- PQcancel - In Unix, need to use kill(conn->postgres_pid, SIGINT) In Windows, pgkill(int pid, int sig) API can be used. Security concern --- pgkill is used to send cancel signal using pipe mechanism, so someone else can create a pipe with our name before we do (since we use the actual name - it's \\.\pipe\pgsinal_), by guessing what pid we will have. If that happens, we'll go into a loop and try to recreate it while logging a warning message to eventlog/stderr. (this happens for every backend). We can't throw an error on this and kill the backend because the pipe is created in the background thread not the main one. Some suggestions -- Once it is detected that already a same Named Pipe already exists, there can be following options: a. try to create with some other name, but in that case how to communicate the new name to client end of pipe. Some solution can be thought if this approach seems to be reasonable, though currently I don't have any in mind. b. give error, as creation of pipe is generally at beginning of process creation(backend) c. any other better solution? 3. Handling of Ctrl-C/SIGTERM Behaviour --- If you kill the client, the child postgres will see connection closure and will shut down. Concern -- will make scripting harder because you cannot start another single backend pg_dump before the old backend noticed it, checkpointed and shut down. It can happen if you forcibly kill pg_dump (or some other client) and then immediately try to start a new one, it's not clear how long you'll have to wait. Suggestions for alternatives for this case - a. There is no expectation that a standalone PG implementation would provide performance for a series of standalone sessions that is equivalent to what you'd get from a persistent server. If that scenario is what's important to you, you'd use a persistent server. b. An extra libpq call to handle this case can be helpful. 4. Secondary connections for data access Proposal --- A single-user connection database with "no administrative hassles" Concerns - As this proposal will not allow any data it stores to be accessed by another connection, so all forms of replication are excluded and all maintenance actions force the database to be unavailable for a period of time. Those two things are barriers of the most major kind to anybody working in an enterprise with connected data and devices. Suggestions for it's use or make it usable a. a usable & scriptable --single mode is justification enough. Having to wait for hours just enter one more command because --single doesn't support any scripts sucks. Especially in recovery situations. b. it's worth having this particular thing because it makes pg_upgrade more robust. c. some competing solutions already provide similar solution (http://www.firebirdsql.org/manual/fbmetasecur-embedded.html). d. we need to make sure that this isn't foreclosing the option of having a multi-process environment with a single user connection. I don't see that it is, but it might be wise to sketch exactly how that case would work before accepting this. 5. Windows Implementation - what
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
Tom Lane writes: >> I agree there is lots of demand for simple single-user databases and I >> wish that too. What I don't agree with is something that casts that >> requirement in stone by architecturally/permanently disallowing >> secondary connections. > > If you want secondary connections, then I think you want a postmaster. I would agree. I think you're both talking above each other, and that what Simon is worried about (but I haven't asked him about that before sending that email) is how to change the application setup to switch from single user mode to multi user mode. IIRC the way to implement single user mode in your application is quite low-level with this patch, so switching to multi-user mode is not about just changing the connection string, or is it? > The case that this patch is meant to address is one where there is only > one client application, period, and you'd rather that the database > starts and stops automatically with that application instead of needing > any management complexity. Now we can debate whether we want only one > process or multiple processes underneath the client application, but > I think the restriction to one client connection is a key *feature* > not a bug, precisely because it removes a whole bunch of user-visible > complexity that we cannot escape otherwise. Well I think your patch would be easier to accept as is if it was documented only as a psql friendly single-user mode. I would really welcome that. > embedded database. We've never had an adequate offering for those > people before. If we ratchet up the management complexity of "single > user" mode then it still won't be an adequate offering for them. Now, if we're talking about single user mode as in embedded database, I really do think this patch should include a solution to run online maintainance, logical and physical backups, replication, archiving and all the production grade features you expect from PostgreSQL. And then I understand Simon's POV about code complexity and bgworkers for examples, which will *need* to be taken care of in that solution. > I think this is nonsense. It's not critical; it's a very small patch > that provides a feature of interest to a limited audience. And I don't Yes, it's providing full psql capabilities where we only had that bizare postgres --single interface. Maybe it will make initdb and debugging it easier too. > believe it's foreclosing providing other operating modes later, unless > maybe people feel this is "almost good enough" and lose motivation to > work on those other operating modes. But if that happens, then I'd say > the demand for the other modes isn't as high as you think. Again, my concern on that point after reading Simon's comments is only about the production procedure you have to follow to switch your application from single user mode to multi user mode. Regards, -- Dimitri Fontaine http://2ndQuadrant.fr PostgreSQL : Expertise, Formation et Support -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
Preface: I think there's some great commentary here, and find myself agreeing pretty whole-heartedly. On Tue, Nov 13, 2012 at 2:45 PM, Simon Riggs wrote: > On 13 November 2012 17:38, Tom Lane wrote: > > Simon Riggs writes: > >> The most popular relational database in the world is Microsoft Access, > >> not MySQL. Access appears desirable because it allows a single user to > >> create and use a database (which is very good). But all business > >> databases have a requirement for at least one of: high availability, > >> multi-user access or downstream processing in other parts of the > >> business. > > > > That's a mighty sweeping claim, which you haven't offered adequate > > evidence for. The fact of the matter is that there is *lots* of demand > > for simple single-user databases, and what I'm proposing is at least a > > first step towards getting there. > > I agree there is lots of demand for simple single-user databases and I > wish that too. What I don't agree with is something that casts that > requirement in stone by architecturally/permanently disallowing > secondary connections. > > Evidence for claims: > * The whole Business Intelligence industry relies on being able to > re-purpose existing data, forming integrated webs of interconnecting > databases. All of that happens after the initial developers write the > first version of the database application. > * Everybody wants a remote backup, whether its for your mobile phone > contact list or your enterprise datastore. > > People are migrating away from embedded databases in droves for these > very reasons. > There seems to be a continuum of different sorts of scenarios of more-to-less concurrency that are desirable for some different reasons. From most-to-least, I can see: 1 - Obviously, there's the case that Postgres is eminently good at, of supporting many users concurrently using a database. We love that, let's not break it :-). 2 - We have found it useful to have some extra work processes that do some useful internal things, such as vacuuming, forcing background writes, collecting statistics. And an online backup requires having a second process. 3 - People doing embedded systems find it attractive to attach all the data to the singular user running the system. Witness the *heavy* deployment of SQLite on Android and iOS. People make an assumption that this is a single-process thing, but I am inclined to be a bit skeptical. What they *do* know is that it's convenient to not spawn extra processes and do IPC. That's not quite the same thing as it being a certainty that they definitely want not to have more than one process. 4 - There are times when there *is* certainty about not wanting there to be more than one process. When running pg_upgrade, or, at certain times, when doing streaming replication node status switches, one might have that certainty. Or when reindexing system tables, which needs single user mode. For us to conflate the 3rd and 4th items seems like a mistake to me. > > The main disadvantage of approaching this via the existing single-user > > mode is that you won't have any autovacuum, bgwriter, etc, support. > > But the flip side is that that lack of infrastructure is a positive > > advantage for certain admittedly narrow use-cases, such as disaster > > recovery and pg_upgrade. So while I agree that this isn't the only > > form of single-user mode that we'd like to support, I think it is *a* > > form we'd like to support, and I don't see why you appear to be against > > having it at all. > > I have no problem with people turning things off, I reject the idea > that we should encourage people to never be able to turn them back on. > Yep. That seems like conflating #2 with #4. It's mighty attractive to have a forcible "single process mode" to add safety to certain activities. I think we need a sharper knife, though, so we don't ablate off stuff like #2, just because someone imagined that "Must Have Single Process!!!" was the right doctrine. > > A more reasonable objection would be that we need to make sure that this > > isn't foreclosing the option of having a multi-process environment with > > a single user connection. I don't see that it is, but it might be wise > > to sketch exactly how that case would work before accepting this. > > Whatever we provide will become the norm. I don't have a problem with > you providing BOTH the proposed single user mode AND the multi-process > single user connection mode in this release. But if you provide just > one of them and its the wrong one, we will be severely hampered in the > future. > > Yes, I am very much against this project producing a new DBMS > architecture that works on top of PostgreSQL data files, yet prevents > maintenance, backup, replication and multi-user modes. > > I see this decision as a critical point for this project, so please > consider this objection and where it comes from. > I don't think we're necessarily *hu
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
Simon Riggs writes: > On 13 November 2012 17:38, Tom Lane wrote: >> ... The fact of the matter is that there is *lots* of demand >> for simple single-user databases, and what I'm proposing is at least a >> first step towards getting there. > I agree there is lots of demand for simple single-user databases and I > wish that too. What I don't agree with is something that casts that > requirement in stone by architecturally/permanently disallowing > secondary connections. If you want secondary connections, then I think you want a postmaster. We already have umpteen ways to limit who can connect (for example, putting the socket in a directory with limited access rights), and in that sort of situation I don't see why you'd really want a database that is only accessible when the "main" client is running. The case that this patch is meant to address is one where there is only one client application, period, and you'd rather that the database starts and stops automatically with that application instead of needing any management complexity. Now we can debate whether we want only one process or multiple processes underneath the client application, but I think the restriction to one client connection is a key *feature* not a bug, precisely because it removes a whole bunch of user-visible complexity that we cannot escape otherwise. > People are migrating away from embedded databases in droves for these > very reasons. [ shrug... ] If they don't want an embedded database, they won't want this either, but there are still plenty of people left who do want an embedded database. We've never had an adequate offering for those people before. If we ratchet up the management complexity of "single user" mode then it still won't be an adequate offering for them. > I see this decision as a critical point for this project, so please > consider this objection and where it comes from. I think this is nonsense. It's not critical; it's a very small patch that provides a feature of interest to a limited audience. And I don't believe it's foreclosing providing other operating modes later, unless maybe people feel this is "almost good enough" and lose motivation to work on those other operating modes. But if that happens, then I'd say the demand for the other modes isn't as high as you think. regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On 13 November 2012 17:38, Tom Lane wrote: > Simon Riggs writes: >> The most popular relational database in the world is Microsoft Access, >> not MySQL. Access appears desirable because it allows a single user to >> create and use a database (which is very good). But all business >> databases have a requirement for at least one of: high availability, >> multi-user access or downstream processing in other parts of the >> business. > > That's a mighty sweeping claim, which you haven't offered adequate > evidence for. The fact of the matter is that there is *lots* of demand > for simple single-user databases, and what I'm proposing is at least a > first step towards getting there. I agree there is lots of demand for simple single-user databases and I wish that too. What I don't agree with is something that casts that requirement in stone by architecturally/permanently disallowing secondary connections. Evidence for claims: * The whole Business Intelligence industry relies on being able to re-purpose existing data, forming integrated webs of interconnecting databases. All of that happens after the initial developers write the first version of the database application. * Everybody wants a remote backup, whether its for your mobile phone contact list or your enterprise datastore. People are migrating away from embedded databases in droves for these very reasons. > The main disadvantage of approaching this via the existing single-user > mode is that you won't have any autovacuum, bgwriter, etc, support. > But the flip side is that that lack of infrastructure is a positive > advantage for certain admittedly narrow use-cases, such as disaster > recovery and pg_upgrade. So while I agree that this isn't the only > form of single-user mode that we'd like to support, I think it is *a* > form we'd like to support, and I don't see why you appear to be against > having it at all. I have no problem with people turning things off, I reject the idea that we should encourage people to never be able to turn them back on. > A more reasonable objection would be that we need to make sure that this > isn't foreclosing the option of having a multi-process environment with > a single user connection. I don't see that it is, but it might be wise > to sketch exactly how that case would work before accepting this. Whatever we provide will become the norm. I don't have a problem with you providing BOTH the proposed single user mode AND the multi-process single user connection mode in this release. But if you provide just one of them and its the wrong one, we will be severely hampered in the future. Yes, I am very much against this project producing a new DBMS architecture that works on top of PostgreSQL data files, yet prevents maintenance, backup, replication and multi-user modes. I see this decision as a critical point for this project, so please consider this objection and where it comes from. -- Simon Riggs http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Tue, Nov 13, 2012 at 12:38 PM, Tom Lane wrote: > Simon Riggs writes: >> The most popular relational database in the world is Microsoft Access, >> not MySQL. Access appears desirable because it allows a single user to >> create and use a database (which is very good). But all business >> databases have a requirement for at least one of: high availability, >> multi-user access or downstream processing in other parts of the >> business. > > That's a mighty sweeping claim, which you haven't offered adequate > evidence for. The fact of the matter is that there is *lots* of demand > for simple single-user databases, and what I'm proposing is at least a > first step towards getting there. > > The main disadvantage of approaching this via the existing single-user > mode is that you won't have any autovacuum, bgwriter, etc, support. > But the flip side is that that lack of infrastructure is a positive > advantage for certain admittedly narrow use-cases, such as disaster > recovery and pg_upgrade. So while I agree that this isn't the only > form of single-user mode that we'd like to support, I think it is *a* > form we'd like to support, and I don't see why you appear to be against > having it at all. > > A more reasonable objection would be that we need to make sure that this > isn't foreclosing the option of having a multi-process environment with > a single user connection. I don't see that it is, but it might be wise > to sketch exactly how that case would work before accepting this. I'm not particularly excited about providing more single-user mode options, but I think it's worth having this particular thing because it makes pg_upgrade more robust. Whether we do anything else is something we can litigate when the time comes. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
Simon Riggs writes: > The most popular relational database in the world is Microsoft Access, > not MySQL. Access appears desirable because it allows a single user to > create and use a database (which is very good). But all business > databases have a requirement for at least one of: high availability, > multi-user access or downstream processing in other parts of the > business. That's a mighty sweeping claim, which you haven't offered adequate evidence for. The fact of the matter is that there is *lots* of demand for simple single-user databases, and what I'm proposing is at least a first step towards getting there. The main disadvantage of approaching this via the existing single-user mode is that you won't have any autovacuum, bgwriter, etc, support. But the flip side is that that lack of infrastructure is a positive advantage for certain admittedly narrow use-cases, such as disaster recovery and pg_upgrade. So while I agree that this isn't the only form of single-user mode that we'd like to support, I think it is *a* form we'd like to support, and I don't see why you appear to be against having it at all. A more reasonable objection would be that we need to make sure that this isn't foreclosing the option of having a multi-process environment with a single user connection. I don't see that it is, but it might be wise to sketch exactly how that case would work before accepting this. regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On 13 November 2012 13:05, Alvaro Herrera wrote: > Simon Riggs escribió: > >> > So even if this solution doesn't meet all requirements of single >> > process solution (and neither I think it is written to address all) >> > but can't we think of it as first version and then based on >> > requirements extend it to have other capabilities: >> > a. to have a mechnism for other background processes (autovacuum, >> > checkpoint, ..). >> > b. more needs to be thought of.. >> >> Why would we spend time trying to put back something that is already >> there? Why not simply avoid removing it in the first place? > > Actually, the whole point of this solution originally was just to serve > pg_upgrade needs, so that it doesn't have to start a complete postmaster > environment just to have to turn off most of what postmaster does, and > with enough protections to disallow everyone else from connecting. I don't see anything that pg_upgrade is doing that causes the need to support a special mode. From other people's comments it's clear that "single user mode" is desirable to many and *will* be widely deployed if we allow it. I support the wish to allow a database server to be limited by configuration to a single user. However, supporting a specifically targeted mode that presents single user as an architectural design/limitation is a regressive step that I am strongly opposed to. The most popular relational database in the world is Microsoft Access, not MySQL. Access appears desirable because it allows a single user to create and use a database (which is very good). But all business databases have a requirement for at least one of: high availability, multi-user access or downstream processing in other parts of the business. Businesses worldwide curse the difficulties caused by having critical business data in desktop databases. And worldwide, there are also many that don't understand the problems that disconnected data causes because they can't see past the initial benefit. The lessons from that are that its OK to start with a database used by a single person, but that database soon needs to allow access from multiple users or automated agents. Many database systems support embedded or single user mode as an architectural option. All of those systems cause headaches in all of the businesses where they are used. They also cause problems on small detached devices such as phones, because even on very small systems there is a requirement for multiple concurrently active processes each of which may need database access. PostgreSQL was designed from the ground up as a multi-user database. This is the very fact that puts us in a good position to become pervasive. A single database system that works the same on all devices, with useful replication to connect data together. The embedded or single mode concept has long been on the "do not want" list. I believe that is a completely rational and strongly desirable thing. Supporting multiple architectures is extra work, and the restrictive architecture bites people in the long term. The fact that its an "easy patch" is not a great argument for changing that position, and in fact, its not easy, since it comes with a request to make it work on Windows (= extra work). The "easy" bit is not proven since people are already starting to ask about bgwriter and autovacuum. In this release there is much work happening around providing additional autonomous agents (bgworker) and other work around flexible replication (BDR), all of which would be nullified by the introduction and eventual wide usage of a restrictive new architecture. Single user configuration option, yes. Architecturally limited special version of PostgreSQL, no. -- Simon Riggs http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
Simon Riggs escribió: > > So even if this solution doesn't meet all requirements of single > > process solution (and neither I think it is written to address all) > > but can't we think of it as first version and then based on > > requirements extend it to have other capabilities: > > a. to have a mechnism for other background processes (autovacuum, > > checkpoint, ..). > > b. more needs to be thought of.. > > Why would we spend time trying to put back something that is already > there? Why not simply avoid removing it in the first place? Actually, the whole point of this solution originally was just to serve pg_upgrade needs, so that it doesn't have to start a complete postmaster environment just to have to turn off most of what postmaster does, and with enough protections to disallow everyone else from connecting. -- Álvaro Herrerahttp://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On 13 November 2012 06:14, Amit kapila wrote: >>I get the installability thang, very very much, I just don't see the >>single process thing as the only solution. At very least an open >>minded analysis of the actual problem and ways of solving it is called >>for, not just reach for a close to hand solution. > > Some other usecase where I have seen it required is in telecom billing apps. > In telecom application where this solution works, needs other maintainence > connections as well. > Some of the reasons for its use are performance and less maintainence > overhead and also their data requirements are > also not so high. > So even if this solution doesn't meet all requirements of single process > solution (and neither I think it is written to address all) but can't we > think of it as first version and then based on requirements extend it to have > other capabilities: > a. to have a mechnism for other background processes (autovacuum, checkpoint, > ..). > b. more needs to be thought of.. Why would we spend time trying to put back something that is already there? Why not simply avoid removing it in the first place? -- Simon Riggs http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On 2012-11-12 19:21:28 +, Simon Riggs wrote: > On 10 September 2012 17:50, Tom Lane wrote: > > > The point of the proposal that I am making is to have a simple, > > low-maintenance solution for people who need a single-application > > database. A compromise somewhere in the middle isn't likely to be an > > improvement for anybody. For instance, if you want to have additional > > connections, you open up a whole collection of communication and > > authentication issues, which potential users of a single-application > > database don't want to cope with. > > So the proposal is to implement a database that can't ever have 2 or > more connections. > ... > It's almost impossible to purchase a CPU these days that doesn't have > multiple cores, so the whole single-process architecture is just dead. > Yes, we want Postgres installed everywhere, but this isn't the way to > achieve that. > > I agree we should allow a PostgreSQL installation to work for a single > user, but I don't see that requires other changes. This idea will > cause endless bugs, thinkos and severely waste our time. So without a > much better justification, I don't think we should do this. I personally think that a usable & scriptable --single mode is justification enough, even if you don't aggree with the other goals. Having to wait for hours just enter one more command because --single doesn't support any scripts sucks. Especially in recovery situations. I also don't think a single-backend without further child processes is all that helpful - but I think this might be a very useful stepping stone. Greetings, Andres Freund -- Andres Freund http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Tuesday, November 13, 2012 3:11 AM Simon Riggs wrote: On 12 November 2012 21:26, Merlin Moncure wrote: >> I couldn't disagree more. The patch is small, logical, and fixes an >> awful problem, namely that --single mode is basically unusable. As to >> your wider point (namely, that you can't connect to it, therefore it's >> bad), it has got to be refuted by numerous competing solutions in the >> market such as http://www.firebirdsql.org/manual/fbmetasecur-embedded.html, >> and many others. As far as I remember even MySQL provides such a mode. > Small is not an argument in favour, just a statement of ease, like >jumping off a cliff. i.e. lemmings. >> While it's not as common as it used to be, now and then a piece of >> software needs to distribute an application as part of a boxed >> product. Postgres is horrible at this and doesn't have to be; imagine >> how much easier the lives of poker tracker would be (for *some* of its >> users) with an integrated standalone mode: google 'poker tracker >> postgresql' and take a good long look at problems people face in this >> scenario. >I get the installability thang, very very much, I just don't see the >single process thing as the only solution. At very least an open >minded analysis of the actual problem and ways of solving it is called >for, not just reach for a close to hand solution. Some other usecase where I have seen it required is in telecom billing apps. In telecom application where this solution works, needs other maintainence connections as well. Some of the reasons for its use are performance and less maintainence overhead and also their data requirements are also not so high. So even if this solution doesn't meet all requirements of single process solution (and neither I think it is written to address all) but can't we think of it as first version and then based on requirements extend it to have other capabilities: a. to have a mechnism for other background processes (autovacuum, checkpoint, ..). b. more needs to be thought of.. With Regards, Amit Kapila. -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Monday, November 12, 2012 8:31 PM Merlin Moncure wrote: On Fri, Sep 14, 2012 at 6:42 AM, Amit kapila wrote: > On Tuesday, September 11, 2012 7:07 PM Amit Kapila wrote: > On Monday, September 10, 2012 8:20 PM Amit Kapila wrote: > On Sunday, September 09, 2012 1:37 PM Amit Kapila wrote: > On Friday, September 07, 2012 11:19 PM Tom Lane wrote: > Heikki Linnakangas writes: > > Would socketpair(2) be simpler? > I have started working on this patch to make it work on Windows. The 3 > main things to make it work are: > >>>The patch which contains Windows implementation as well is attached with >>>this mail. It contains changes related to following >>>1. waitpid >>>2. socketpair >>3. fork-exec > >>>The following is still left: >>> 1. Error handling in all paths > >> The modified version 2 contains error handling in all paths. > I didn't see that this patch was added to a commitfest -- should it > have been? I very much like Tom's proposed starting point for this > feature as a replacement for --single. Hate to see this die on the > vine. Would some testing on windows be what's needed to get the ball > rolling? After Windows implementation, I have done first level tests also to make sure it works. I think Tom is the right person to comment on how to see this patch move forward. I am not sure what's in his mind that he didn't provide the feedback or proceeded to complete it. Is it due to time or he might have forseen some design or usecase problem, if it's due to time then I think it can be persuaded. With Regards, Amit Kapila. -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On 12 November 2012 21:26, Merlin Moncure wrote: > I couldn't disagree more. The patch is small, logical, and fixes an > awful problem, namely that --single mode is basically unusable. As to > your wider point (namely, that you can't connect to it, therefore it's > bad), it has got to be refuted by numerous competing solutions in the > market such as http://www.firebirdsql.org/manual/fbmetasecur-embedded.html, > and many others. Small is not an argument in favour, just a statement of ease, like jumping off a cliff. i.e. lemmings. > While it's not as common as it used to be, now and then a piece of > software needs to distribute an application as part of a boxed > product. Postgres is horrible at this and doesn't have to be; imagine > how much easier the lives of poker tracker would be (for *some* of its > users) with an integrated standalone mode: google 'poker tracker > postgresql' and take a good long look at problems people face in this > scenario. I get the installability thang, very very much, I just don't see the single process thing as the only solution. At very least an open minded analysis of the actual problem and ways of solving it is called for, not just reach for a close to hand solution. I don't ever want to hear someone reject a patch cos it would mess up poker tracker. The point is it complicates the code, introduces restrictions into what is possible and is just more inertia onto development. -- Simon Riggs http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Mon, Nov 12, 2012 at 1:21 PM, Simon Riggs wrote: > On 10 September 2012 17:50, Tom Lane wrote: > >> The point of the proposal that I am making is to have a simple, >> low-maintenance solution for people who need a single-application >> database. A compromise somewhere in the middle isn't likely to be an >> improvement for anybody. For instance, if you want to have additional >> connections, you open up a whole collection of communication and >> authentication issues, which potential users of a single-application >> database don't want to cope with. > > So the proposal is to implement a database that can't ever have 2 or > more connections. And so any data it stores cannot ever be accessed by > another connection, so all forms of replication are excluded and all > maintenance actions force the database to be unavailable for a period > of time. Those two things are barriers of the most major kind to > anybody working in an enterprise with connected data and devices. The > only people that want this are people with a very short term view of > the purpose of their applications, and disregard for the value and > permanence of the data stored. They may not want to cope with those > issues *now* but they will later and won't thank us for implementing > it in a way that means it can never be achieved. > > To be honest, I can't believe I'm reading this. > > And worse, it's on our Don't Want list, and nobody has said stop. > > It's almost impossible to purchase a CPU these days that doesn't have > multiple cores, so the whole single-process architecture is just dead. > Yes, we want Postgres installed everywhere, but this isn't the way to > achieve that. > > I agree we should allow a PostgreSQL installation to work for a single > user, but I don't see that requires other changes. This idea will > cause endless bugs, thinkos and severely waste our time. So without a > much better justification, I don't think we should do this. I couldn't disagree more. The patch is small, logical, and fixes an awful problem, namely that --single mode is basically unusable. As to your wider point (namely, that you can't connect to it, therefore it's bad), it has got to be refuted by numerous competing solutions in the market such as http://www.firebirdsql.org/manual/fbmetasecur-embedded.html, and many others. While it's not as common as it used to be, now and then a piece of software needs to distribute an application as part of a boxed product. Postgres is horrible at this and doesn't have to be; imagine how much easier the lives of poker tracker would be (for *some* of its users) with an integrated standalone mode: google 'poker tracker postgresql' and take a good long look at problems people face in this scenario. merlin -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On 10 September 2012 17:50, Tom Lane wrote: > The point of the proposal that I am making is to have a simple, > low-maintenance solution for people who need a single-application > database. A compromise somewhere in the middle isn't likely to be an > improvement for anybody. For instance, if you want to have additional > connections, you open up a whole collection of communication and > authentication issues, which potential users of a single-application > database don't want to cope with. So the proposal is to implement a database that can't ever have 2 or more connections. And so any data it stores cannot ever be accessed by another connection, so all forms of replication are excluded and all maintenance actions force the database to be unavailable for a period of time. Those two things are barriers of the most major kind to anybody working in an enterprise with connected data and devices. The only people that want this are people with a very short term view of the purpose of their applications, and disregard for the value and permanence of the data stored. They may not want to cope with those issues *now* but they will later and won't thank us for implementing it in a way that means it can never be achieved. To be honest, I can't believe I'm reading this. And worse, it's on our Don't Want list, and nobody has said stop. It's almost impossible to purchase a CPU these days that doesn't have multiple cores, so the whole single-process architecture is just dead. Yes, we want Postgres installed everywhere, but this isn't the way to achieve that. I agree we should allow a PostgreSQL installation to work for a single user, but I don't see that requires other changes. This idea will cause endless bugs, thinkos and severely waste our time. So without a much better justification, I don't think we should do this. -- Simon Riggs http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Fri, Sep 14, 2012 at 6:42 AM, Amit kapila wrote: > On Tuesday, September 11, 2012 7:07 PM Amit Kapila wrote: > On Monday, September 10, 2012 8:20 PM Amit Kapila wrote: > On Sunday, September 09, 2012 1:37 PM Amit Kapila wrote: > On Friday, September 07, 2012 11:19 PM Tom Lane wrote: > Heikki Linnakangas writes: > Would socketpair(2) be simpler? > > > >I've not done anything yet about the potential security issues >associated with untrusted libpq connection strings. I think this is still at the proof-of-concept stage; in particular, it's probably time to see if we can make it work on Windows before we worry more about that. > >>> I have started working on this patch to make it work on Windows. The 3 > main things to make it work are: > >>The patch which contains Windows implementation as well is attached with this >>mail. It contains changes related to following >>1. waitpid >>2. socketpair >>3. fork-exec > >>The following is still left: >> 1. Error handling in all paths > > The modified version 2 contains error handling in all paths. I didn't see that this patch was added to a commitfest -- should it have been? I very much like Tom's proposed starting point for this feature as a replacement for --single. Hate to see this die on the vine. Would some testing on windows be what's needed to get the ball rolling? merlin -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Tuesday, September 11, 2012 7:07 PM Amit Kapila wrote: On Monday, September 10, 2012 8:20 PM Amit Kapila wrote: On Sunday, September 09, 2012 1:37 PM Amit Kapila wrote: On Friday, September 07, 2012 11:19 PM Tom Lane wrote: Heikki Linnakangas writes: Would socketpair(2) be simpler? I've not done anything yet about the potential security issues associated with untrusted libpq connection strings. I think this >>>is still at the proof-of-concept stage; in particular, it's probably >>> time to see if we can make it work on Windows before we worry more >>>about that. >> I have started working on this patch to make it work on Windows. The 3 main things to make it work are: >The patch which contains Windows implementation as well is attached with this >mail. It contains changes related to following >1. waitpid >2. socketpair >3. fork-exec >The following is still left: > 1. Error handling in all paths The modified version 2 contains error handling in all paths. With Regards, Amit Kapila*** a/src/backend/main/main.c --- b/src/backend/main/main.c *** *** 191,196 main(int argc, char *argv[]) --- 191,198 AuxiliaryProcessMain(argc, argv); /* does not return */ else if (argc > 1 && strcmp(argv[1], "--describe-config") == 0) GucInfoMain(); /* does not return */ + else if (argc > 1 && strncmp(argv[1], "--child=", 8) == 0) + ChildPostgresMain(argc, argv, get_current_username(progname)); /* does not return */ else if (argc > 1 && strcmp(argv[1], "--single") == 0) PostgresMain(argc, argv, get_current_username(progname)); /* does not return */ else *** a/src/backend/postmaster/postmaster.c --- b/src/backend/postmaster/postmaster.c *** *** 463,468 typedef struct --- 463,469 static void read_backend_variables(char *id, Port *port); static void restore_backend_variables(BackendParameters *param, Port *port); + static void read_standalone_child_variables(char *id, int *psock); #ifndef WIN32 static bool save_backend_variables(BackendParameters *param, Port *port); *** *** 4268,4273 ExitPostmaster(int status) --- 4269,4365 proc_exit(status); } + + /* + * ChildPostgresMain - start a new-style standalone postgres process + * + * This may not belong here, but it does share a lot of code with ConnCreate + * and BackendInitialize. Basically what it has to do is set up a + * MyProcPort structure and then hand off control to PostgresMain. + * Beware that not very much stuff is initialized yet. + * + * In the future it might be interesting to support a "standalone + * multiprocess" mode in which we have a postmaster process that doesn't + * listen for connections, but does supervise autovacuum, bgwriter, etc + * auxiliary processes. So that's another reason why postmaster.c might be + * the right place for this. + */ + void + ChildPostgresMain(int argc, char *argv[], const char *username) + { + Port *port; + #ifdef WIN32 + charparamHandleStr[32]; + #endif + + /* +* Fire up essential subsystems: error and memory management +*/ + MemoryContextInit(); + + /* +* Build a Port structure for the client connection +*/ + if (!(port = (Port *) calloc(1, sizeof(Port + ereport(FATAL, + (errcode(ERRCODE_OUT_OF_MEMORY), +errmsg("out of memory"))); + + /* +* GSSAPI specific state struct must exist even though we won't use it +*/ + #if defined(ENABLE_GSS) || defined(ENABLE_SSPI) + port->gss = (pg_gssinfo *) calloc(1, sizeof(pg_gssinfo)); + if (!port->gss) + ereport(FATAL, + (errcode(ERRCODE_OUT_OF_MEMORY), +errmsg("out of memory"))); + #endif + + #ifndef WIN32 + /* The file descriptor of the client socket is the argument of --child */ + if (sscanf(argv[1], "--child=%d", &port->sock) != 1) + ereport(FATAL, + (errcode(ERRCODE_INVALID_PARAMETER_VALUE), +errmsg("invalid argument for --child: \"%s\"", argv[1]))); + #else + /* The file descriptor of the client socket is the argument of --child */ + if (sscanf(argv[1], "--child=%s", paramHandleStr) != 1) + ereport(FATAL, + (errcode(ERRCODE_INVALID_PARAMETER_VALUE), +errmsg("invalid argument for --child: \"%s\"", argv[1]))); + + read_standalone_child_variables(paramHandleStr, &port->sock); + + #endif + + /* Default assumption about protocol to use */ + FrontendProtocol = port->proto = PG_PROTOCOL_LATEST; + + /* save process start time */ + port->SessionStartTime = GetCurrentTimestam
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Monday, September 10, 2012 8:20 PM Amit Kapila wrote: On Sunday, September 09, 2012 1:37 PM Amit Kapila wrote: On Friday, September 07, 2012 11:19 PM Tom Lane wrote: Heikki Linnakangas writes: >>> Would socketpair(2) be simpler? >>>I've not done anything yet about the potential security issues >>>associated with untrusted libpq connection strings. I think this >>is still at the proof-of-concept stage; in particular, it's probably >> time to see if we can make it work on Windows before we worry more >>about that. > I have started working on this patch to make it work on Windows. The 3 main things to make it work are: The patch which contains Windows implementation as well is attached with this mail. It contains changes related to following 1. waitpid 2. socketpair 3. fork-exec The following is still left: 1. Error handling in all paths 2. During test, I found if i try to run with admin user, it throws error but psql doesn't comes out. I will look into this issue. However as in previous mail discussion there is a decision pending whether in standalone mode we need admin user behavior. 3. Will do some more test in Windows. Currently I have prepared a patch on top of your changes, please let me know if that is okay. Also, it will be better for me if you can tell me how I can further contribute. With Regards, Amit Kapiladiff --git a/src/backend/main/main.c b/src/backend/main/main.c index 33c5a0a..968959b 100644 --- a/src/backend/main/main.c +++ b/src/backend/main/main.c @@ -191,6 +191,8 @@ main(int argc, char *argv[]) AuxiliaryProcessMain(argc, argv); /* does not return */ else if (argc > 1 && strcmp(argv[1], "--describe-config") == 0) GucInfoMain(); /* does not return */ + else if (argc > 1 && strncmp(argv[1], "--child=", 8) == 0) + ChildPostgresMain(argc, argv, get_current_username(progname)); /* does not return */ else if (argc > 1 && strcmp(argv[1], "--single") == 0) PostgresMain(argc, argv, get_current_username(progname)); /* does not return */ else diff --git a/src/backend/postmaster/postmaster.c b/src/backend/postmaster/postmaster.c index 73520a6..c5730bd 100644 --- a/src/backend/postmaster/postmaster.c +++ b/src/backend/postmaster/postmaster.c @@ -463,6 +463,7 @@ typedef struct static void read_backend_variables(char *id, Port *port); static void restore_backend_variables(BackendParameters *param, Port *port); +static void read_standalone_child_variables(char *id, int *psock); #ifndef WIN32 static bool save_backend_variables(BackendParameters *param, Port *port); @@ -4268,6 +4269,97 @@ ExitPostmaster(int status) proc_exit(status); } + +/* + * ChildPostgresMain - start a new-style standalone postgres process + * + * This may not belong here, but it does share a lot of code with ConnCreate + * and BackendInitialize. Basically what it has to do is set up a + * MyProcPort structure and then hand off control to PostgresMain. + * Beware that not very much stuff is initialized yet. + * + * In the future it might be interesting to support a "standalone + * multiprocess" mode in which we have a postmaster process that doesn't + * listen for connections, but does supervise autovacuum, bgwriter, etc + * auxiliary processes. So that's another reason why postmaster.c might be + * the right place for this. + */ +void +ChildPostgresMain(int argc, char *argv[], const char *username) +{ + Port *port; +#ifdef WIN32 + charparamHandleStr[32]; +#endif + + /* +* Fire up essential subsystems: error and memory management +*/ + MemoryContextInit(); + + /* +* Build a Port structure for the client connection +*/ + if (!(port = (Port *) calloc(1, sizeof(Port + ereport(FATAL, + (errcode(ERRCODE_OUT_OF_MEMORY), +errmsg("out of memory"))); + + /* +* GSSAPI specific state struct must exist even though we won't use it +*/ +#if defined(ENABLE_GSS) || defined(ENABLE_SSPI) + port->gss = (pg_gssinfo *) calloc(1, sizeof(pg_gssinfo)); + if (!port->gss) + ereport(FATAL, + (errcode(ERRCODE_OUT_OF_MEMORY), +errmsg("out of memory"))); +#endif + +#ifndef WIN32 + /* The file descriptor of the client socket is the argument of --child */ + if (sscanf(argv[1], "--child=%d", &port->sock) != 1) + ereport(FATAL, + (errcode(ERRCODE_INVALID_PARAMETER_VALUE), +errmsg("invalid argument for --child: \"%s\"", argv[1]))); +#else +/* The file descriptor of the client socket is the argument of --child */ + if (sscanf(argv[1], "--child=%s", paramHandleStr) != 1) + ereport(FATAL, + (errcode
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Mon, Sep 10, 2012 at 9:59 AM, Josh Berkus wrote: > >> The point of the proposal that I am making is to have a simple, >> low-maintenance solution for people who need a single-application >> database. A compromise somewhere in the middle isn't likely to be an >> improvement for anybody. For instance, if you want to have additional >> connections, you open up a whole collection of communication and >> authentication issues, which potential users of a single-application >> database don't want to cope with. > > Yes, exactly. > > In fact, most of the folks who would want an embedded PostgreSQL either > want no authentication at all, or only a single password. So supporting > authentication options other than trust or md5 is not required, or > desired AFAIK. I agree people who want embedded postgres probably want no authentication. For the sake of test/local development use cases, I still question the usefulness of a quasi-embeddable mode that does not support multiple executors as SQLite does -- it's pretty murderous for the testing use case for a number of people if they intend to have parity with their production environment. I could see this being useful for, say, iOS (although there is a definite chance that one will want to use multiple threads/simultaneous xacts in applications), or a network router, except that pg_xlog and the catalog are rather enormous for those use cases. So while the embedded use case is really appealing in general, I'm still intuitively quite skeptical of the omission of multi-executor support, especially considering that people have gotten used to being able to that with the incredibly popular SQLite. Could EXEC_BACKEND be repurposed -- even on *nix-- to make this work someday? -- fdr -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
Josh Berkus wrote: > In fact, most of the folks who would want an embedded PostgreSQL > either want no authentication at all, or only a single password. > So supporting authentication options other than trust or md5 is > not required, or desired AFAIK. I don't know whether it's worth the trouble of doing so, but serializable transactions could skip all the SSI predicate locking and conflict checking when in single-connection mode. With only one connection the transactions could never overlap, so there would be no chance of serialization anomalies when running snapshot isolation. The reason I wonder whether it is worth the trouble is that it would only really matter if someone had code they wanted to run under both normal and single-connection modes. For single-connection only, they could just choose REPEATABLE READ to get exactly the same semantics. -Kevin -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
> The point of the proposal that I am making is to have a simple, > low-maintenance solution for people who need a single-application > database. A compromise somewhere in the middle isn't likely to be an > improvement for anybody. For instance, if you want to have additional > connections, you open up a whole collection of communication and > authentication issues, which potential users of a single-application > database don't want to cope with. Yes, exactly. In fact, most of the folks who would want an embedded PostgreSQL either want no authentication at all, or only a single password. So supporting authentication options other than trust or md5 is not required, or desired AFAIK. -- Josh Berkus PostgreSQL Experts Inc. http://pgexperts.com -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
Gurjeet Singh writes: > On Mon, Sep 10, 2012 at 11:43 AM, Heikki Linnakangas wrote: >> [scratches head] How's that different from the normal postmaster mode? > As I described in later paragraphs, it'd behave like an embedded database, > like SQLite etc., so the database will startup and shutdown with the > application, and provide other advantages we're currently trying to > provide, like zero-maintenance. But it will not mandate that only one > application talk to it at a time, and allow as many applications as it > would in postmaster mode. So the database would be online as long as any > application is connected to it, and it will shutdown when the last > application disconnects. I am having a hard time getting excited about that. To me it sounds like it's a regular postmaster, except with a response-time problem for connections that occur when there had been no active client before. The point of the proposal that I am making is to have a simple, low-maintenance solution for people who need a single-application database. A compromise somewhere in the middle isn't likely to be an improvement for anybody. For instance, if you want to have additional connections, you open up a whole collection of communication and authentication issues, which potential users of a single-application database don't want to cope with. > As being implemented right now, there's very little difference between > --single and --child modes. I guess I am asking for a --child mode > implementation that is closer to a postmaster than --single. There are good reasons for wanting something that is closer to --single: pg_upgrade being one, and having a friendlier user interface for disaster recovery in --single mode being another. In these cases, you not only don't need the capability for additional applications to connect, it is actually important that it's impossible for them to do so. regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Mon, Sep 10, 2012 at 11:43 AM, Heikki Linnakangas wrote: > On 10.09.2012 18:12, Gurjeet Singh wrote: > >> On Sun, Sep 2, 2012 at 8:23 PM, Tom Lane wrote: >> >> Notably, while the lack of any background processes is just what you want >>> for pg_upgrade and disaster recovery, an ordinary application is probably >>> going to want to rely on autovacuum; and we need bgwriter and other >>> background processes for best performance. So I'm speculating about >>> having a postmaster process that isn't listening on any ports, but is >>> managing background processes in addition to a single child backend. >>> That's for another day though. >>> >> >> Since we are forking a child process anyway, and potentially other >> auxiliary processes too, would it make sense to allow multiple backends >> too >> (allow multiple local applications connect to this instance)? I believe (I >> may be wrong) that embedded databases (SQLLite et. al.) use a library >> interface, in that the application makes a library call and waits for that >> API call to finish (unless, of course, the library supports async >> operations or the application uses threads). The implementation you are >> proposing uses socket communication, which lends itself very easily to >> client-server model, and if possible, it should be leveraged to provide >> for >> multiple applications talking to one local DB. >> > > [scratches head] How's that different from the normal postmaster mode? As I described in later paragraphs, it'd behave like an embedded database, like SQLite etc., so the database will startup and shutdown with the application, and provide other advantages we're currently trying to provide, like zero-maintenance. But it will not mandate that only one application talk to it at a time, and allow as many applications as it would in postmaster mode. So the database would be online as long as any application is connected to it, and it will shutdown when the last application disconnects. As being implemented right now, there's very little difference between --single and --child modes. I guess I am asking for a --child mode implementation that is closer to a postmaster than --single. Best regards, -- Gurjeet Singh http://gurjeet.singh.im/
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On 10.09.2012 18:12, Gurjeet Singh wrote: On Sun, Sep 2, 2012 at 8:23 PM, Tom Lane wrote: Notably, while the lack of any background processes is just what you want for pg_upgrade and disaster recovery, an ordinary application is probably going to want to rely on autovacuum; and we need bgwriter and other background processes for best performance. So I'm speculating about having a postmaster process that isn't listening on any ports, but is managing background processes in addition to a single child backend. That's for another day though. Since we are forking a child process anyway, and potentially other auxiliary processes too, would it make sense to allow multiple backends too (allow multiple local applications connect to this instance)? I believe (I may be wrong) that embedded databases (SQLLite et. al.) use a library interface, in that the application makes a library call and waits for that API call to finish (unless, of course, the library supports async operations or the application uses threads). The implementation you are proposing uses socket communication, which lends itself very easily to client-server model, and if possible, it should be leveraged to provide for multiple applications talking to one local DB. [scratches head] How's that different from the normal postmaster mode? - Heikki -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Mon, Sep 10, 2012 at 11:12 AM, Gurjeet Singh wrote: > On Sun, Sep 2, 2012 at 8:23 PM, Tom Lane wrote: >> >> Notably, while the lack of any background processes is just what you want >> for pg_upgrade and disaster recovery, an ordinary application is probably >> going to want to rely on autovacuum; and we need bgwriter and other >> background processes for best performance. So I'm speculating about >> having a postmaster process that isn't listening on any ports, but is >> managing background processes in addition to a single child backend. >> That's for another day though. > > > Since we are forking a child process anyway, and potentially other auxiliary > processes too, would it make sense to allow multiple backends too (allow > multiple local applications connect to this instance)? I believe (I may be > wrong) that embedded databases (SQLLite et. al.) use a library interface, in > that the application makes a library call and waits for that API call to > finish (unless, of course, the library supports async operations or the > application uses threads). The implementation you are proposing uses socket > communication, which lends itself very easily to client-server model, and if > possible, it should be leveraged to provide for multiple applications > talking to one local DB. > > I have this use case in mind: An application is running using this > interface, and an admin now wishes to do some maintenance, or inspect > something, so they can launch local pgAdmin using the same connection string > as used by the original application. This will allow an admin to perform > tuning, etc. without having to first shutdown the application. > > Here's how this might impact the design (I may very well be missing many > other things, and I have no idea if this is implementable or not): > > .) Database starts when the first such application is launched. > .) Database shuts down when the last such application disconnects. > .) Postgres behaves much like a regular Postgres installation, except that > it does not accept connections over TCP/IP or Unix Doamin Sockets. > .) The above implies that we use regular Postmaster machinery, and not the > --sinlgle machinery. > .) Second and subsequent applications use the postmaster.pid (or something > similar) to find an already running instance, and connect to it. > .) There's a race condition where the second application is starting up, > hoping to connect to an already running insatnce, but the first application > disconnects (and hence shuts down the DB) before the second one can > successfully connect. > > I haven't thought much about the security implications of this yet. Maybe > the socket permissions would restrict an unauthorized user user from > connecting to this instance. That's kind of the reason why I suggested up thread tring to decouple the *starting* of the backend with the "options" to PQ connect... A "Helper function" in libpq could easily start the backend, and possibly return a conninfostring to give PQconnectdb... But if they are decoupled, I could easily envision an app that "pauses" it's use of the backend to allow some other libpq access to it for a period. You'd have to "trust" whatever else you let talk on the FD to the backend, but it might be useful... -- Aidan Van Dyk Create like a god, ai...@highrise.ca command like a king, http://www.highrise.ca/ work like a slave. -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Sun, Sep 2, 2012 at 8:23 PM, Tom Lane wrote: > Notably, while the lack of any background processes is just what you want > for pg_upgrade and disaster recovery, an ordinary application is probably > going to want to rely on autovacuum; and we need bgwriter and other > background processes for best performance. So I'm speculating about > having a postmaster process that isn't listening on any ports, but is > managing background processes in addition to a single child backend. > That's for another day though. > Since we are forking a child process anyway, and potentially other auxiliary processes too, would it make sense to allow multiple backends too (allow multiple local applications connect to this instance)? I believe (I may be wrong) that embedded databases (SQLLite et. al.) use a library interface, in that the application makes a library call and waits for that API call to finish (unless, of course, the library supports async operations or the application uses threads). The implementation you are proposing uses socket communication, which lends itself very easily to client-server model, and if possible, it should be leveraged to provide for multiple applications talking to one local DB. I have this use case in mind: An application is running using this interface, and an admin now wishes to do some maintenance, or inspect something, so they can launch local pgAdmin using the same connection string as used by the original application. This will allow an admin to perform tuning, etc. without having to first shutdown the application. Here's how this might impact the design (I may very well be missing many other things, and I have no idea if this is implementable or not): .) Database starts when the first such application is launched. .) Database shuts down when the last such application disconnects. .) Postgres behaves much like a regular Postgres installation, except that it does not accept connections over TCP/IP or Unix Doamin Sockets. .) The above implies that we use regular Postmaster machinery, and not the --sinlgle machinery. .) Second and subsequent applications use the postmaster.pid (or something similar) to find an already running instance, and connect to it. .) There's a race condition where the second application is starting up, hoping to connect to an already running insatnce, but the first application disconnects (and hence shuts down the DB) before the second one can successfully connect. I haven't thought much about the security implications of this yet. Maybe the socket permissions would restrict an unauthorized user user from connecting to this instance. -- Gurjeet Singh http://gurjeet.singh.im/
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Sunday, September 09, 2012 1:37 PM Amit Kapila wrote: On Friday, September 07, 2012 11:19 PM Tom Lane wrote: Heikki Linnakangas writes: >>> Would socketpair(2) be simpler? >>I've not done anything yet about the potential security issues >>associated with untrusted libpq connection strings. I think this >>is still at the proof-of-concept stage; in particular, it's probably >> time to see if we can make it work on Windows before we worry more >>about that. > I have started working on this patch to make it work on Windows. The 3 main things to make it work are: > 1. Windows equivalent for socketpair - This as suggested previously in this thread earlier code of pgpipe can suffice > the need. Infact I have checked on net as well, most implementations are similar to pgpipe implementation. So I > prefered to use the existing code which was removed. > 2. Windows equivalent for fork-execv - This part can be done by CreateProcess,it can be similar to internal_forkexec > except for path where it uses shared memory to pass parameters, I am trying to directly pass parameters to > CreateProcess. Directly passing parameters doesn't suffice for all parameters, as for socket we need to duplicate the socket using WSADuplicateSocket() which returns little big structure which is better to be passed via shared memory. > 3. Windows equivalent for waitpid - Actually there can be 2 ways to accomplish this > a. use waitforsingleobject with process handle, but in some places it is mentioned it might not work for all > windows versions. Can someone pls confirm about. I shall try on my PC to test the same. > b. use existing infrastructure of waitpid, however it is not for single process and it might need some changes to > make it work for single process or may be we can use it directly. However currently it is in postmaster.c, so >it need to be moved so that we can access it from fe-connect.c in libpq as well. > c. suggest if you know of other ways to handle it or which from above 2 would be better? I have used method - a (waitforsingleobject) and it worked fine. With the above implementation, it is working on Windows. Now the work left is as follows: 1. Refactoring of code 2. Error handling in paths 3. Check if anything is missing and implement for same. 4. Test the patch for Windows. Any comments/suggestions? With Regards, Amit Kapila -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Sunday, September 09, 2012 8:46 PM Tom Lane wrote: Amit kapila writes: >> 1. does this follow the behavior that admin users will not be allowed to invoke postgres child process? > That's an interesting question. I'm not sure if we'd want to disable > the no-root check on the Unix side, but it might make sense to. But > this has no bearing on what libpq does, does it? No, it has no bearing with what libpq does. Its only related to postgres invocation, because as soon as postgres get invoked, it first checks about root. If we want the root check can be avoided easily by checking argv. >> 2. to find standalone_backend incase user didn't input, do we need mechanism similar to getInstallationPaths()? > No. There is no reason at all for libpq to think it is part of a > postgres-supplied program, so it can't use any relative-path tricks, > even if it had the program's argv[0] which it does not. Either the > compiled-in path works, or the user has to give one. > (But having said that, if Windows provides a way for a DLL to find > out its own filesystem location, maybe relative path from there would > work.) At first, I shall go ahead with the current way (Either the compiled-in path works, or the user has to give one.). -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
Hi, On Wednesday, September 05, 2012 06:00:18 PM Tom Lane wrote: > "anara...@anarazel.de" writes: > > I am not saying its bad that it is slower, that's absolutely OK. Just > > that it will take a variable amount of time till you can run pgdump > > again and its not easily detectable without looping and trying again. > > Well, that's why the proposed libpq code is written to wait for the > child postgres to exit when closing the connection. > > Admittedly, if you forcibly kill pg_dump (or some other client) and then > immediately try to start a new one, it's not clear how long you'll have > to wait. Yep, thats the case I was talking about upthread ;) On Monday, September 03, 2012 11:04:15 PM Andres Freund wrote: > Don't we expect some more diligence in applications using this against > letting such a child continue to run after ctrl-c/SIGTERMing e.g. pg_dump > in comparison to closing a normal database connection? I would really expect any postgres supplied core tool to try to handle that case. Thats not exactly trivial to do right and requires cooperation of the application. Because that requires knowledge about the operation mode in those anyway I don't really see an extra libpq call as problematic... > But so what? Anything we might do in this space is going to > have pluses and minuses. True. Greetings, Andres -- Andres Freund http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
Amit kapila writes: > 1. does this follow the behavior that admin users will not be allowed to > invoke postgres child process? That's an interesting question. I'm not sure if we'd want to disable the no-root check on the Unix side, but it might make sense to. But this has no bearing on what libpq does, does it? > 2. to find standalone_backend incase user didn't input, do we need mechanism > similar to getInstallationPaths()? No. There is no reason at all for libpq to think it is part of a postgres-supplied program, so it can't use any relative-path tricks, even if it had the program's argv[0] which it does not. Either the compiled-in path works, or the user has to give one. (But having said that, if Windows provides a way for a DLL to find out its own filesystem location, maybe relative path from there would work.) regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Friday, September 07, 2012 11:19 PM Tom Lane wrote: Heikki Linnakangas writes: >> Would socketpair(2) be simpler? >I've not done anything yet about the potential security issues >associated with untrusted libpq connection strings. I think this >is still at the proof-of-concept stage; in particular, it's probably > time to see if we can make it work on Windows before we worry more >about that. I have started working on this patch to make it work on Windows. The 3 main things to make it work are: 1. Windows equivalent for socketpair - This as suggested previously in this thread earlier code of pgpipe can suffice the need. Infact I have checked on net as well, most implementations are similar to pgpipe implementation. So I prefered to use the existing code which was removed. 2. Windows equivalent for fork-execv - This part can be done by CreateProcess,it can be similar to internal_forkexec except for path where it uses shared memory to pass parameters, I am trying to directly pass parameters to CreateProcess. 3. Windows equivalent for waitpid - Actually there can be 2 ways to accomplish this a. use waitforsingleobject with process handle, but in some places it is mentioned it might not work for all windows versions. Can someone pls confirm about. I shall try on my PC to test the same. b. use existing infrastructure of waitpid, however it is not for single process and it might need some changes to make it work for single process or may be we can use it directly. However currently it is in postmaster.c, so it need to be moved so that we can access it from fe-connect.c in libpq as well. c. suggest if you know of other ways to handle it or which from above 2 would be better? Some other doubts: 1. does this follow the behavior that admin users will not be allowed to invoke postgres child process? 2. to find standalone_backend incase user didn't input, do we need mechanism similar to getInstallationPaths()? Any other comments/suggestions? With Regards, Amit Kapila. -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
A Dijous, 6 de setembre de 2012 00:30:53, Josh Berkus va escriure: > > In general I think the selling point for such a feature would be "no > > administrative hassles", and I believe that has to go not only for the > > end-user experience but also for the application-developer experience. > > If you have to manage checkpointing and vacuuming in the application, > > you're probably soon going to look for another database. > > Well, don't discount the development/testing case. If you do agile or > TDD (a lot of people do), you often have a workload which looks like: > > 1) Start framework > 2) Start database > 3) Load database with test data > 4) Run tests > 5) Print results > 6) Shut down database > > In a case like that, you can live without checkpointing, even; the > database is ephemeral. > > In other words, let's make this a feature and document it for use in > testing, and that it's not really usable for production embedded apps yet. +1. Some projects such as tryton would benefit from this feature. -- Albert Cervera i Areny http://www.NaN-tic.com Tel: +34 93 553 18 03 http://twitter.com/albertnan http://www.nan-tic.com/blog -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Wed, Sep 5, 2012 at 10:29 PM, Andrew Dunstan wrote: > > On 09/05/2012 10:14 PM, Tom Lane wrote: > >> >> The people who would be interested in this are currently using something >> like SQLite within a single application program. >> > > Exactly. I think it's worth stating that this has a HUGE potential > audience, and if we can get to this the deployment of Postgres could > mushroom enormously. I'm really quite excited about it. > /me shares the feeling :) -- Gurjeet Singh
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On 9/2/12 7:23 PM, Tom Lane wrote: 4. As coded, the backend assumes the incoming pipe is on its FD 0 and the outgoing pipe is on its FD 1. This made the command line simple but I'm having second thoughts about it: if anything inside the backend tries to read stdin or write stdout, unpleasant things will happen. It might be better to not reassign the pipe FD numbers. In that case we'd have to pass them on the command line, so that the syntax would be something like "postgres --child 4,5 -D pgdata ...". Would it be sufficient to just hard-code the FD's to use? I'm not sure why someone would need to change them, so long as we steer clear of STD(IN|OUT|ERR)... -- Jim C. Nasby, Database Architect j...@nasby.net 512.569.9461 (cell) http://jim.nasby.net -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Friday, September 07, 2012 11:21:00 PM Merlin Moncure wrote: > On Thu, Sep 6, 2012 at 12:56 PM, Jeff Davis wrote: > > On Wed, 2012-09-05 at 17:03 -0400, Tom Lane wrote: > >> In general I think the selling point for such a feature would be "no > >> administrative hassles", and I believe that has to go not only for the > >> end-user experience but also for the application-developer experience. > >> If you have to manage checkpointing and vacuuming in the application, > >> you're probably soon going to look for another database. > > > > Maybe there could be some hooks (e.g., right after completing a > > statement) that see whether a vacuum or checkpoint is required? VACUUM > > can't be run in a transaction block[1], so there are some details to > > work out, but it might be a workable approach. > > If it was me, I'd want finer grained control of if/when automatic > background optimization work happened. Something like > DoBackgroundWork(int forThisManySeconds). Of course, for that to > work, we'd need to have resumable vacuum. > > I like the idea of keeping everything single threaded. To me this path seems to be the best way to never get the feature at all... Andres -- Andres Freund http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Thu, Sep 6, 2012 at 12:56 PM, Jeff Davis wrote: > On Wed, 2012-09-05 at 17:03 -0400, Tom Lane wrote: >> In general I think the selling point for such a feature would be "no >> administrative hassles", and I believe that has to go not only for the >> end-user experience but also for the application-developer experience. >> If you have to manage checkpointing and vacuuming in the application, >> you're probably soon going to look for another database. > > Maybe there could be some hooks (e.g., right after completing a > statement) that see whether a vacuum or checkpoint is required? VACUUM > can't be run in a transaction block[1], so there are some details to > work out, but it might be a workable approach. If it was me, I'd want finer grained control of if/when automatic background optimization work happened. Something like DoBackgroundWork(int forThisManySeconds). Of course, for that to work, we'd need to have resumable vacuum. I like the idea of keeping everything single threaded. merlin -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
Heikki Linnakangas writes: > On 07.09.2012 10:49, Tom Lane wrote: >> I'm a bit tempted though to pull out and apply the portions of the >> patch that replace libpq's assorted ad-hoc closesocket() calls with >> a centralized pqDropConnection routine. I think that's probably a good >> idea independently of this feature. > Sounds good. Done; here's the rebased patch. regards, tom lane diff --git a/src/backend/main/main.c b/src/backend/main/main.c index 33c5a0a4e645624515016397da0011423d993c70..968959b85ef53aa2ec0ef8c5c5a9bc544291bfe5 100644 *** a/src/backend/main/main.c --- b/src/backend/main/main.c *** main(int argc, char *argv[]) *** 191,196 --- 191,198 AuxiliaryProcessMain(argc, argv); /* does not return */ else if (argc > 1 && strcmp(argv[1], "--describe-config") == 0) GucInfoMain(); /* does not return */ + else if (argc > 1 && strncmp(argv[1], "--child=", 8) == 0) + ChildPostgresMain(argc, argv, get_current_username(progname)); /* does not return */ else if (argc > 1 && strcmp(argv[1], "--single") == 0) PostgresMain(argc, argv, get_current_username(progname)); /* does not return */ else diff --git a/src/backend/postmaster/postmaster.c b/src/backend/postmaster/postmaster.c index 73520a6ca2f4d3300f3d8939c0e9412064a911c3..5a51fa9cb9abb39b0ac02d21a25e5940159582ea 100644 *** a/src/backend/postmaster/postmaster.c --- b/src/backend/postmaster/postmaster.c *** ExitPostmaster(int status) *** 4268,4273 --- 4268,4350 proc_exit(status); } + + /* + * ChildPostgresMain - start a new-style standalone postgres process + * + * This may not belong here, but it does share a lot of code with ConnCreate + * and BackendInitialize. Basically what it has to do is set up a + * MyProcPort structure and then hand off control to PostgresMain. + * Beware that not very much stuff is initialized yet. + * + * In the future it might be interesting to support a "standalone + * multiprocess" mode in which we have a postmaster process that doesn't + * listen for connections, but does supervise autovacuum, bgwriter, etc + * auxiliary processes. So that's another reason why postmaster.c might be + * the right place for this. + */ + void + ChildPostgresMain(int argc, char *argv[], const char *username) + { + Port *port; + + /* + * Fire up essential subsystems: error and memory management + */ + MemoryContextInit(); + + /* + * Build a Port structure for the client connection + */ + if (!(port = (Port *) calloc(1, sizeof(Port + ereport(FATAL, + (errcode(ERRCODE_OUT_OF_MEMORY), + errmsg("out of memory"))); + + /* + * GSSAPI specific state struct must exist even though we won't use it + */ + #if defined(ENABLE_GSS) || defined(ENABLE_SSPI) + port->gss = (pg_gssinfo *) calloc(1, sizeof(pg_gssinfo)); + if (!port->gss) + ereport(FATAL, + (errcode(ERRCODE_OUT_OF_MEMORY), + errmsg("out of memory"))); + #endif + + /* The file descriptor of the client socket is the argument of --child */ + if (sscanf(argv[1], "--child=%d", &port->sock) != 1) + ereport(FATAL, + (errcode(ERRCODE_INVALID_PARAMETER_VALUE), + errmsg("invalid argument for --child: \"%s\"", argv[1]))); + + /* Default assumption about protocol to use */ + FrontendProtocol = port->proto = PG_PROTOCOL_LATEST; + + /* save process start time */ + port->SessionStartTime = GetCurrentTimestamp(); + MyStartTime = timestamptz_to_time_t(port->SessionStartTime); + + /* set these to empty in case they are needed */ + port->remote_host = ""; + port->remote_port = ""; + + MyProcPort = port; + + /* + * We can now initialize libpq and then enable reporting of ereport errors + * to the client. + */ + pq_init(); /* initialize libpq to talk to client */ + whereToSendOutput = DestRemote; /* now safe to ereport to client */ + + /* And pass off control to PostgresMain */ + PostgresMain(argc, argv, username); + + abort(); /* not reached */ + } + + /* * sigusr1_handler - handle signal conditions from child processes */ diff --git a/src/backend/tcop/postgres.c b/src/backend/tcop/postgres.c index f1248a851bf90188da8d3a7e8b61ac99bf78ebbd..c6a7de63c77f79c85be2170d4e5254b790f3fd5e 100644 *** a/src/backend/tcop/postgres.c --- b/src/backend/tcop/postgres.c *** process_postgres_switches(int argc, char *** 3257,3264 { gucsource = PGC_S_ARGV; /* switches came from command line */ ! /* Ignore the initial --single argument, if present */ ! if (argc > 1 && strcmp(argv[1], "--single") == 0) { argv++; argc--; --- 3257,3266 { gucsource = PGC_S_ARGV; /* switches came from command line */ ! /* Ignore the initial --single or --child argument, if present */ ! if (argc > 1 && ! (strcmp(argv[1], "--single") == 0 || ! strncmp(argv[1], "--child=", 8) == 0)) { argv++; argc--; *** PostgresMain(int argc, char *argv[],
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
Heikki Linnakangas writes: > It's worth noting that now that libpq constructs the command line to > execute "postgres --child= -D ", we'll be stuck with that set > of arguments forever, because libpq needs to be able to talk to > different versions. Or at least we'd need to teach libpq to check the > version of binary and act accordingly, if we change that syntax. That's > probably OK, I don't feel any pressure to change those command line > arguments anyway. Yeah. The -D syntax seems safe enough from here. One thing that's on my to-do list for this patch is to add a -v switch to set the protocol version, so that we don't need to assume that libpq and backend have the same default protocol version. regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On 07.09.2012 10:49, Tom Lane wrote: Heikki Linnakangas writes: Would socketpair(2) be simpler? Attached is a revised version of the patch that uses socketpair(2). This is definitely a lot less invasive --- the backend side of the patch, in particular, is far shorter, and there are fewer portability hazards since we're not trying to replace sockets with pipes. I've not done anything yet about the potential security issues associated with untrusted libpq connection strings. I think this is still at the proof-of-concept stage; in particular, it's probably time to see if we can make it work on Windows before we worry more about that. I'm a bit tempted though to pull out and apply the portions of the patch that replace libpq's assorted ad-hoc closesocket() calls with a centralized pqDropConnection routine. I think that's probably a good idea independently of this feature. Sounds good. It's worth noting that now that libpq constructs the command line to execute "postgres --child= -D ", we'll be stuck with that set of arguments forever, because libpq needs to be able to talk to different versions. Or at least we'd need to teach libpq to check the version of binary and act accordingly, if we change that syntax. That's probably OK, I don't feel any pressure to change those command line arguments anyway. - Heikki -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
Heikki Linnakangas writes: > Would socketpair(2) be simpler? Attached is a revised version of the patch that uses socketpair(2). This is definitely a lot less invasive --- the backend side of the patch, in particular, is far shorter, and there are fewer portability hazards since we're not trying to replace sockets with pipes. I've not done anything yet about the potential security issues associated with untrusted libpq connection strings. I think this is still at the proof-of-concept stage; in particular, it's probably time to see if we can make it work on Windows before we worry more about that. I'm a bit tempted though to pull out and apply the portions of the patch that replace libpq's assorted ad-hoc closesocket() calls with a centralized pqDropConnection routine. I think that's probably a good idea independently of this feature. regards, tom lane diff --git a/src/backend/main/main.c b/src/backend/main/main.c index 33c5a0a4e645624515016397da0011423d993c70..968959b85ef53aa2ec0ef8c5c5a9bc544291bfe5 100644 *** a/src/backend/main/main.c --- b/src/backend/main/main.c *** main(int argc, char *argv[]) *** 191,196 --- 191,198 AuxiliaryProcessMain(argc, argv); /* does not return */ else if (argc > 1 && strcmp(argv[1], "--describe-config") == 0) GucInfoMain(); /* does not return */ + else if (argc > 1 && strncmp(argv[1], "--child=", 8) == 0) + ChildPostgresMain(argc, argv, get_current_username(progname)); /* does not return */ else if (argc > 1 && strcmp(argv[1], "--single") == 0) PostgresMain(argc, argv, get_current_username(progname)); /* does not return */ else diff --git a/src/backend/postmaster/postmaster.c b/src/backend/postmaster/postmaster.c index 73520a6ca2f4d3300f3d8939c0e9412064a911c3..5a51fa9cb9abb39b0ac02d21a25e5940159582ea 100644 *** a/src/backend/postmaster/postmaster.c --- b/src/backend/postmaster/postmaster.c *** ExitPostmaster(int status) *** 4268,4273 --- 4268,4350 proc_exit(status); } + + /* + * ChildPostgresMain - start a new-style standalone postgres process + * + * This may not belong here, but it does share a lot of code with ConnCreate + * and BackendInitialize. Basically what it has to do is set up a + * MyProcPort structure and then hand off control to PostgresMain. + * Beware that not very much stuff is initialized yet. + * + * In the future it might be interesting to support a "standalone + * multiprocess" mode in which we have a postmaster process that doesn't + * listen for connections, but does supervise autovacuum, bgwriter, etc + * auxiliary processes. So that's another reason why postmaster.c might be + * the right place for this. + */ + void + ChildPostgresMain(int argc, char *argv[], const char *username) + { + Port *port; + + /* + * Fire up essential subsystems: error and memory management + */ + MemoryContextInit(); + + /* + * Build a Port structure for the client connection + */ + if (!(port = (Port *) calloc(1, sizeof(Port + ereport(FATAL, + (errcode(ERRCODE_OUT_OF_MEMORY), + errmsg("out of memory"))); + + /* + * GSSAPI specific state struct must exist even though we won't use it + */ + #if defined(ENABLE_GSS) || defined(ENABLE_SSPI) + port->gss = (pg_gssinfo *) calloc(1, sizeof(pg_gssinfo)); + if (!port->gss) + ereport(FATAL, + (errcode(ERRCODE_OUT_OF_MEMORY), + errmsg("out of memory"))); + #endif + + /* The file descriptor of the client socket is the argument of --child */ + if (sscanf(argv[1], "--child=%d", &port->sock) != 1) + ereport(FATAL, + (errcode(ERRCODE_INVALID_PARAMETER_VALUE), + errmsg("invalid argument for --child: \"%s\"", argv[1]))); + + /* Default assumption about protocol to use */ + FrontendProtocol = port->proto = PG_PROTOCOL_LATEST; + + /* save process start time */ + port->SessionStartTime = GetCurrentTimestamp(); + MyStartTime = timestamptz_to_time_t(port->SessionStartTime); + + /* set these to empty in case they are needed */ + port->remote_host = ""; + port->remote_port = ""; + + MyProcPort = port; + + /* + * We can now initialize libpq and then enable reporting of ereport errors + * to the client. + */ + pq_init(); /* initialize libpq to talk to client */ + whereToSendOutput = DestRemote; /* now safe to ereport to client */ + + /* And pass off control to PostgresMain */ + PostgresMain(argc, argv, username); + + abort(); /* not reached */ + } + + /* * sigusr1_handler - handle signal conditions from child processes */ diff --git a/src/backend/tcop/postgres.c b/src/backend/tcop/postgres.c index f1248a851bf90188da8d3a7e8b61ac99bf78ebbd..c6a7de63c77f79c85be2170d4e5254b790f3fd5e 100644 *** a/src/backend/tcop/postgres.c --- b/src/backend/tcop/postgres.c *** process_postgres_switches(int argc, char *** 3257,3264 { gucsource = PGC_S_ARGV; /* switches came from command line */ ! /* Ignore the
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Wed, 2012-09-05 at 17:03 -0400, Tom Lane wrote: > In general I think the selling point for such a feature would be "no > administrative hassles", and I believe that has to go not only for the > end-user experience but also for the application-developer experience. > If you have to manage checkpointing and vacuuming in the application, > you're probably soon going to look for another database. Maybe there could be some hooks (e.g., right after completing a statement) that see whether a vacuum or checkpoint is required? VACUUM can't be run in a transaction block[1], so there are some details to work out, but it might be a workable approach. Regards, Jeff Davis [1]: It seems like the only reason for that is so a multi-table vacuum doesn't hold locks for longer than it needs to, but that's not much of a concern in this case. -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Wed, Sep 5, 2012 at 7:14 PM, Tom Lane wrote: > This seems to me to be going in exactly the wrong direction. What > I visualize this feature as responding to is demand for a *simple*, > minimal configuration, minimal administration, quasi-embedded database. > What you propose above is not that, but is if anything even more > complicated for an application to deal with than a regular persistent > server. More complication is *the wrong thing* for this use case. > > The people who would be interested in this are currently using something > like SQLite within a single application program. It hasn't got any of > the features you're suggesting either, and I don't think anybody wishes > it did. I am failing to understand how one could easily replicate the SQLite feature of (even fairly poorly) using multiple processes addressing one database, and supporting multiple executors per-database (which correspond to every open 'connection' in SQLite, as far as I can understand). My best thoughts are in the direction of EXEC_BACKEND and hooking up to a cluster post-facto, but I wasn't really looking for solutions so much as to raise this (to me) important use-case. I'm just thinking about all the enormously popular prefork based web servers out there like unicorn (Ruby), gunicorn (Python), and even without forking language-specific database abstractions like that seen in Go ("database/sql") that have decided to make pooling the default interaction. It is easiest to use these prefork embedded servers in both in development and production, so people (rather sensibly) do that -- better parity, and no fuss. I really would rather not see that regress by appropriating special mechanics for test/development scenarios with regards to managing database connections (e.g. exactly one of them), so how do we not make that a restriction, unless I misunderstood and was a non-restriction already? -- fdr -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On 09/05/2012 10:14 PM, Tom Lane wrote: Aidan Van Dyk writes: So, in the spirit of not painting ourselves into a tiny corner here on the whole "single backend" and "embedded database" problem with pg options, can we generalize this a bit? Any way we could make psql connect to a "given fd", as an option? In theory, that could be something opened by some out-side-of-postgresql tunnel with 3rd party auth in the same app that uses libpq directly, or it could be a fd prepared by something that specifically launched a single-backend postgres, like in the case of pg_upgrade, pg_uprade itself, and passed to psql, etc, which would be passed in as options. This seems to me to be going in exactly the wrong direction. What I visualize this feature as responding to is demand for a *simple*, minimal configuration, minimal administration, quasi-embedded database. What you propose above is not that, but is if anything even more complicated for an application to deal with than a regular persistent server. More complication is *the wrong thing* for this use case. The people who would be interested in this are currently using something like SQLite within a single application program. It hasn't got any of the features you're suggesting either, and I don't think anybody wishes it did. Exactly. I think it's worth stating that this has a HUGE potential audience, and if we can get to this the deployment of Postgres could mushroom enormously. I'm really quite excited about it. cheers andrew -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
Aidan Van Dyk writes: > So, in the spirit of not painting ourselves into a tiny corner here on > the whole "single backend" and "embedded database" problem with pg > options, can we generalize this a bit? > Any way we could make psql connect to a "given fd", as an option? In > theory, that could be something opened by some out-side-of-postgresql > tunnel with 3rd party auth in the same app that uses libpq directly, > or it could be a fd prepared by something that specifically launched > a single-backend postgres, like in the case of pg_upgrade, pg_uprade > itself, and passed to psql, etc, which would be passed in as options. This seems to me to be going in exactly the wrong direction. What I visualize this feature as responding to is demand for a *simple*, minimal configuration, minimal administration, quasi-embedded database. What you propose above is not that, but is if anything even more complicated for an application to deal with than a regular persistent server. More complication is *the wrong thing* for this use case. The people who would be interested in this are currently using something like SQLite within a single application program. It hasn't got any of the features you're suggesting either, and I don't think anybody wishes it did. regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
So, in the spirit of not painting ourselves into a tiny corner here on the whole "single backend" and "embedded database" problem with pg options, can we generalize this a bit? Any way we could make psql connect to a "given fd", as an option? In theory, that could be something opened by some out-side-of-postgresql tunnel with 3rd party auth in the same app that uses libpq directly, or it could be a fd prepared by something that specifically launched a single-backend postgres, like in the case of pg_upgrade, pg_uprade itself, and passed to psql, etc, which would be passed in as options. In theory, that might even allow the possibility of starting the single-backend only once and passing it to multiple clients in succession, instead of having to stop/start the backend between each client. And it would allow the possiblity of "something" (pg_upgrade, or some other application) to control the start/stop of the backend outside the libpq connection. Now, I'm familiar with the abilities related to passing fd's around in Linux, but have no idea if we'd have comparable methods to use on Windows. a. On Wed, Sep 5, 2012 at 8:11 PM, Daniel Farina wrote: > On Wed, Sep 5, 2012 at 3:17 PM, Peter Eisentraut wrote: >> On 9/5/12 5:59 PM, Daniel Farina wrote: >>> I agree with this, even though in theory (but not in practice) >>> creative use of unix sockets (sorry windows, perhaps some >>> port-allocating and URL mangling can be done instead) and conventions >>> for those would allow even better almost-like-embedded results, >>> methinks. That may still be able to happen. >> >> Sure, everyone who cares can already do this, but some people probably >> don't care enough. Also, making this portable and robust for everyone >> to use, not just your local environment, is pretty tricky. See >> pg_upgrade test script, for a prominent example. > > To my knowledge, no one has even really seriously tried to package it > yet and then told the tale of woe, and it was an especially > un-gratifying exercise for quite a while on account of multiple > postgreses not getting along on the same machine because of SysV > shmem. > > The bar for testing is a lot different than pg_upgrade (where a > negative consequence is confusing and stressful downtime), and many > programs use fork/threads and multiple connections even in testing, > making its requirements different. > > So consider me still skeptical given the current reasoning that unix > sockets can't be a good-or-better substitute, and especially > accounting for programs that need multiple backends. > > -- > fdr > > > -- > Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) > To make changes to your subscription: > http://www.postgresql.org/mailpref/pgsql-hackers > -- Aidan Van Dyk Create like a god, ai...@highrise.ca command like a king, http://www.highrise.ca/ work like a slave. -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Wed, Sep 5, 2012 at 3:17 PM, Peter Eisentraut wrote: > On 9/5/12 5:59 PM, Daniel Farina wrote: >> I agree with this, even though in theory (but not in practice) >> creative use of unix sockets (sorry windows, perhaps some >> port-allocating and URL mangling can be done instead) and conventions >> for those would allow even better almost-like-embedded results, >> methinks. That may still be able to happen. > > Sure, everyone who cares can already do this, but some people probably > don't care enough. Also, making this portable and robust for everyone > to use, not just your local environment, is pretty tricky. See > pg_upgrade test script, for a prominent example. To my knowledge, no one has even really seriously tried to package it yet and then told the tale of woe, and it was an especially un-gratifying exercise for quite a while on account of multiple postgreses not getting along on the same machine because of SysV shmem. The bar for testing is a lot different than pg_upgrade (where a negative consequence is confusing and stressful downtime), and many programs use fork/threads and multiple connections even in testing, making its requirements different. So consider me still skeptical given the current reasoning that unix sockets can't be a good-or-better substitute, and especially accounting for programs that need multiple backends. -- fdr -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
> Um ... true with respect to autovacuum, perhaps, but what about > checkpoints? A standalone backend will never perform a checkpoint > unless explicitly told to. Hmmm, that's definitely an issue. > (Before we invented the bgwriter, the > postmaster was in charge of launching checkpoints every so often.) > Again, this is probably just what you want for disaster recovery, but > it wouldn't be terribly friendly for an embedded-database application. Yeah, we'd have to put in a clock-based thing which did checkpoints every 5 minutes and VACUUM ANALYZE every hour or something. That seems like a chunk of extra code. > In general I think the selling point for such a feature would be "no > administrative hassles", and I believe that has to go not only for the > end-user experience but also for the application-developer experience. > If you have to manage checkpointing and vacuuming in the application, > you're probably soon going to look for another database. Well, don't discount the development/testing case. If you do agile or TDD (a lot of people do), you often have a workload which looks like: 1) Start framework 2) Start database 3) Load database with test data 4) Run tests 5) Print results 6) Shut down database In a case like that, you can live without checkpointing, even; the database is ephemeral. In other words, let's make this a feature and document it for use in testing, and that it's not really usable for production embedded apps yet. -- Josh Berkus PostgreSQL Experts Inc. http://pgexperts.com -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On 9/5/12 2:50 PM, Peter Eisentraut wrote: > On 9/5/12 5:03 PM, Tom Lane wrote: >> I don't see people wanting to use this feature for unit tests. > > If this is going to become an official feature (as opposed to an > internal interface only for use by pg_upgrade), then I think that's > exactly what people will want to use it for. In fact, it might even > make it more likely that people will write unit tests suits to begin with. Heck, *I'll* use it for unit tests. -- Josh Berkus PostgreSQL Experts Inc. http://pgexperts.com -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On 9/5/12 5:59 PM, Daniel Farina wrote: > I agree with this, even though in theory (but not in practice) > creative use of unix sockets (sorry windows, perhaps some > port-allocating and URL mangling can be done instead) and conventions > for those would allow even better almost-like-embedded results, > methinks. That may still be able to happen. Sure, everyone who cares can already do this, but some people probably don't care enough. Also, making this portable and robust for everyone to use, not just your local environment, is pretty tricky. See pg_upgrade test script, for a prominent example. -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On Wed, Sep 5, 2012 at 2:50 PM, Peter Eisentraut wrote: > On 9/5/12 5:03 PM, Tom Lane wrote: >> I don't see people wanting to use this feature for unit tests. > > If this is going to become an official feature (as opposed to an > internal interface only for use by pg_upgrade), then I think that's > exactly what people will want to use it for. In fact, it might even > make it more likely that people will write unit tests suits to begin with. I agree with this, even though in theory (but not in practice) creative use of unix sockets (sorry windows, perhaps some port-allocating and URL mangling can be done instead) and conventions for those would allow even better almost-like-embedded results, methinks. That may still be able to happen. The biggest improvement to that situation is the recent drastic reduction in use of shared memory, and that only became a thing recently. -- fdr -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Re: [HACKERS] Proof of concept: standalone backend with full FE/BE protocol
On 9/5/12 5:03 PM, Tom Lane wrote: > I don't see people wanting to use this feature for unit tests. If this is going to become an official feature (as opposed to an internal interface only for use by pg_upgrade), then I think that's exactly what people will want to use it for. In fact, it might even make it more likely that people will write unit tests suits to begin with. -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers