Re: factor out encoding dependent json/jsonb regression tests

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: factor out encoding dependent json/jsonb regression tests
Date: 2015-10-05 18:01:49
Message-ID: 5612BB0D.4090604@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 10/05/2015 01:53 PM, Tom Lane wrote:
> Andrew Dunstan <andrew(at)dunslane(dot)net> writes:
>> I'm rather tired of having to update two sets of output files every time
>> we change the json and jsonb regression tests. Also, other committers
>> occasionally fall over this trap and fail to update json_1.out and
>> jsonb_1.out, as do almost all other patch authors. I propose that we
>> remove those tests to quite small json_utf8.sql and jsonb_utf8.sql
>> (which will each have 2 output files) and that will allow us to get rid
>> of json_1.out and jsonb_1.out. I'd like to backpatch this to 9.4 for
>> jsonb and 9.2 for json. Alternatively, for >= 9.4 we could just add the
>> jsonb encoding-specific tests to json_utf8.sql, to avoid a greater
>> proliferation of tests.
> I'd vote for your alternative plan (ie, just one new regression .sql
> file). It will be so small that having two even-smaller ones seems silly.
>
> Is "json_utf8" really the best name, or should it be "json_encoding"?

I'm fine with "json_enccoding". I'll do it that way.

cheers

andrew

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Oleksii Kliukin 2015-10-05 18:04:47 Re: run pg_rewind on an uncleanly shut down cluster.
Previous Message Tom Lane 2015-10-05 17:53:33 Re: factor out encoding dependent json/jsonb regression tests