On Tue, 6 Oct 2026 at 03:09, Tatsuya Kawata <[email protected]> wrote: > > 1. I'm confused by the following in explain.sql. We don't support > > parallel function scans, so the comments and setting of > > max_parallel_workers_per_gather seem bogus: > > Function Scan isn't parallel-aware, but generate_series() is parallel > safe, so with debug_parallel_query = regress the whole plan runs in a > parallel worker. EXPLAIN only reads the storage information from the > leader's tuplestore, so the Storage line doesn't appear in that case. > > Indeed, when I removed the setting and ran make check with > debug_parallel_query = regress, the Storage line disappeared from all > three of the new tests and the explain test failed. > > So I kept the setting and rewrote the comment to make the reason > clearer.
That doesn't seem great. On checking, I see Material has the same issue, but that's not been highlighted by the buildfarm members running debug_parallel_query = regress due to lack of EXPLAIN ANALYZE test that has a Material node. I've just posted a patch to fix that in [1]. Can you follow that code and resolve this issue in your patch? David [1] https://postgr.es/m/caaphdvouwcms-b27mpdkd5_ruftanvrxpgn9u3qoqbbihk+...@mail.gmail.com
