Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#4874 closed defect (fixed)

PostGIS 3.2 / GEOS 3.10dev windows - subdivide regress

Reported by: robe Owned by: robe
Priority: blocker Milestone: PostGIS 3.2.0
Component: QA/buildbots Version: master
Keywords: Cc:

Description

 ./regress/core/subdivide .. failed (diff expected obtained: /projects/postgis/tmp/3.2.0dev_pg13_geos3.10_gdal3.2.0w64/test_90_diff)
-----------------------------------------------------------------------------
--- ./regress/core/subdivide_expected   2021-03-02 15:19:47.256187600 -0500
+++ /projects/postgis/tmp/3.2.0dev_pg13_geos3.10_gdal3.2.0w64/test_90_out       2021-03-09 22:08:13.942357400 -0500
@@ -4,6 +4,7 @@
 ERROR:  lwgeom_subdivide_prec: cannot subdivide to fewer than 5 vertices per output
 #3522|POINT(1 1)
 #3744|1600000000000000
+SELECT 1857
 4|29321996468.6|29321996468.6|256
 #4211|0.00008316000
 #4217|0.00002463668
-----------------------

I'm getting this locally using winnie's geos 3.10 build from today. Also seeing same issue on winnie. Debbie seems okay so might be windows specific.

The regress looks extremely odd though - like how did a selcted get in there. I think it might be coming from this line:

\i :regdir/core/regress_big_polygon.sql

Change History (3)

comment:1 by robe, 3 years ago

yeh same error on winnie - so not just me. But it must be coming from those notices not being squashed for some reason. Why only this test is puzzling.

comment:2 by robe, 3 years ago

Resolution: fixed
Status: newclosed

I'm still not clear what is different between 3.1 and master. They looked the same to me, but anyway I changed master at

[66cb4dd37638f5a3f8f7755ce9bb4041ed4ec817/git]

just turning quiet on and that seemed to do the trick.

I'm not sure why it was turned off to begin with, and didn't try just taking that line out entirely. Only thought is maybe it's a timing issue that is different with 3.2 from 3.1.

comment:3 by robe, 3 years ago

aha I think I found the smoking gun. It's Sandro :)

I just noticed during run this is the first commit that started failing

[7e6e08512ade1c3ca14d191894d8737480e59ada/git]

Note: See TracTickets for help on using tickets.