Doc: add some commentary about ExecutorRun's NoMovement special case.
authorTom Lane <tgl@sss.pgh.pa.us>
Tue, 10 Dec 2024 22:17:28 +0000 (17:17 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Tue, 10 Dec 2024 22:17:28 +0000 (17:17 -0500)
commit982890530379235974a7e99fd72e862ae40f1400
treec94677fa488822e069c88ba83e1f34d596a40eaa
parent8b9cbf4922756d912309567f83f81e41f1737ac7
Doc: add some commentary about ExecutorRun's NoMovement special case.

Robert Haas expressed concern about whether commit 3eea7a0c9 exposed
the parallel-execution machinery to a case it isn't tested for, namely
a second non-parallel execution of a plan after a parallel execution.
Investigation shows that that can't happen because of pquery.c's
manipulation of the scan direction, but it sure wasn't obvious to
start with.  Add some commentary about that.

Discussion: https://postgr.es/m/CA+TgmoagyKQy=HFw+wLo0AKTYWHui+iKswZ8Jnqqd-cFby-WVg@mail.gmail.com
src/backend/executor/execMain.c