[postgis-tickets] r14654 - Do not rely on storage order on querying loaded shapefiles

Sandro Santilli strk at keybit.net
Tue Feb 9 08:52:07 PST 2016


Author: strk
Date: 2016-02-09 08:52:07 -0800 (Tue, 09 Feb 2016)
New Revision: 14654

Modified:
   trunk/regress/loader/Arc.select.sql
   trunk/regress/loader/ArcM.select.sql
   trunk/regress/loader/ArcZ.select.sql
   trunk/regress/loader/MultiPoint.select.sql
   trunk/regress/loader/MultiPointM.select.sql
   trunk/regress/loader/MultiPointZ.select.sql
   trunk/regress/loader/PointZ.select.sql
   trunk/regress/loader/Polygon.select.sql
   trunk/regress/loader/PolygonM.select.sql
   trunk/regress/loader/PolygonZ.select.sql
   trunk/regress/loader/TSIPolygon.select.sql
   trunk/regress/loader/TSTIPolygon.select.sql
   trunk/regress/loader/TSTPolygon.select.sql
Log:
Do not rely on storage order on querying loaded shapefiles

Fixes runs with RUNTESTFLAGS=--dumprestore
See #3444

Modified: trunk/regress/loader/Arc.select.sql
===================================================================
--- trunk/regress/loader/Arc.select.sql	2016-02-09 16:40:11 UTC (rev 14653)
+++ trunk/regress/loader/Arc.select.sql	2016-02-09 16:52:07 UTC (rev 14654)
@@ -1,3 +1,3 @@
-select ST_Ashexewkb(the_geom::geometry, 'NDR') from loadedshp order by 1;
-select ST_Ashexewkb(the_geom::geometry, 'XDR') from loadedshp order by 1;
-select ST_Asewkt(the_geom::geometry) from loadedshp order by 1;
+select ST_Ashexewkb(the_geom::geometry, 'NDR') from loadedshp order by gid;
+select ST_Ashexewkb(the_geom::geometry, 'XDR') from loadedshp order by gid;
+select ST_Asewkt(the_geom::geometry) from loadedshp order by gid;

Modified: trunk/regress/loader/ArcM.select.sql
===================================================================
--- trunk/regress/loader/ArcM.select.sql	2016-02-09 16:40:11 UTC (rev 14653)
+++ trunk/regress/loader/ArcM.select.sql	2016-02-09 16:52:07 UTC (rev 14654)
@@ -1,3 +1,3 @@
-select ST_Ashexewkb(the_geom::geometry, 'NDR') from loadedshp order by 1;
-select ST_Ashexewkb(the_geom::geometry, 'XDR') from loadedshp order by 1;
-select ST_Asewkt(the_geom::geometry) from loadedshp order by 1;
+select ST_Ashexewkb(the_geom::geometry, 'NDR') from loadedshp order by gid;
+select ST_Ashexewkb(the_geom::geometry, 'XDR') from loadedshp order by gid;
+select ST_Asewkt(the_geom::geometry) from loadedshp order by gid;

Modified: trunk/regress/loader/ArcZ.select.sql
===================================================================
--- trunk/regress/loader/ArcZ.select.sql	2016-02-09 16:40:11 UTC (rev 14653)
+++ trunk/regress/loader/ArcZ.select.sql	2016-02-09 16:52:07 UTC (rev 14654)
@@ -1,3 +1,3 @@
-select ST_Ashexewkb(the_geom::geometry, 'NDR') from loadedshp order by 1;
-select ST_Ashexewkb(the_geom::geometry, 'XDR') from loadedshp order by 1;
-select ST_Asewkt(the_geom::geometry) from loadedshp order by 1;
+select ST_Ashexewkb(the_geom::geometry, 'NDR') from loadedshp order by gid;
+select ST_Ashexewkb(the_geom::geometry, 'XDR') from loadedshp order by gid;
+select ST_Asewkt(the_geom::geometry) from loadedshp order by gid;

Modified: trunk/regress/loader/MultiPoint.select.sql
===================================================================
--- trunk/regress/loader/MultiPoint.select.sql	2016-02-09 16:40:11 UTC (rev 14653)
+++ trunk/regress/loader/MultiPoint.select.sql	2016-02-09 16:52:07 UTC (rev 14654)
@@ -1,2 +1,2 @@
-select ST_Asewkt(the_geom::geometry) from loadedshp;
+select ST_Asewkt(the_geom::geometry) from loadedshp ORDER BY gid;
 

Modified: trunk/regress/loader/MultiPointM.select.sql
===================================================================
--- trunk/regress/loader/MultiPointM.select.sql	2016-02-09 16:40:11 UTC (rev 14653)
+++ trunk/regress/loader/MultiPointM.select.sql	2016-02-09 16:52:07 UTC (rev 14654)
@@ -1,2 +1,2 @@
-select ST_Asewkt(the_geom::geometry) from loadedshp;
+select ST_Asewkt(the_geom::geometry) from loadedshp ORDER BY gid;
 

Modified: trunk/regress/loader/MultiPointZ.select.sql
===================================================================
--- trunk/regress/loader/MultiPointZ.select.sql	2016-02-09 16:40:11 UTC (rev 14653)
+++ trunk/regress/loader/MultiPointZ.select.sql	2016-02-09 16:52:07 UTC (rev 14654)
@@ -1,2 +1,2 @@
-select ST_Asewkt(the_geom::geometry) from loadedshp;
+select ST_Asewkt(the_geom::geometry) from loadedshp ORDER BY gid;
 

Modified: trunk/regress/loader/PointZ.select.sql
===================================================================
--- trunk/regress/loader/PointZ.select.sql	2016-02-09 16:40:11 UTC (rev 14653)
+++ trunk/regress/loader/PointZ.select.sql	2016-02-09 16:52:07 UTC (rev 14654)
@@ -1 +1 @@
-select ST_Asewkt(the_geom::geometry) from loadedshp order by 1;
+select ST_Asewkt(the_geom::geometry) from loadedshp order by gid;

Modified: trunk/regress/loader/Polygon.select.sql
===================================================================
--- trunk/regress/loader/Polygon.select.sql	2016-02-09 16:40:11 UTC (rev 14653)
+++ trunk/regress/loader/Polygon.select.sql	2016-02-09 16:52:07 UTC (rev 14654)
@@ -1,2 +1,2 @@
-select ST_Asewkt(the_geom::geometry) from loadedshp;
+select ST_Asewkt(the_geom::geometry) from loadedshp ORDER BY gid;
 

Modified: trunk/regress/loader/PolygonM.select.sql
===================================================================
--- trunk/regress/loader/PolygonM.select.sql	2016-02-09 16:40:11 UTC (rev 14653)
+++ trunk/regress/loader/PolygonM.select.sql	2016-02-09 16:52:07 UTC (rev 14654)
@@ -1,2 +1,2 @@
-select ST_Asewkt(the_geom::geometry) from loadedshp;
+select ST_Asewkt(the_geom::geometry) from loadedshp ORDER BY gid;
 

Modified: trunk/regress/loader/PolygonZ.select.sql
===================================================================
--- trunk/regress/loader/PolygonZ.select.sql	2016-02-09 16:40:11 UTC (rev 14653)
+++ trunk/regress/loader/PolygonZ.select.sql	2016-02-09 16:52:07 UTC (rev 14654)
@@ -1,2 +1,2 @@
-select ST_Asewkt(the_geom::geometry) from loadedshp;
+select ST_Asewkt(the_geom::geometry) from loadedshp ORDER BY gid;
 

Modified: trunk/regress/loader/TSIPolygon.select.sql
===================================================================
--- trunk/regress/loader/TSIPolygon.select.sql	2016-02-09 16:40:11 UTC (rev 14653)
+++ trunk/regress/loader/TSIPolygon.select.sql	2016-02-09 16:52:07 UTC (rev 14654)
@@ -1,2 +1,2 @@
-select ST_Asewkt(the_geom) from loadedshp;
+select ST_Asewkt(the_geom) from loadedshp ORDER BY gid;
 

Modified: trunk/regress/loader/TSTIPolygon.select.sql
===================================================================
--- trunk/regress/loader/TSTIPolygon.select.sql	2016-02-09 16:40:11 UTC (rev 14653)
+++ trunk/regress/loader/TSTIPolygon.select.sql	2016-02-09 16:52:07 UTC (rev 14654)
@@ -1,2 +1,2 @@
-select ST_Asewkt(the_geom) from loadedshp;
+select ST_Asewkt(the_geom) from loadedshp ORDER BY gid;
 

Modified: trunk/regress/loader/TSTPolygon.select.sql
===================================================================
--- trunk/regress/loader/TSTPolygon.select.sql	2016-02-09 16:40:11 UTC (rev 14653)
+++ trunk/regress/loader/TSTPolygon.select.sql	2016-02-09 16:52:07 UTC (rev 14654)
@@ -1,2 +1,2 @@
-select ST_Asewkt(the_geom) from loadedshp;
+select ST_Asewkt(the_geom) from loadedshp ORDER BY gid;
 



More information about the postgis-tickets mailing list