[postgis-tickets] [PostGIS] #5427: PG17 change broke our code

PostGIS trac at osgeo.org
Mon Jul 3 13:15:33 PDT 2023


#5427: PG17 change broke our code
--------------------------------+---------------------------
 Reporter:  robe                |      Owner:  pramsey
     Type:  defect              |     Status:  new
 Priority:  blocker             |  Milestone:  PostGIS 3.5.0
Component:  postgis             |    Version:  master
 Keywords:  PostgreSQL 17 pg17  |
--------------------------------+---------------------------
 {{{
 /pg/rel/pg16w64/include/postgresql/internal  -D_GNU_SOURCE   -c -o
 gserialized_estimate.o gserialized_estimate.c
 gserialized_estimate.c: In function ‘compute_gserialized_stats_mode’:
 gserialized_estimate.c:1521:46: error: ‘VacAttrStats’ has no member named
 ‘attr’
  1521 |  histo_cells_target =
 (int)pow((double)(stats->attr->attstattarget), (double)ndims);
       |                                              ^~
 make[1]: *** [<builtin>: gserialized_estimate.o] Error 1
 make[1]: Leaving directory
 '/var/lib/jenkins/workspace/postgis/regress_pgdev/branches/3.4/postgis'
 make: *** [GNUmakefile:25: all] Error 1
 Build step 'Execute shell' marked build as failure
 Finished: FAILURE
 }}}

 guessing the cause was this commit:
 https://git.postgresql.org/gitweb/?p=postgresql.git;a=commit;h=c69bdf837f161ae3e699496be10215dbb2315917

 {{{
 Take pg_attribute out of VacAttrStats

 The VacAttrStats structure contained the whole Form_pg_attribute for a
 column, but it actually only needs attstattarget from there.  So
 remove the Form_pg_attribute field and make a separate field for
 attstattarget.  This simplifies some code for extended statistics that
 doesn't deal with a column but an expression, which had to fake up
 pg_attribute rows to satisfy internal APIs.  Also, we can remove some
 comments that essentially said "don't look at pg_attribute directly".
 }}}
-- 
Ticket URL: <https://trac.osgeo.org/postgis/ticket/5427>
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.


More information about the postgis-tickets mailing list