Re: Query is over 2x slower with jit=on
Pierre Ducroquet <p.psql@pinaraf.info>
From: Pierre Ducroquet <p.psql@pinaraf.info>
To: pgsql-hackers@lists.postgresql.org
Date: 2018-08-22T18:48:48Z
Lists: pgsql-hackers
On Wednesday, August 22, 2018 6:51:55 PM CEST Andres Freund wrote: > On 2018-08-22 18:39:18 +0200, Andreas Joseph Krogh wrote: > > Just to be clear; The query really runs slower (wall-clock time), it's not > > just the timing. > > I bet it's not actually running slower, it "just" takes longer to start > up due to the JITing in each worker. I suspect what we should do is to > multiple the cost limits by the number of workers, to model that. But > without the fixed instrumentation that's harder to see... It depends on the query. It has been shown in other threads that query can indeed take longer to run because of JITing : if the cost is too low to fire LLVM optimizer, the generated code can be so bad it will be slower than the non-JIT executor. Cf for instance a previous discussion here : http://www.postgresql-archive.org/PATCH-LLVM-tuple-deforming-improvements-td6029385.html I think it would be interesting to try the query from this thread with a patch forcing the LLVM codegen to O1 (I found no PassManager there to play with, it seems to be an off/on/extreme switch ; patch 0001-LLVM-Use-the-O1-CodeGen- level.patch from thread mentioned above).
Commits
-
Fix issues around EXPLAIN with JIT.
- e97c4d967ba5 11.0 landed
- c03c1449c092 12.0 landed
-
Collect JIT instrumentation from workers.
- e63441c3f5ca 11.0 landed
- 33001fd7a707 12.0 landed
-
Make EXPLAIN output for JIT compilation more dense.
- 6859bd2632d8 11.0 landed
- 52050ad8ebec 12.0 landed
-
Save/restore SPI's global variables in SPI_connect() and SPI_finish().
- 825f10fbda7a 11.0 cited
-
Rationalize handling of single and double quotes in bootstrap data.
- 55d26ff638f0 11.0 cited