From: | Christoph Berg <myon(at)debian(dot)org> |
---|---|
To: | Stefan Huehner <stefan(at)huehner(dot)org> |
Cc: | pgsql-pkg-debian(at)postgresql(dot)org, stefan(dot)huehner(at)openbravo(dot)com |
Subject: | Re: pgsql 9.3 for debian/sid in pgdg repo has asserts enabled causing big performance issues |
Date: | 2017-06-10 20:21:02 |
Message-ID: | 20170610202102.vz5l4eudghktvwgj@msg.df7cb.de |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-pkg-debian |
Re: Stefan Huehner 2017-06-10 <20170610132747(dot)GD11504(at)huehner(dot)biz>
> In short:
> The following part from latest changelog:
> * debian/rules: Add stub to enable cassert builds (disabled by default).
>
> does not work in practice as in debian/rules file the
>
> #CASSERT_FLAGS := --enable-cassert
>
> line is not commented out as it should have been.
>
> That causes severe peformance issues at least in our use-case of batch-inserting lots of data with runtime going from ca. 1min to >=6min.
Hi,
I'm sorry for not properly announcing and documenting this recent
change properly, we should put a note on the wiki page.
What's happening is that in git the line is indeed commented out, but
then the apt.postgresql.org build scripts will activate the line when
building for sid-pgdg (which is the whole point why that stub logic
got introduced). The idea is that one of the distributions on
apt.postgresql.org should have cassert enabled so the testsuites
exercise more test cases, for the server packages as well as for
extension packages. As sid is Debian's development branch, it made
sense to enable cassert there.
You seem to use the sid-pgdg packages for production data - is there a
reason you are not using jessie-pgdg or stretch-pgdg?
Christoph
From | Date | Subject | |
---|---|---|---|
Next Message | Stefan Huehner | 2017-06-10 23:15:21 | Re: pgsql 9.3 for debian/sid in pgdg repo has asserts enabled causing big performance issues |
Previous Message | Stefan Huehner | 2017-06-10 13:27:47 | pgsql 9.3 for debian/sid in pgdg repo has asserts enabled causing big performance issues |