RE: Improve logging when using Huge Pages

Поиск
Список
Период
Сортировка
От Shinoda, Noriyoshi (PN Japan FSIP)
Тема RE: Improve logging when using Huge Pages
Дата
Msg-id TU4PR8401MB11529CCEE0FCC07EEE9F8FA7EE859@TU4PR8401MB1152.NAMPRD84.PROD.OUTLOOK.COM
обсуждение исходный текст
Ответ на Re: Improve logging when using Huge Pages  (Justin Pryzby <pryzby@telsasoft.com>)
Ответы Re: Improve logging when using Huge Pages  (Justin Pryzby <pryzby@telsasoft.com>)
Re: Improve logging when using Huge Pages  (Masahiko Sawada <sawada.mshk@gmail.com>)
Список pgsql-hackers
Hi,
Thank you for your comment.
The attached patch stops message splitting.
This patch also limits the timing of message output when huge_pages = try and HugePages is not used.

Regards,
Noriyoshi Shinoda

-----Original Message-----
From: Justin Pryzby [mailto:pryzby@telsasoft.com]
Sent: Friday, October 22, 2021 11:38 AM
To: Shinoda, Noriyoshi (PN Japan FSIP) <noriyoshi.shinoda@hpe.com>
Cc: masao.fujii@oss.nttdata.com; pgsql-hackers@postgresql.org; rjuju123@gmail.com; tgl@sss.pgh.pa.us; Kyotaro Horiguchi
<horikyota.ntt@gmail.com>
Subject: Re: Improve logging when using Huge Pages

+               ereport(LOG, (errmsg("Anonymous shared memory was
+ allocated %s huge pages.", with_hugepages ? "with" : "without")));

You shouldn't break a sentence into pieces like this, since it breaks translation.  You don't want an untranslated
"without"to appear in the middle of the translated message. 

There are cases where a component *shouldn't* be translated, like this one:
where "numeric" should not be translated.

src/backend/utils/adt/numeric.c:                                         errmsg("invalid input syntax for type %s:
\"%s\"",
src/backend/utils/adt/numeric.c-                                                        "numeric", str)));

--
Justin

Вложения

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

Предыдущее
От: "tanghy.fnst@fujitsu.com"
Дата:
Сообщение: RE: Skipping logical replication transactions on subscriber side
Следующее
От: Shinya Kato
Дата:
Сообщение: Re: [PATCH] Added TRANSFORM FOR for COMMENT tab completion