Re: STATISTICS retained in CREATE TABLE ... LIKE (INCLUDING ALL)?
От | Alvaro Herrera |
---|---|
Тема | Re: STATISTICS retained in CREATE TABLE ... LIKE (INCLUDING ALL)? |
Дата | |
Msg-id | 20180305220815.3of7vohbbhhzngpk@alvherre.pgsql обсуждение исходный текст |
Ответ на | Re: STATISTICS retained in CREATE TABLE ... LIKE (INCLUDING ALL)? (Tomas Vondra <tomas.vondra@2ndquadrant.com>) |
Список | pgsql-hackers |
I admit to much head-scratching, erasing my entire ccache cache, the autoconf cache and doing two complete rebuilds from scratch, because I was seeing 40 errors in regression tests. But it turned out to be about this hunk, which was identical to the idea I had while skimming David's original, "hey why don't we just copy the list": > +/* > + * transformExtendedStatistics > + * handle extended statistics > + * > + * Right now, there's nothing to do here, so we just copy the list. > + */ > static void > transformExtendedStatistics(CreateStmtContext *cxt) > { > - ListCell *lc; > - > - foreach(lc, cxt->extstats) > - cxt->alist = lappend(cxt->alist, lfirst(lc)); > + cxt->alist = list_copy(cxt->extstats); > } > > /* But as it turns out, it's wrong! list_concat() is what is needed here, not list_copy. -- Álvaro Herrera https://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
В списке pgsql-hackers по дате отправления: