Re: ANALYZE ONLY
| От | Melih Mutlu |
|---|---|
| Тема | Re: ANALYZE ONLY |
| Дата | |
| Msg-id | CAGPVpCTDU-j7c81trGtqyjQTzZrNTAr60DxwdOUL6G_gg41_5A@mail.gmail.com обсуждение исходный текст |
| Ответ на | ANALYZE ONLY (Michael Harris <harmic@gmail.com>) |
| Ответы |
Re: ANALYZE ONLY
|
| Список | pgsql-hackers |
Hi Michael,
Michael Harris <harmic@gmail.com>, 23 Ağu 2024 Cum, 13:01 tarihinde şunu yazdı:
V2 of the patch is attached.
-ANALYZE [ ( <replaceable class="parameter">option</replaceable> [, ...] ) ] [ <replaceable class="parameter">table_and_columns</replaceable> [, ...] ]
+ANALYZE [ ( <replaceable class="parameter">option</replaceable> [, ...] ) ] [ [ ONLY ] <replaceable class="parameter">table_and_columns</replaceable> [, ...] ]
I believe moving "[ ONLY ]" to the definitions of table_and_columns below, as you did with "[ * ]", might be better to be consistent with other places (see [1])
+ if ((options & VACOPT_VACUUM) && is_partitioned_table && ! include_children)
There are also some issues with coding conventions in some places (e.g. the space between "!" and "include_children" abode). I think running pgindent would resolve such issue in most places.
Regards,
-- Melih Mutlu
Microsoft
В списке pgsql-hackers по дате отправления: