Re: pgsql: Clone extended stats in CREATE TABLE (LIKE INCLUDING ALL)

From: Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>
To: Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: pgsql: Clone extended stats in CREATE TABLE (LIKE INCLUDING ALL)
Date: 2018-03-06 02:53:15
Message-ID: 20180306025315.ubtz35byjk3qtza5@alvherre.pgsql
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

Thomas Munro wrote:
> On Tue, Mar 6, 2018 at 11:39 AM, Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> wrote:
> > Clone extended stats in CREATE TABLE (LIKE INCLUDING ALL)
>
> Is this commit responsible for this valgrind error?
>
> https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=lousyjack&dt=2018-03-05%2023%3A03%3A01
>
> ==430== Invalid read of size 2
> ==430== at 0x4C33F6F: memcpy(at)GLIBC_2(dot)2(dot)5 (vg_replace_strmem.c:1021)
> ==430== by 0x483419: heap_fill_tuple (heaptuple.c:276)
> ==430== by 0x4842C6: heap_form_tuple (heaptuple.c:771)
> ==430== by 0x6474B5: CreateStatistics (statscmds.c:322)
> ==430== by 0x865784: ProcessUtilitySlow (utility.c:1668)

Uh ... it probably is. I'll take a look tomorrow, but my guess is that
my last minute removal of the conversion to Name is to blame.

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

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Peter Eisentraut 2018-03-06 15:58:00 pgsql: Tests for Kerberos/GSSAPI authentication
Previous Message Thomas Munro 2018-03-06 02:08:53 Re: pgsql: Clone extended stats in CREATE TABLE (LIKE INCLUDING ALL)

Browse pgsql-hackers by date

  From Date Subject
Next Message Yugo Nagata 2018-03-06 03:09:31 Re: [PATCH] GET DIAGNOSTICS FUNCTION_NAME
Previous Message Michael Paquier 2018-03-06 02:08:55 Re: Kerberos test suite