On 08/01/2018 05:58 PM, Andres Freund wrote:
Hi,
On 2018-08-01 11:15:38 +0200, Tomas Vondra wrote:
On 08/01/2018 10:40 AM, Michael Banck wrote:
If this was one week before feature freeze, I would agree with you
that it makes sense to ship it with the restart requirement rather
than not shipping it at all. But we're several commitfests away from
v12, so making an effort to having this work without a downtime
looks like a reasonable requirement to me.
Why would all those pieces had to be committed at once? Why not to
commit what we have now (with the restart) and then remove the
restriction in a later commit?
Sure, if all the pieces existed in various degrees of solidness (with
the earlier pieces committable, but later ones needing work), I'd feel
*much* less concerned about it.
That's not what I meant, sorry for not being clearer. My point was that
I see the "without restart" as desirable but optional, and would not
mind treating it as a future improvement.
In a way, the question is how far can we reasonably push the patch
author(s) to implement stuff we consider desirable, but he/she/they
decided it's not worth the time investment at this point.
We push people to only implement something really consistent all the
time.
Sure, but it's somewhat subjective matter - to me this limitation does
not make this particular patch inconsistent. If we can remove it, great.
If not, it's still immensely useful improvement.
To me, it seems like an immensely useful feature even with the restart,
and I don't think the restart is a major burden for most systems (it can
be, if your system has no maintenance windows, or course).
I think it a problem, my problem is more that I don't think it's really
a solution for the problem.
Sure, if there are issues with this approach, that would make it
unacceptable. I'm not sure why would it be an issue for replicas (which
is what you mention elsewhere), considering those don't write data and
so can't fail to update a checksum?
regards
--
Tomas Vondra http://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
