Originally reported as GEOT-532:
The class org.geotools.data.postgis.PostGISDialect uses ST_Simplify() instead of ST_SimplifyPreserveTopology() - see lines 333 and 344.
ST_Simplify will often create extremely small polygons for polygons that have a portion that is quite thin (when the result should be a large polygon).
ST_SimplifyPreserveTopology() is much better at avoiding this (however, its a bit slower).
No, it's not a bit slower, it's a tragedy better not applied at all. This was already discussed in the other report, not sure why another ticket was opened for this one?
Just a note of caution - I believe the perpendicular offset code needs valid polygons to work (at least what the code implied).
The fix jody and I put in should try to fix invalid polygons, but it will not always work - you'll still likely have to deal with them.
Not sure what the impact of st_simplify (i.e. invalid polygons) is with the perpendicular offset code... Might not make any difference at all...
Closing as won't fix as the suggested solution is slower than not simplifying at all (which is an option allowed by the store)
Mass transitioning all resolved issues that have not been updated in the last month to closed state