|From:||Andres Freund <andres(at)anarazel(dot)de>|
|Subject:||Time to change pg_regress diffs to unified by default?|
|Views:||Raw Message | Whole Thread | Download mbox | Resend email|
I personally, and I know of a bunch of other regular contributors, find
context diffs very hard to read. Besides general dislike, for things
like regression test output context diffs are just not well suited.
the salient point (ERROR: 50 is outside the valid range for parameter "effective_io_concurrency" (0 .. 0))
is 130 lines into the diff, whereas it's right at the start in a unified diff.
Issues with one error that causes a lot of followup output changes are
really common in our regression suite.
I personally have PG_REGRESS_DIFF_OPTS set to -dU10, but that doesn't
help much analyzing buildfarm output.
Therefore I propose changing the defaults in pg_regress.c.
|Next Message||Michael Paquier||2017-04-06 22:33:36||Re: Letting the client choose the protocol to use during a SASL exchange|
|Previous Message||Simon Riggs||2017-04-06 22:29:42||Re: Letting the client choose the protocol to use during a SASL exchange|