Skip to main content

Browse the archive

https://github.com/postgres/postgres
02 May 2024, 11:03:54 UTC
Revision 2243ef8dd6fbe1f4ae3ffc7db1ee9af9427ecadb authored by Andres Freund on 12 August 2023, 20:09:45 UTC, committed by Andres Freund on 12 August 2023, 22:08:08 UTC 
ci: macos: Remove use of -Dsegsize_blocks=6
 
The option causes a measurable slowdown. Macos is, by far, the most expensive
platform for CI, therefore it doesn't make sense to run such a test there.

d3b111e3205 used a small segment size for two tasks, one with autoconf, one
with meson. In hindsight that is a bit overkill, it's unlikely that the option
would silently break. Thus don't move the -Dsegsize_blocks=6, just remove
it. I did however change the autoconf test to use 6 instead of 8 blocks, as
long as we allow it, a non-power-of-two test seems like a good idea.

While at it, add a comment explaining why we use a small segment size for CI.

Author: Andres Freund <andres@anarazel.de>
Reviewed-by: Heikki Linnakangas <hlinnaka@iki.fi>
Discussion: https://postgr.es/m/20230808021541.7lbzdefvma7qmn3w@awork3.anarazel.de
Backpatch: 16-, where d3b111e3205 introduced the use of -Dsegsize_blocks=6
1 parent 4bec616
History
Tip revision: 2243ef8dd6fbe1f4ae3ffc7db1ee9af9427ecadb authored by Andres Freund on 12 August 2023, 20:09:45 UTC
ci: macos: Remove use of -Dsegsize_blocks=6
Tip revision: 2243ef8
File Mode Size
config
contrib
doc
src
.cirrus.yml -rw-r--r-- 24.6 KB
.dir-locals.el -rw-r--r-- 730 bytes
.editorconfig -rw-r--r-- 183 bytes
.git-blame-ignore-revs -rw-r--r-- 10.2 KB
.gitattributes -rw-r--r-- 1.5 KB
.gitignore -rw-r--r-- 674 bytes
COPYRIGHT -rw-r--r-- 1.2 KB
GNUmakefile.in -rw-r--r-- 4.2 KB
HISTORY -rw-r--r-- 277 bytes
Makefile -rw-r--r-- 1.8 KB
README -rw-r--r-- 1.2 KB
README.git -rw-r--r-- 721 bytes
aclocal.m4 -rw-r--r-- 365 bytes
configure -rwxr-xr-x 570.5 KB
configure.ac -rw-r--r-- 85.1 KB
meson.build -rw-r--r-- 99.5 KB
meson_options.txt -rw-r--r-- 6.1 KB

README

back to top