Uninterruptible slow geo_ops.c - Mailing list pgsql-hackers

From Alvaro Herrera
Subject Uninterruptible slow geo_ops.c
Date
Msg-id 20151211174826.GF2762@alvherre.pgsql
Whole thread Raw
Responses Re: Uninterruptible slow geo_ops.c  (Marco Nenciarini <marco.nenciarini@2ndquadrant.it>)
Re: Uninterruptible slow geo_ops.c  (Alvaro Herrera <alvherre@2ndquadrant.com>)
Re: [HACKERS] Uninterruptible slow geo_ops.c  (Emre Hasegeli <emre@hasegeli.com>)
List pgsql-hackers
Hi,

A customer of ours hit some very slow code while running the
@>(polygon, polygon) operator with some big polygons.  I'm not familiar
with this stuff but I think the problem is that the algorithm converges
too slowly to a solution and also has some pretty expensive calls
somewhere.  (Perhaps there is also a problem that the algorithm *never*
converges for some inputs ...)

While I'm not familiar with the code itself, and can't post the exact
slow query just yet, I have noticed that it is missing a
CHECK_FOR_INTERRUPTS() call to enable cancelling the slow query.  I'd
backpatch this all the way back.  (The exact issue they hit is mutual
recursion between touched_lseg_between_poly and lseg_between_poly.
Since the latter also recurses on itself, the best way forward seem to
add a check for interrupts in the loop there.)

I will follow up on the actual slowness later, as warranted.

--
Álvaro Herrera                http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

Attachment

pgsql-hackers by date:

Previous
From: Aleksander Alekseev
Date:
Subject: Re: Patch: fix lock contention for HASHHDR.mutex
Next
From: Jim Nasby
Date:
Subject: Remove array_nulls?