Robert Haas <robertmh...@gmail.com> writes: > On Tue, Jul 1, 2014 at 8:22 AM, Christoph Berg <c...@df7cb.de> wrote: >> I have two comments on the patch: >> >> The choice to make the behavior depend first on PG_OOM_ADJUST_FILE and >> only secondly on PG_OOM_ADJUST_VALUE seems the wrong way round to me.
> Of course, we have no guarantee that the Linux kernel guys won't > change this again. Apparently "we don't break userspace" is a > somewhat selectively-enforced principle. Yeah, I'm unexcited about this proposal. In any case, given the two existing APIs we have to deal with, allowing PG_OOM_ADJUST_VALUE to default to "0" is sane in both APIs but a default for the file name can work for only one. >> The other bit is the non-deprecation of OOM_ADJ in >> contrib/start-scripts/linux. It took me a while to understand the >> logic of the variables used there - the interface would be much >> clearer if it just was like this: >> ... and then use PG_OOM_ADJUST_FILE below instead of manually figuring >> out which proc file to write to by looking at OOM.*_ADJ. > I can't help but agree with this, though. Fair enough. I went for a minimum-change approach when hacking that script, but we could change it some more in the name of readability. Will do something about that. regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers