Re: [PATCH] Query Jumbling for CALL and SET utility statements

Поиск
Список
Период
Сортировка
От Drouvot, Bertrand
Тема Re: [PATCH] Query Jumbling for CALL and SET utility statements
Дата
Msg-id 09c04aba-7f12-e524-20f7-3f92c1479fdf@amazon.com
обсуждение исходный текст
Ответ на Re: [PATCH] Query Jumbling for CALL and SET utility statements  ("Imseih (AWS), Sami" <simseih@amazon.com>)
Ответы Re: [PATCH] Query Jumbling for CALL and SET utility statements  ("Imseih (AWS), Sami" <simseih@amazon.com>)
Re: [PATCH] Query Jumbling for CALL and SET utility statements  (Jeremy Schneider <schnjere@amazon.com>)
Список pgsql-hackers

Hi,

On 9/1/22 5:13 PM, Imseih (AWS), Sami wrote:
@font-face {font-family:"Cambria Math"; panose-1:2 4 5 3 5 4 6 3 2 4;}@font-face {font-family:Calibri; panose-1:2 15 5 2 2 2 4 3 2 4;}@font-face {font-family:Consolas; panose-1:2 11 6 9 2 2 4 3 2 4;}p.MsoNormal, li.MsoNormal, div.MsoNormal {margin:0in; font-size:11.0pt; font-family:"Calibri",sans-serif;}pre {mso-style-priority:99; mso-style-link:"HTML Preformatted Char"; margin:0in; font-size:10.0pt; font-family:"Courier New";}span.HTMLPreformattedChar {mso-style-name:"HTML Preformatted Char"; mso-style-priority:99; mso-style-link:"HTML Preformatted"; font-family:Consolas;}span.EmailStyle22 {mso-style-type:personal-reply; font-family:"Calibri",sans-serif; color:windowtext;}.MsoChpDefault {mso-style-type:export-only; font-size:10.0pt;}div.WordSection1 {page:WordSection1;}

> Please find attached v2 as an attempt to do so.

+1 to the idea.

Thanks for looking at it!

I think it will be better to evaluate jstate instead of

JUMBLE_UTILITY, such as:

 

if (query->utilityStmt && !jstate)

 

instead of

 

if (query->utilityStmt && !JUMBLE_UTILITY(query->utilityStmt))

 

This will allow for support of potentially other utility statements

In the future without having to teach pg_stat_statements about them.

If a jstate is set for the utility statements, pgss will do the right thing.

 

 

Fair point, thanks!

v3 including this change is attached.

Thanks,
--

Bertrand Drouvot
PostgreSQL Contributors Team
RDS Open Source Databases
Amazon Web Services: https://aws.amazon.com
Вложения

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

Предыдущее
От: David Rowley
Дата:
Сообщение: Re: Add the ability to limit the amount of memory that can be allocated to backends.
Следующее
От: Daniel Gustafsson
Дата:
Сообщение: Re: plpgsql-trigger.html: Format TG_ variables as table (patch)