Re: [COMMITTERS] pgsql: Improve performance of timezone loading,especially pg_timezone_

Поиск
Список
Период
Сортировка
От David Rowley
Тема Re: [COMMITTERS] pgsql: Improve performance of timezone loading,especially pg_timezone_
Дата
Msg-id CAKJS1f8ZCTq8i05zGwBFCO3X8GxtBJeeTzn3YEMARdDBB+T=3Q@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [COMMITTERS] pgsql: Improve performance of timezone loading,especially pg_timezone_  (David Rowley <david.rowley@2ndquadrant.com>)
Ответы Re: [COMMITTERS] pgsql: Improve performance of timezone loading,especially pg_timezone_  (David Rowley <david.rowley@2ndquadrant.com>)
Re: [COMMITTERS] pgsql: Improve performance of timezone loading, especially pg_timezone_  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-committers
On 7 May 2017 at 21:03, David Rowley <david.rowley@2ndquadrant.com> wrote:
> Perhaps we just need to put the NUL char back, to trim off the filename again:
>
> /* If that didn't work, fall through to do it the hard way */
> fullname[fullnamelen] = '\0';
>
> but I've not yet looked into why the file is missing in the first place.

OK, so it looks like GenerateTimezoneFiles in Install.pm for the MSVC
build does not quite do what make install does for src/timezone.
Nothing seems to pass the -p parameter as the following is doing:

install: all installdirs
ifeq (,$(with_system_tzdata))
$(ZIC) -d '$(DESTDIR)$(datadir)/timezone' -p '$(POSIXRULES)' $(TZDATAFILES)

I've attached a patch for review. My perl skills are at "trial and
error" level, so please review carefully.

The attached also adds the NUL char back to fullname in pg_open_tzfile().

--
 David Rowley                   http://www.2ndQuadrant.com/
 PostgreSQL Development, 24x7 Support, Training & Services

Вложения

В списке pgsql-committers по дате отправления:

Предыдущее
От: David Rowley
Дата:
Сообщение: Re: [COMMITTERS] pgsql: Improve performance of timezone loading,especially pg_timezone_
Следующее
От: David Rowley
Дата:
Сообщение: Re: [COMMITTERS] pgsql: Improve performance of timezone loading,especially pg_timezone_