| From: | Melanie Plageman <melanieplageman(at)gmail(dot)com> |
|---|---|
| To: | Sami Imseih <samimseih(at)gmail(dot)com> |
| Cc: | Benoit Lobréau <benoit(dot)lobreau(at)dalibo(dot)com>, Tomas Vondra <tomas(at)vondra(dot)me>, Guillaume Lelarge <guillaume(at)lelarge(dot)info>, Alena Rybakina <a(dot)rybakina(at)postgrespro(dot)ru>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, "Imseih (AWS), Sami" <simseih(at)amazon(dot)com>, Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>, Robert Haas <robertmhaas(at)gmail(dot)com>, Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>, "Andrey M(dot) Borodin" <x4mmm(at)yandex-team(dot)ru> |
| Subject: | Re: Logging parallel worker draught |
| Date: | 2025-04-07 16:41:24 |
| Message-ID: | CAAKRu_Yu+2ah=G-yeLe04YzCp1FMCpWXBQTXPCy+SArojVVLPw@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Mon, Feb 3, 2025 at 12:37 AM Sami Imseih <samimseih(at)gmail(dot)com> wrote:
>
> Besides that, I think this is ready for committer.
I started looking at this, and I like the idea.
A few comments: I don't understand what 0002 is. For starters, the
commit message says something about pg_stat_database, and there are no
changes related to that.
Also, we already have basically identical logging coming from
parallel_vacuum_process_all_indexes() and viewable in existing output.
Not only does your implementation not replace this, it is odd that
setting your new guc to none does not disable this. It seems a bit
inconsistent. I'm not sure what the exact right behavior is here,
though.
Since your last update, it seems parallel gin index build has been
committed, so perhaps you want to add that.
- Melanie
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Jacob Champion | 2025-04-07 16:41:25 | Re: [PoC] Federated Authn/z with OAUTHBEARER |
| Previous Message | Andres Freund | 2025-04-07 16:38:28 | Re: Enhancing Memory Context Statistics Reporting |