Alvaro Herrera <alvherre@2ndquadrant.com> writes:
> Tom Lane wrote:
>> We could stabilize this test result by forcing lc_messages = C in
>> the foreign server options. However, that would lose regression
>> coverage of situations where the remote locale is different, which
>> doesn't seem like a terribly good thing.
> I don't understand what is the benefit of having a different locale
> setting if there's no way that the test would pass except with a very
> specific locale. In other words, what coverage are we really losing by
> going this route?
What the current situation proves (or at least gives evidence for)
is that postgres_fdw doesn't have hidden dependencies on the remote server
running with the same lc_messages that prevails locally. As an example
--- admittedly a bit far-fetched, because this shouldn't ever get past
code review --- if postgres_fdw were checking for the presence of specific
strings in error messages from the remote, we could hope that the
buildfarm would catch that. But if we force the remote lc_messages value
throughout the test, we lose that type of coverage.
regards, tom lane