Re: [PATCH v3 0/3] Add a JSON Schema for trace2 events
- From
Johannes Schindelin <johannes.schindelin@gmx.de>
- Date
- Jul 31, 2019, 11:00 UTC
- Message-ID
- <nycvar.QRO.7.76.6.1907311256030.21907@tvgsbejvaqbjf.bet>
- In-Reply-To
- <20190726135318.GN20404@szeder.dev>
Hi Gábor,
On Fri, 26 Jul 2019, SZEDER Gábor wrote:
Show 14 quoted lines
> On Fri, Jul 26, 2019 at 02:12:39PM +0200, Johannes Schindelin wrote: > > > Besides, I have to admit that I am quite disheartened to see these lines > > in the log: > > > > -- snip -- > > 2019/07/26 11:39:28 Validated items: 0 > > 2019/07/26 11:39:32 Validated items: 10000 > > These are necessary for Travis CI, which aborts the build if it > doesn't receive output for 10 minutes, thinking that the tests hang > somewhere. Yeah, less lines like these would be sufficient to avoid > that timeout, but, of course, the real solution would be to not > validate all of 'make test'.
To be crystal clear: I was not disheartened by seeing the *amount* of lines, but by the sheer amount of time burned up just to verify a feature that only very, very few users will ever need.
The entire validation's runtime rivals the one of a full run of the entire test suite. (Which _already_ takes too long to be effective. Just look how even at least one long-time contributor often simply skips running it before submitting patches, most likely because they don't want to wait.)
Ciao, Dscho