Thread: Allow pg_dump to dump sequences using NO_MAXVALUE and NO_MINVALUEb

Allow pg_dump to dump sequences using NO_MAXVALUE and NO_MINVALUEb

From
Rod Taylor
Date:
- Move SEQ_MAXVALUE, SEQ_MINVALUE definitions to sequence.h

- Add check in pg_dump to see if the value returned is the max /min
values and replace with NO MAXVALUE, NO MINVALUE.

- Change START and INCREMENT to use START WITH and INCREMENT BY syntax.
This makes it a touch easier to port to other databases with sequences
(Oracle).  PostgreSQL supports both syntaxes already.

--
Rod Taylor <rbt@rbt.ca>

PGP Key: http://www.rbt.ca/rbtpub.asc

Attachment

Re: Allow pg_dump to dump sequences using NO_MAXVALUE and

From
Gavin Sherry
Date:
On 23 Feb 2003, Rod Taylor wrote:

> - Move SEQ_MAXVALUE, SEQ_MINVALUE definitions to sequence.h
> 
> - Add check in pg_dump to see if the value returned is the max /min
> values and replace with NO MAXVALUE, NO MINVALUE.
> 
> - Change START and INCREMENT to use START WITH and INCREMENT BY syntax. 
> This makes it a touch easier to port to other databases with sequences
> (Oracle).  PostgreSQL supports both syntaxes already.

+       char            bufm[100],
+                               bufx[100];

This seems to be an arbitary size. Why not set it to the actual maximum
length?

Also:

+       snprintf(bufm, 100, INT64_FORMAT, SEQ_MINVALUE);
+       snprintf(bufx, 100, INT64_FORMAT, SEQ_MAXVALUE);

sizeof(bufm), sizeof(bufx) is probably the more
maintenance-friendly/standard way to do it.

Thanks,

Gavin