|From:||John Naylor <jcnaylor(at)gmail(dot)com>|
|Subject:||Re: inconsistency and inefficiency in setup_conversion()|
|Views:||Raw Message | Whole Thread | Download mbox|
On 12/1/18, Dmitry Dolgov <9erthalion6(at)gmail(dot)com> wrote:
> I see that the author keeps patch updated, but I'm a bit worried because of
> lack of full review since probably May. I'm moving it to the next CF, let's
> if there would be more feedback.
> P.S. adding Daniel, since he is assigned as a reviewer.
Having heard nothing in a while, I've removed Daniel as a reviewer to
make room for someone else. He is, of course free to re-add himself.
v8 is attached.
Since it's been a few months since last discussion, I'd like to
summarize the purpose of this patch and advocate for its inclusion in
In the intro thread , I showed that object comments on some
conversions are wrong, and hard to fix given the current setup. This
is a documentation bug of sorts.
Currently, utils/mb/conversion_procs/Makefile has an ad-hoc script to
generate the SQL file, which has to be duplicated in the MSVC tooling,
and executed by initdb.c. Storing the conversions in .dat files
removes the need for any of that.
This patch shaves 5-6% off of initdb. Not as much as hoped, but still
a nice bonus.
|Next Message||Tomas Vondra||2018-12-14 21:35:56||Re: valgrind issues on Fedora 28|
|Previous Message||Tom Lane||2018-12-14 21:11:43||Re: 'infinity'::Interval should be added|