Re: [PATCHES] Bug in WAL backup documentation

2006-11-07 Thread Bruno Wolff III
On Sun, Nov 05, 2006 at 11:49:36 -0500,
  Tom Lane [EMAIL PROTECTED] wrote:
 
 As already discussed upthread, anyone who wants the path can get it from
 `pwd` or local equivalent --- and that mechanism is robust (as long as
 the directory move doesn't happen while any particular instance of the
 script is running).  I don't see why we should go out of our way to
 provide a bad substitute for pwd.

I think you also still need read access to the intervening directories.
If the command works by walking up and matching inode numbers with names,
then it will break if it can't read the names. (For example /bin/pwd
breaks when it can't read a parent directories filenames.)

---(end of broadcast)---
TIP 7: You can help support the PostgreSQL project by donating at

http://www.postgresql.org/about/donate


Re: [PATCHES] [HACKERS] Performance testing of COPY (SELECT) TO

2006-08-28 Thread Bruno Wolff III
On Mon, Aug 28, 2006 at 19:35:11 +0200,
  Zoltan Boszormenyi [EMAIL PROTECTED] wrote:
 
 (BTW, is there anyone as high-ranking as them,
 or the committee is a duumvirate? :-) )

There is a group referred to as core that is the final arbitrator of things.
Tom and Bruce are both members of this group.
Tom and Bruce tend to be the most visibly active committers for getting
patches committed for people that can't do it themselves. So you will see them
speak up more than others on the patches list.

---(end of broadcast)---
TIP 6: explain analyze is your friend


Re: [PATCHES] [HACKERS] Inconsistent syntax in GRANT

2006-01-06 Thread Bruno Wolff III
On Fri, Jan 06, 2006 at 19:11:27 +0200,
  Marko Kreen [EMAIL PROTECTED] wrote:
 On 1/6/06, Bruce Momjian pgman@candle.pha.pa.us wrote:
 
 Considering there's no currval() without nextval(), what point
 is disallowing currval() when user is able to call nextval()?
 
 I rather want to allow nextval/currval and disable setval as it
 allows regular user to DoS the database.

What I was thinking with this, is that you might allow someone the ability
to insert records into a table which would make use of nextval, but not
allow them to run nextval directly. But after inserting a record allow them
to use currval to see what value was assigned.
People could still mess with things by doing INSERTs and aborting the
transaction, so this may not be the best example for why you would want this.

---(end of broadcast)---
TIP 4: Have you searched our list archives?

   http://archives.postgresql.org


Re: [PATCHES] [HACKERS] Inconsistent syntax in GRANT

2006-01-05 Thread Bruno Wolff III
On Thu, Jan 05, 2006 at 11:44:24 -0800,
  Josh Berkus josh@agliodbs.com wrote:
 Bruce, Tom,
 
   The permissions for a sequence aren't the same as they are for a
   table. We've sort of ignored the point to date, but if we're going to
   add special syntax for granting on a sequence, I don't think we should
   continue to ignore it.
 
  Uh, how are they different?   You mean just UPDATE and none of the
  others do anything?
 
 Yes, it would be nice to have real permissions for sequences, specifically 
 USE (which allows nextval() and currval()) and UPDATE (which would allow 
 setval() ).   However, I don't know that the added functionality would 
 justify breaking backwards-compatibility.

It might be nice to split nextval and currval access as well. nextval access
corresponds to INSERT and currval access to SELECT.

---(end of broadcast)---
TIP 1: if posting/reading through Usenet, please send an appropriate
   subscribe-nomail command to [EMAIL PROTECTED] so that your
   message can get through to the mailing list cleanly


Re: [PATCHES] CRC32 function

2005-08-25 Thread Bruno Wolff III
On Thu, Aug 25, 2005 at 16:14:04 -0400,
  Bruce Momjian pgman@candle.pha.pa.us wrote:
 Ilia Kantor wrote:
  Both backend and users may have a nice use of the function.
  
   
  
  Nice and fast hashing when one doesn't need encryption.
 
 We already have MD5 encryption in the server.  Why would someone want
 CRC32?

Lower CPU utiliization.

---(end of broadcast)---
TIP 2: Don't 'kill -9' the postmaster


Re: [PATCHES] CRC32 function

2005-08-25 Thread Bruno Wolff III
On Thu, Aug 25, 2005 at 18:44:13 -0400,
  Tom Lane [EMAIL PROTECTED] wrote:
 Bruno Wolff III [EMAIL PROTECTED] writes:
Bruce Momjian pgman@candle.pha.pa.us wrote:
  We already have MD5 encryption in the server.  Why would someone want
  CRC32?
 
  Lower CPU utiliization.
 
 Like Bruce, I don't really think there is demand for such a function.
 But if we were going to offer it, it at least ought to use the existing
 implementation in pg_crc.c, instead of duplicating code yet again.

Maybe I should have elaborated. I was just responding directly to Bruce's
question. I doubt the CPU usage is a big deal in typical use and that
that the already available cryptographic hashes have advantages such that
I don't expect many people to use CRC32.

---(end of broadcast)---
TIP 2: Don't 'kill -9' the postmaster


Re: [PATCHES] [patch 0/2] Add Fortuna PRNG to pgcrypto

2005-07-08 Thread Bruno Wolff III
On Fri, Jul 08, 2005 at 20:54:40 +0300,
  Marko Kreen marko@l-t.ee wrote:
 
 The idea is to initially seed Fortuna with randomness from
 system and later feed SHA1 of user data into it too.  Just
 to keep it from degenerating into pure PRNG.

How is fortuna getting entropy?
Wouldn't this be better placed in the OS as the source of /dev/urandom rather
than as part of Postgres?

---(end of broadcast)---
TIP 5: don't forget to increase your free space map settings


Re: [PATCHES] [patch 0/2] Add Fortuna PRNG to pgcrypto

2005-07-08 Thread Bruno Wolff III
On Fri, Jul 08, 2005 at 23:03:49 +0300,
  Marko Kreen marko@l-t.ee wrote:
 
 Well, those OS'es that already have urandom/random, don't need
 it.  And those that don't - I really don't feel responsibility
 to write one...

But fortuna is essentially a high quality /dev/urandom. It doesn't make seem to
much sense to implement it on top on /dev/urandom.

---(end of broadcast)---
TIP 4: Have you searched our list archives?

   http://archives.postgresql.org


Re: [PATCHES] thousands comma numeric formatting in psql

2005-06-21 Thread Bruno Wolff III
On Tue, Jun 21, 2005 at 08:42:16 +0300,
  Eugen Nedelcu [EMAIL PROTECTED] wrote:
 
 One solution to deal with this is to use to_char function, but for
 complex selects against multiple tables it's not a good option.

Why not? You only have to apply it to the output expressions that
need it.

Note that if you output numbers like this, you also need to be able to
read them back in. I don't think adding complexity for doing that is
worth not having to add a few to_char calls in your select queries.

---(end of broadcast)---
TIP 6: Have you searched our list archives?

   http://archives.postgresql.org


Re: [PATCHES] thousands comma numeric formatting in psql

2005-06-21 Thread Bruno Wolff III
On Tue, Jun 21, 2005 at 16:03:43 +0300,
  Eugen Nedelcu [EMAIL PROTECTED] wrote:
 
 This is a patch for psql client and not for the backend. It's role
 is to output numbers to screen in easy readable form (2,345,675,454,543
 is much easier to read then 2345675454543.456). I think graphical 
 clients like pgAdmin or phppgadmin have a way to do this. I don't know 
 this for sure but I will investigate it.

I did miss that this was a psql only feature. I still don't see a great
need, but it isn't going to cause a problem for copy or pg_dump that way.

---(end of broadcast)---
TIP 5: Have you checked our extensive FAQ?

   http://www.postgresql.org/docs/faq


Re: [PATCHES] default database creation with initdb

2005-06-18 Thread Bruno Wolff III
On Sat, Jun 18, 2005 at 09:27:49 -0400,
  Robert Treat [EMAIL PROTECTED] wrote:
 On Saturday 18 June 2005 04:55, Andreas Pflug wrote:
  Magnus Hagander wrote:
   Umm. Tiny item, but your comment still refers to the database as
   pg_system ;-)
  
 
 What is the purpose of this database? A generalized, shared resource for tool 
 makers and add-on packages to store information in PostgreSQL, or a working 
 database that is usable (and to be used) out of the box for new users?  I 
 really don't think we want the latter... I can see users connecting via psql 
 and then playing around with different add/create type statements.  It is all 
 too common a question from newbies... does postgresql have a default 
 database to get started with? They'll see this database and begin creating 
 schema and using this as thier main database, and I think we ought to avoid 
 that. If people don't like pg_system, pg_addons seem like a much safer name 
 to go with imho. 

I believe the intention is that things that need to connect to some
database to do their work (e.g. psql -l, createuser) will connect to that
database. createdb will still connect to template1, but now will be less
likely to have a conflict with another user being connected to template1
at the same time. I didn't check the patch to see if the behavior of the
psql -l and createuser were changed or if just the initdb behavior was
changed.

I don't think it will be a big deal if people put stuff in that database.

---(end of broadcast)---
TIP 5: Have you checked our extensive FAQ?

   http://www.postgresql.org/docs/faq


Re: [PATCHES] O_DIRECT for WAL writes

2005-06-03 Thread Bruno Wolff III
On Fri, Jun 03, 2005 at 09:43:13 -0700,
  Mary Edie Meredith [EMAIL PROTECTED] wrote:
 
 Looking at this from another angle, is there really any way that you can
 say a write is truly guaranteed in the event of a failure?  I think in
 the end to be safe, you cannot.  That's why (and I'm not telling you
 anything new) there is no substitute for backups and log archiving for
 databases. Databases must be able to recognize the last _good
 transaction logged and roll forward to that from the backup (including
 detecting partial writes to the log).  I'm sure the PostgreSQL community
 has worked hard to do the equivalent of that within the PostgreSQL
 architecture. 

Some assumptions are made about what order blocks are written to the disk.
If these assumptions are not true, you may not be able to recover using
the WAL log and have to resort to falling back to your last consistant
snapshot.

---(end of broadcast)---
TIP 6: Have you searched our list archives?

   http://archives.postgresql.org


Re: [PATCHES] Updated COPY CSV patch

2004-04-13 Thread Bruno Wolff III
On Tue, Apr 13, 2004 at 06:58:24 -0400,
  Andrew Dunstan [EMAIL PROTECTED] wrote:
 
 One area that we should think about as an enhancement is NOT NULL fields.
 As it stands now, we will get what we normally get when we try to insert
 a  null into a NOT NULL field, namely an error. If the field has a simple
 literal default we could force that. And in the special case of
 text/varchar fields, it would be reasonable to force an empty string even
 if no default is set. There isn't a nice easy answer, I'm afraid. We
 shouldn't hold up putting this in on that account, but handling this
 better is certainly a TODO.

If you try to insert NULLs into a nonnull field you should get an error.
If you have unquoted empty strings, and are not using the empty string as 
the NULL marker, then you can just not set the NULL code to be the empty
string. If you need to turn this on and off by column, then perhaps it
would be better to do that externally.

As for setting default values, I think that is a good idea. I suggested
a while back. There could be another keyword, DEFAULT, on the COPY FROM
command that is used to define a code that will be replaced by the
default value (or NULL if there is no default for a column) similar to
how the NULL code is replaced by NULL.

---(end of broadcast)---
TIP 4: Don't 'kill -9' the postmaster


Re: [PATCHES] COPY for CSV documentation

2004-04-12 Thread Bruno Wolff III
On Mon, Apr 12, 2004 at 02:26:14 -0400,
  Andrew Dunstan [EMAIL PROTECTED] wrote:
 
 a few points:
 
 . in CSV mode, NULL should default to '' - that was in what I sent in.

Postgres normally treats an empty string as an empty string. Are you sure
you really want it to be treated as a NULL by default in this one place?

---(end of broadcast)---
TIP 2: you can get off all lists at once with the unregister command
(send unregister YourEmailAddressHere to [EMAIL PROTECTED])


Re: [PATCHES] COPY for CSV documentation

2004-04-12 Thread Bruno Wolff III
On Mon, Apr 12, 2004 at 10:30:22 -0400,
  Bruce Momjian [EMAIL PROTECTED] wrote:
 
 It is my understanding that \N is a valid column value (no backslash
 escape in CSV, right?), so we can't use it for NULL.
 The only thing I can think of is for NULL to be:
 
   ,,
 
 (no quotes) and a zero-length string to be:
 
   ,,
 
 How do most applications handle those two cases?  If they accept either,
 can we use that so we can read our own CSV files without losing the NULL
 specification?

I think the above are going to be treated as equvialent by most CSV parsers.

There doesn't seem to be a standard for CSV. From what I found describing
it, there isn't any feature for distinguishing NULLs from empty strings.
So whatever gets done is going to be application specific.

---(end of broadcast)---
TIP 8: explain analyze is your friend