Hi,
On 2025-03-21 02:15, Fujii Masao wrote:
Thanks for your review!
>> Personally, I feel 1st patch may be sufficient, but I would appreciate
>> any feedback.
>
> Agreed.
>
> - errdetail("Consistent recovery state has not been yet
> reached.")));
> + errdetail("Consistent recovery state has not been yet
> reached, or snappshot is pending because subtransaction is
> overflowed."),
> + errhint("In the latter case, find and close the transaction
> with more than %d subtransactions", PGPROC_MAX_CACHED_SUBXIDS)));
>
> This message might be too detailed. Instead, how about simplifying it
> to something like: "Consistent recovery state has not been reached,
> or snapshot is not ready for hot standby."
Agreed.
Do you also think the errhint message is unnecessary?
I agree with your idea to add a description of the overflowed
subtransaction in the manual, but I'm not sure all users will be able to
find it.
Some people may not understand what needs to be done to make the
snapshot ready for hot standby.
I think adding an errhint may help those users.
> We can then update the documentation to clarify that overflowed
> subtransactions
> may delay snapshot readiness for hot standby and explain how to address
> it.
> For example, the current description - "it will begin accepting
> connections once
> the recovery has brought the system to a consistent state." - should be
> updated
> to reflect this condition.
--
Regards,
--
Atsushi Torikoshi
Seconded from NTT DATA GROUP CORPORATION to SRA OSS K.K.
