Re: Bogus reports from coverage.postgresql.org

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>
Cc: Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>, Stephen Frost <sfrost(at)snowman(dot)net>, Magnus Hagander <magnus(at)hagander(dot)net>, PostgreSQL WWW <pgsql-www(at)lists(dot)postgresql(dot)org>
Subject: Re: Bogus reports from coverage.postgresql.org
Date: 2018-03-14 16:46:39
Message-ID: 6735.1521045999@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-www

I wrote:
> Our process swaps steps 1 and 2. What I now suspect is that
> we accidentally get away with that for basic mode, but it
> doesn't work with lcov_branch_coverage.

I tested that theory by manually doing things in the "correct" order,
and it didn't change the results:

$ /usr/bin/lcov --gcov-tool /usr/bin/gcov -q --no-external -c -i -d . -d . -o lcov_base.info
geninfo: Note: --initial does not generate branch coverage data
$ make check
...
$ /usr/bin/lcov --gcov-tool /usr/bin/gcov -q --no-external -c -d . -d . -o lcov_test.info
$ rm -rf coverage
$ /usr/bin/genhtml -q --legend -o coverage --title='PostgreSQL 11devel' --num-spaces=4 --prefix='/home/tgl/pgsql' lcov_base.info lcov_test.info
$ grep bogus coverage/src/backend/optimizer/util/predtest.c.*
coverage/src/backend/optimizer/util/predtest.c.gcov.html:<span class="lineNum"> 492 </span> :<span class="lineCov"> 10186 : elog(ERROR, &quot;predicate_classify returned a bogus value&quot;);</span>
coverage/src/backend/optimizer/util/predtest.c.gcov.html:<span class="lineNum"> 803 </span> :<span class="lineCov"> 59482 : elog(ERROR, &quot;predicate_classify returned a bogus value&quot;);</span>

I'm still a bit suspicious that we may have a process problem,
but it's looking like there may be grounds for an lcov bug
report.

regards, tom lane

In response to

Responses

Browse pgsql-www by date

  From Date Subject
Next Message Alvaro Herrera 2018-03-14 17:50:08 Re: Main page of wiki.postgresql.org
Previous Message Tom Lane 2018-03-14 16:31:45 Re: Bogus reports from coverage.postgresql.org