[HACKERS] Bug in Dependencies Code in 7.3.x?
I've found a situation that doesn't look correct to me in 7.3.1 (and
presumably 7.3 as well). If I alter a column so that it no longer uses a
sequence for default values and then try to drop the aforementioned
sequence, the dependency checking code does not allow me to drop the
sequence on the basis that it is still deing used. As I have removed the
default setting, however, this should not be the case, as far as I can
tell. The following illustrates my finding.
junk= create table testing
junk- (testing_id serial not null primary key);
NOTICE: CREATE TABLE will create implicit sequence
'testing_testing_id_seq' for SERIAL column 'testing.testing_id'
NOTICE: CREATE TABLE / PRIMARY KEY will create implicit index
'testing_pkey' for table 'testing'
CREATE TABLE
junk= \d
List of relations
Schema | Name | Type | Owner
++--+
public | testing| table| tara
public | testing_testing_id_seq | sequence | tara
(2 rows)
junk= \d testing
Table public.testing
Column | Type |Modifiers
+-+-
testing_id | integer | not null default
nextval('public.testing_testing_id_seq'::text)
Indexes: testing_pkey primary key btree (testing_id)
junk= alter table testing
junk- alter column testing_id drop default;
ALTER TABLE
junk= \d testing
Table public.testing
Column | Type | Modifiers
+-+---
testing_id | integer | not null
Indexes: testing_pkey primary key btree (testing_id)
junk= drop sequence testing_testing_id_seq;
ERROR: Cannot drop sequence testing_testing_id_seq because table
testing column testing_id requires it
You may drop table testing column testing_id instead
--
Tara Piorkowski
System Administrator, vilaj.com, LLC
http://www.vilaj.com/
---(end of broadcast)---
TIP 2: you can get off all lists at once with the unregister command
(send unregister YourEmailAddressHere to [EMAIL PROTECTED])
Re: [HACKERS] Bug in Dependencies Code in 7.3.x?
Tom Lane wrote: Tara Piorkowski [EMAIL PROTECTED] writes: junk= create table testing junk- (testing_id serial not null primary key); NOTICE: CREATE TABLE will create implicit sequence 'testing_testing_id_seq' for SERIAL column 'testing.testing_id' NOTICE: CREATE TABLE / PRIMARY KEY will create implicit index 'testing_pkey' for table 'testing' CREATE TABLE junk= alter table testing junk- alter column testing_id drop default; ALTER TABLE I'd be inclined to say that the bug here is that you shouldn't be allowed to do ALTER COLUMN DROP DEFAULT (nor SET DEFAULT for that matter) on a SERIAL column. The default expression is part of the implementation of SERIAL, not an independently tweakable entity. Tom - I am actually inclined to agree with you. This is not an issue I see coming up for us again, so making it work as you described above seems to me does not impact us negatively at all. This was a fluke for us as our application code was dependent on the manner in which the way pre-7.3 versions of PostgreSQL truncated sequence names. In running tests on the software we wrote to migrate our 7.2 databases to become 7.3 databases we just happened to stumble across this issue by accident (literally). Regardless, my thinking had been that I was looking at an INT with a DEFAULT set, in which case I think this would be a bonified bug, thus my report. However, it makes more sense to consider the column to be of type SERIAL, despite the labelling. Perhaps the best result would be to not allow the DROP DEFAULT on a SERIAL column and to label the SERIAL column as such (as opposed to an INT with DEFAULT). Thanks, as always, for taking the time to consider my posting and respond. - Tara -- Tara Piorkowski System Administrator, vilaj.com, LLC http://www.vilaj.com/ ---(end of broadcast)--- TIP 1: subscribe and unsubscribe commands go to [EMAIL PROTECTED]
Re: [HACKERS] [GENERAL] PostgreSQL v7.3b5 Packaged for Testing ...
Marc G. Fournier wrote: At this point, we are looking for confirmation that all the platforms are building and running the regression tests correctly, and, unless any major bug reports are submitted, any further changes to this branch will be either documentation related, or tweaks to ensure as many platforms work as possible before release. All is well with 7.3b5 on Mac OS X 10.2.1 (Jaguar) on a PowerBook G4 [gcc (GCC) 3.1 20020420 (prerelease)]. == All 89 tests passed. == -- Tara Piorkowski System Administrator, vilaj.com, LLC http://www.vilaj.com/ ---(end of broadcast)--- TIP 2: you can get off all lists at once with the unregister command (send unregister YourEmailAddressHere to [EMAIL PROTECTED])
Re: [HACKERS] Request for supported platforms
Bruce, Not sure. I know Tom Lane ran a check on 10.1.x and didn't have any errors, so I figured the differences I saw reflected changes made by Apple between 10.1.x and 10.2.x. Then, I was surprised when I saw Adam's report, so I thought perhaps he was running on a PPC G3 and there were differences between G3 and G4 floating point handling (I had run the original regression tests on a PowerBook G4/400 MHz). So, I ran the same exact scenario on my aging iMac (G3/266 Mhz). Those results corroborated my original test (geometry failed; horology cleared up, though, as expected). So, something is different between Adam's system and the two I have easy access to. I have confirmed that both my systems are running 10.2.1. Similarly, the tests were both run on 7.3b3 from the full tarball on ftp.us.postgresql.org. The new iMac G3 regression.diffs output is immediately below. Regardless, it would appear that Mac OS X 10.2.x is well supported. Tara *** ./expected/geometry-powerpc-darwin.out Mon Dec 11 11:45:16 2000 --- ./results/geometry.out Tue Oct 29 07:59:59 2002 *** *** 127,133 | (-5,-12) | [(10,-10),(-3,-4)]| (-1.60487804878049,-4.64390243902439) | (10,10)| [(10,-10),(-3,-4)]| (2.39024390243902,-6.48780487804878) | (0,0) | [(-100,200),(30,-40)] | (0.0028402365895872,15.384614860264) ! | (-10,0)| [(-100,200),(30,-40)] | (-9.99715942258202,15.3864610140472) | (-3,4) | [(-100,200),(30,-40)] | (-2.99789812267519,15.3851688427303) | (5.1,34.5) | [(-100,200),(30,-40)] | (5.09647083221496,15.3836744976925) | (-5,-12) | [(-100,200),(30,-40)] | (-4.99494420845634,15.3855375281616) --- 127,133 | (-5,-12) | [(10,-10),(-3,-4)]| (-1.60487804878049,-4.64390243902439) | (10,10)| [(10,-10),(-3,-4)]| (2.39024390243902,-6.48780487804878) | (0,0) | [(-100,200),(30,-40)] | (0.0028402365895872,15.384614860264) ! | (-10,0)| [(-100,200),(30,-40)] | (-9.99715942258202,15.3864610140473) | (-3,4) | [(-100,200),(30,-40)] | (-2.99789812267519,15.3851688427303) | (5.1,34.5) | [(-100,200),(30,-40)] | (5.09647083221496,15.3836744976925) | (-5,-12) | [(-100,200),(30,-40)] | (-4.99494420845634,15.3855375281616) == Bruce Momjian wrote: Strange. I just got report from another OSX 10.2.1 user saying regression tests passed: 10.2.1, Adam Witney ([EMAIL PROTECTED] The proper value seems to be: 15.3864610140472 or 15.3864610140473 in ./expected/geometry-powerpc-darwin.out. Which is it, folks? --- Tara Piorkowski wrote: Tom Lane wrote: Bruce Momjian writes: Folks. start sending in those plaform reports, OS name and version number please. I've checked CVS tip on: HPUX 10.20, using both gcc and vendor's cc PPC Linux Mac OS X 10.1 Here's the regression.diffs file from Mac OS X 10.2.1 (Jaguar). It failed geometry (rounding) and horology (not sure). gcc -v returns Apple Computer, Inc. GCC version 1161, based on gcc version 3.1 20020420 (prerelease). *** ./expected/geometry-powerpc-darwin.out Mon Dec 11 11:45:16 2000 --- ./results/geometry.out Sun Oct 27 21:27:11 2002 *** *** 127,133 | (-5,-12) | [(10,-10),(-3,-4)]| (-1.60487804878049,-4.64390243902439) | (10,10)| [(10,-10),(-3,-4)]| (2.39024390243902,-6.48780487804878) | (0,0) | [(-100,200),(30,-40)] | (0.0028402365895872,15.384614860264) ! | (-10,0)| [(-100,200),(30,-40)] | (-9.99715942258202,15.3864610140472) | (-3,4) | [(-100,200),(30,-40)] | (-2.99789812267519,15.3851688427303) | (5.1,34.5) | [(-100,200),(30,-40)] | (5.09647083221496,15.3836744976925) | (-5,-12) | [(-100,200),(30,-40)] | (-4.99494420845634,15.3855375281616) --- 127,133 | (-5,-12) | [(10,-10),(-3,-4)]| (-1.60487804878049,-4.64390243902439) | (10,10)| [(10,-10),(-3,-4)]| (2.39024390243902,-6.48780487804878) | (0,0) | [(-100,200),(30,-40)] | (0.0028402365895872,15.384614860264) ! | (-10,0)| [(-100,200),(30,-40)] | (-9.99715942258202,15.3864610140473) | (-3,4) | [(-100,200),(30,-40)] | (-2.99789812267519,15.3851688427303) | (5.1,34.5) | [(-100,200),(30,-40)] | (5.09647083221496,15.3836744976925) | (-5,-12) | [(-100,200),(30,-40)] | (-4.99494420845634,15.3855375281616) == *** ./expected
Re: [HACKERS] Request for supported platforms
Tom Lane wrote: Bruce Momjian writes: Folks. start sending in those plaform reports, OS name and version number please. I've checked CVS tip on: HPUX 10.20, using both gcc and vendor's cc PPC Linux Mac OS X 10.1 Here's the regression.diffs file from Mac OS X 10.2.1 (Jaguar). It failed geometry (rounding) and horology (not sure). gcc -v returns Apple Computer, Inc. GCC version 1161, based on gcc version 3.1 20020420 (prerelease). *** ./expected/geometry-powerpc-darwin.out Mon Dec 11 11:45:16 2000 --- ./results/geometry.out Sun Oct 27 21:27:11 2002 *** *** 127,133 | (-5,-12) | [(10,-10),(-3,-4)]| (-1.60487804878049,-4.64390243902439) | (10,10)| [(10,-10),(-3,-4)]| (2.39024390243902,-6.48780487804878) | (0,0) | [(-100,200),(30,-40)] | (0.0028402365895872,15.384614860264) ! | (-10,0)| [(-100,200),(30,-40)] | (-9.99715942258202,15.3864610140472) | (-3,4) | [(-100,200),(30,-40)] | (-2.99789812267519,15.3851688427303) | (5.1,34.5) | [(-100,200),(30,-40)] | (5.09647083221496,15.3836744976925) | (-5,-12) | [(-100,200),(30,-40)] | (-4.99494420845634,15.3855375281616) --- 127,133 | (-5,-12) | [(10,-10),(-3,-4)]| (-1.60487804878049,-4.64390243902439) | (10,10)| [(10,-10),(-3,-4)]| (2.39024390243902,-6.48780487804878) | (0,0) | [(-100,200),(30,-40)] | (0.0028402365895872,15.384614860264) ! | (-10,0)| [(-100,200),(30,-40)] | (-9.99715942258202,15.3864610140473) | (-3,4) | [(-100,200),(30,-40)] | (-2.99789812267519,15.3851688427303) | (5.1,34.5) | [(-100,200),(30,-40)] | (5.09647083221496,15.3836744976925) | (-5,-12) | [(-100,200),(30,-40)] | (-4.99494420845634,15.3855375281616) == *** ./expected/horology.out Wed Sep 18 17:35:25 2002 --- ./results/horology.out Sun Oct 27 21:27:12 2002 *** *** 537,549 SELECT (timestamp with time zone 'today' = (timestamp with time zone 'tomorrow' - interval '1 day')) as True; True -- ! t (1 row) SELECT (timestamp with time zone 'tomorrow' = (timestamp with time zone 'yesterday' + interval '2 days')) as True; True -- ! t (1 row) SELECT (timestamp with time zone 'tomorrow' 'now') as True; --- 537,549 SELECT (timestamp with time zone 'today' = (timestamp with time zone 'tomorrow' - interval '1 day')) as True; True -- ! f (1 row) SELECT (timestamp with time zone 'tomorrow' = (timestamp with time zone 'yesterday' + interval '2 days')) as True; True -- ! f (1 row) SELECT (timestamp with time zone 'tomorrow' 'now') as True; == -- Tara Piorkowski System Administrator, vilaj.com, LLC http://www.vilaj.com/ ---(end of broadcast)--- TIP 3: 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
