Opened 13 years ago
Closed 13 years ago
#1135 closed enhancement (fixed)
Small regression test updates
Reported by: | ads | Owned by: | strk |
---|---|---|---|
Priority: | medium | Milestone: | PostGIS 2.0.0 |
Component: | build | Version: | 1.5.X |
Keywords: | Cc: | ads@…, andreas.scherbaum@…, ashwin.kumar3@… |
Description
As posted earlier on the devel mailinglist:
http://postgis.refractions.net/pipermail/postgis-devel/2010-October/014441.html (the date is invalid because of server issues)
here are two patches for the regression tests. One patch is for 1.5.3, one for 2.0.0SVN.
From my mail to the mailinglist:
we recently implemented PostGIS in Greenplum (a PostgreSQL fork). The regression tests throwed some errors on us, but mainly only a different output order in the affected .expected files.
Greenplum is a distributed shared-nothing architecture. Unlike PostgreSQL, we will always get the rows in a different order - unless we order them explicitely, like it is required in the SQL standard.
Thinking about it, there is a (although very small) chance, that synchronized sequentiell scans in 8.3 and above will also break these regression tests - as example, if a backup is running at the same time.
Attached you can find a small patch which adds "order by 1" in some of the regression tests, and also changes the output order in some of the .expected files. The data itself is not changed. The patch works with PostgreSQL 9.04 and PostGIS 1.5.3.
Attachments (2)
Change History (10)
by , 13 years ago
Attachment: | regress-1.5.3.diff added |
---|
by , 13 years ago
Attachment: | regress-2.0.0SVN.diff added |
---|
comment:1 by , 13 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
comment:2 by , 13 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
comment:4 by , 13 years ago
Resolution: | fixed |
---|---|
Status: | closed → reopened |
see ticket #1140. This is not working on my windows VC++ 9.0 and 9.1beta3 builds in trunk. I'll try my 8.4 next
comment:6 by , 13 years ago
Component: | postgis → build/upgrade/install |
---|
comment:7 by , 13 years ago
robe, as #1140 is closed, will you take care of double checking this one ? Also, it's reported to happen in 1.5, do you want it fixed in 1.5 branch as well ?
comment:8 by , 13 years ago
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
Assuming everyone is happy about this as is.
Committed to 1.5 branch as r7655 and to trunk as r7656
Thanks a lot!