Opened 16 years ago

Last modified 16 years ago

#109 closed defect (fixed)

Some operators not supported for Circular

Reported by: robe Owned by: mcayland
Priority: low Milestone: PostGIS 1.4.0
Component: postgis Version: 1.4
Keywords: Cc:

Description

What steps will reproduce the problem? 1.

SELECT foo1.the_geom ~= foo2.the_geom

FROM ((SELECT

ST_LineToCurve(ST_Buffer(ST_SetSRID(ST_Point(i,j),4326), j)) As the_geom

FROM generate_series(-10,50,10) As i

CROSS JOIN generate_series(40,70, 20) As j)) As

foo1 CROSS JOIN ((SELECT ST_LineToCurve(ST_Buffer(ST_SetSRID(ST_Point(i,j),4326), j)) As the_geom

FROM generate_series(-10,50,10) As i

CROSS JOIN generate_series(40,70, 20) As j)) As foo2

LIMIT 3;

What is the expected output? Either not supported for x and y or for it to do something What do you see instead? psql:torturetest.sql:22279: ERROR: lwgeom_same: unknown geometry type: 13 psql:torturetest.sql:22555: ERROR: lwgeom_same: unknown geometry type: 8

I've only tested against latest 1.4. I'm not sure how hard it is to get these to support circular or if its important, but should at least give the type descriptor instead of number.

I also haven't checked log to see how many of these throw this error.

Change History (5)

comment:1 by robe, 16 years ago

This as far as I can tell is just isolated to lwgeom_same

comment:2 by mcayland, 16 years ago

Yeah, I've already swapped over quite a few messages to report the geometry type rather than just the number. I would say we should just do this here, since nobody has complained about it before.

I don't see the fact that we are not 100% feature-complete as being a bug. As long as we don't crash the backend and display a useful message then we should be fine.

ATB,

Mark.

comment:3 by mcayland, 16 years ago

'Fix' committed as r3636. Does that look better?

ATB,

Mark.

comment:4 by mleslie, 16 years ago

Looks complete from here.

comment:5 by pramsey, 16 years ago

<i>(No comment was entered for this change.)</i>

Note: See TracTickets for help on using tickets.