[postgis-tickets] [PostGIS] #3665: Bugs in BRIN support

[PostGIS]

#3665: Bugs in BRIN support
----------------------+---------------------------
 Reporter:  rjuju     |      Owner:  pramsey
     Type:  defect    |     Status:  new
 Priority:  critical  |  Milestone:  PostGIS 2.3.1
Component:  postgis   |    Version:  2.3.x
 Keywords:            |
----------------------+---------------------------
 I found two bugs in the first BRIN support implementation:

 - index corruption bug.  It can happen if a geometry is inserted or
 updated on a previously summarized range (new ranges are not automatically
 summuraized), and isn't contained in the stored bounding box.  This bug is
 fixed in attached fix_brin_corruption.diff

 Please note that this patch expose some required C function which were
 previously only used for GiST indexes.

 - memory leak.  The leak is actually in an existing postgis function.  I
 believe it has been unnoticed until now because IFAICT it's only used in
 GiST indexes, and postgres use a per-tuple MemoryContext for such indexes.
 This bug is fixed in attached fix_brin_memory_leak.diff

 These patches should be applied on 2.3 and trunk.

--
Ticket URL: <https://trac.osgeo.org/postgis/ticket/3665>
PostGIS <http://trac.osgeo.org/postgis/>
The PostGIS Trac is used for bug, enhancement & task tracking, a user and developer wiki, and a view into the subversion code repository of PostGIS project.