public inbox for [email protected]  

help / color / mirror / Atom feed

From: Daniel Gustafsson <[email protected]>
To: Илья Чердаков <[email protected]>
Cc: PostgreSQL Hackers <[email protected]>
Cc: [email protected]
Subject: Re: Environment variable to disable diffs file output
Date: Mon, 6 Apr 2026 16:11:02 +0200
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>

> On 6 Apr 2026, at 09:55, Илья Чердаков <[email protected]> wrote:

> I'd like to add a environment variable to the patch to
> disable diff output. Sometimes you have to be creative
> with exploratory testing, modifying test files, etc.,
> and multiple diff outputs can become a nuisance. This
> doesn't change the default behavior, but it does give
> you control over diff output in tests for specific
> scenarios where a large number of failures are expected
> (mass runs or exploratory testing with changing tests).
> 
> The env usage and implementation are somewhat similar
> to PG_REGRESS_DIFF_OPTS
>     PG_REGRESS_DISABLE_DIFFS_OUTPUT=1 make check -s.
> 
> For now, I suggest to simply disabling it, but I'm also
> considering passing the number of lines to output via env
> or via flag pg_regress instead of completely disabling it.
> Something like
>     PG_REGRESS_DIFFS_NUMBER_LINES_OUTPUT=20 make check
> and then when an error occurs, 20 lines will be output
> instead of the default 80. Setting it to 0 disables
> diffs output.

I'm not sure I entirely understand the problem.  If you expect lots of
failures, but also don't want to see the test failures, what is the use of
running the tests?  Why not run the subset you actually care about and expand
that set as testing fixes bugs/issues?

--
Daniel Gustafsson

view thread (4+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected]
  Subject: Re: Environment variable to disable diffs file output
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox