The thing I was trying to do was use the GUC hook function to make
sure that the required GUC variables are also set before GUC reports
autovac as enabled. This seemed cleaner to me, but apparently it
won't work since it seems that autovac_enabled is read from GUC
before the stats variables, and t
Christopher Kings-Lynne wrote:
> >>The thing I was trying to do was use the GUC hook function to make
> >>sure that the required GUC variables are also set before GUC reports
> >>autovac as enabled. This seemed cleaner to me, but apparently it
> >>won't work since it seems that autovac_enabled is
Matthew T. O'Connor wrote:
> On Fri, 2004-07-16 at 17:13, Bruce Momjian wrote:
> > Peter Eisentraut wrote:
> > > A nitpick on that parameter name: There is not reason to abbreviate
> > > "autovacuum": there is enough space. And the "_enabled" is redundant.
> >
> > Agreed. Nitpicks are importan
Can we please have it default to enabled :)
We can but without also enabling statistics it will not work. Do we
want to enable both by default?
Weeell...it just seemed to me that we won't cut down on the support
mails unless it's on by default... I mean at some point in the future,
we WILL hav
Christopher Kings-Lynne wrote:
> >>Can we please have it default to enabled :)
> >
> >
> > We can but without also enabling statistics it will not work. Do we
> > want to enable both by default?
>
> Weeell...it just seemed to me that we won't cut down on the support
> mails unless it's on by d
"Matthew T. O'Connor" <[EMAIL PROTECTED]> writes:
> What I wanted to do was have GUC prevent autovacuum = true
> when the stats collector is not enabled, reguardless of what is in
> postgresql.conf.
GUC can't do that, but I think you can just hack the boolean variable
during startup --- compare wh
> Exactly. However, I can work around the GUC ordering issue just by
> having the postmaster check all the required variables before launching
> autovac. What I wanted to do was have GUC prevent autovacuum = true
> when the stats collector is not enabled, reguardless of what is in
> postgresql.co
This is the error that openjade produced:
openjade:ref/create_database.sgml:177:7:E: document type does not allow
element "PARA" here; missing one of "FOOTNOTE", "ITEMIZEDLIST",
"ORDEREDLIST", "VARIABLELIST", "CAUTION", "IMPORTANT", "NOTE", "TIP",
"WARNING", "BLOCKQUOTE", "INFORMALEXAMPLE" star
Bruce Momjian <[EMAIL PROTECTED]> writes:
> Yes, I would think it would become default enabled someday. Should we
> wait one release or do it for 7.5?
I think it's a bit premature to have it on by default, seeing that it's
still far from being in final form. In my mind it's still a pretty
experi
"Jonathan M. Gardner" <[EMAIL PROTECTED]> writes:
> openjade:ref/create_database.sgml:194:11:E: end tag for "PARA" omitted,
> but OMITTAG NO was specified
> openjade:ref/create_database.sgml:175:2: start tag was here
Yeah, I see it too on my RHL 8 machine. The DTD being used at
postgresql.org se
[ ... some desultory reading of PITR patch ... ]
What is the point of having both archive_program and archive_dest as
GUC variables? Wouldn't it be simpler to fold them into one parameter,
viz
archive_command = 'cp %s /archivedir'
For that matter, do we need a separate archive_mode bool
Tom Lane wrote:
> [ ... some desultory reading of PITR patch ... ]
>
> What is the point of having both archive_program and archive_dest as
> GUC variables? Wouldn't it be simpler to fold them into one parameter,
> viz
>
> archive_command = 'cp %s /archivedir'
>
> For that matter, do we n
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
Attached is a diff for subtransactions. I modified the SQL reference
section and touched on the various commands therein. I didn't mention
anything about cursors or portals, because I bet they will be resolved
before long.
I looked at the tutorial
Bruce Momjian <[EMAIL PROTECTED]> writes:
> Tom Lane wrote:
>> What is the point of having both archive_program and archive_dest as
>> GUC variables?
> I assume archive_dest is used for both archive and recovery of archives.
You assume wrong; it's not used there. There isn't any real good
reason
Jonathan M. Gardner wrote:
> Attached is a diff for subtransactions. I modified the SQL reference
> section and touched on the various commands therein. I didn't mention
> anything about cursors or portals, because I bet they will be
> resolved before long.
It looks like you better wait until the
"Jonathan M. Gardner" <[EMAIL PROTECTED]> writes:
> Attached is a diff for subtransactions.
Uh, I hate to tell you this, but you wasted your time because the syntax
accepted by CVS tip is not going to be the final syntax. None of what
you wrote here will be useful as soon as Alvaro's pending patc
The attached patch and additional src/backend/postmaster/syslogger.c
implements the logfile subprocess as discussed.
TODO:
- documentation
- win32 code (forkexec) is included, but not tested (no build env)
Functions (all are superuser only):
int4 pg_reload_conf()
Sends SIGHUP to postmaster
bool
Tom Lane wrote:
> Bruce Momjian <[EMAIL PROTECTED]> writes:
> > Tom Lane wrote:
> >> What is the point of having both archive_program and archive_dest as
> >> GUC variables?
>
> > I assume archive_dest is used for both archive and recovery of archives.
>
> You assume wrong; it's not used there.
Bruce Momjian wrote:
OK, the attached applied patch opens text files in binary mode in psql.
As you said, it already handles CRLF OK, and we need this to read
control-Z in copy files.
I assume we can write control-Z in text mode just fine, right?
I honestly don't know. Unless I see a knowledg
Ok, here is a new patch for to integrate pg_autovacuum into the backend.
This patch adds the following to that last patch:
* updated to latest CVS
* changed GUC vars from autovac_* to autovacuum_*
* changed autovac_enabled GUC variable to autovacuum
* changed autovacuum GUC variable default to fal
Bruce Momjian <[EMAIL PROTECTED]> writes:
> So you want to merge them all into a single command string. That does
> seem less error-prone. I see a few variables that turn off
> when set to '' like unix_socket_*. How would this command string work?
> How do you specify the WAL file name to trans
21 matches
Mail list logo
