Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test sql/test/BugTracker-2011/Tests/history.Bug-2909.sql produces extra profiling output since changeset 817328b74e2f #3117

Closed
monetdb-team opened this issue Nov 30, 2020 · 0 comments
Labels
bug Something isn't working normal SQL

Comments

@monetdb-team
Copy link

Date: 2012-07-10 19:11:49 +0200
From: @drstmane
To: SQL devs <>
Version: 11.11.7 (Jul2012-SP1)

Last updated: 2012-09-21 11:51:26 +0200

Comment 17440

Date: 2012-07-10 19:11:49 +0200
From: @drstmane

Since changeset 817328b74e2f
"
partial de-Mx mal_interpreter Move the performance profiling to the runtime module.
"
cf.,
http://dev.monetdb.org/hg/MonetDB/rev/817328b74e2f

test sql/test/BugTracker-2011/Tests/history.Bug-2909.sql fails, producing extra profiling output, cf.,
http://monetdb.cwi.nl/testweb/tests/44546:a01144d4a0f9/GNU-Fedora-x86_64-propcheck-assert/sql/mTests/test/BugTracker-2011/history.Bug-2909.out.diff.html

Approving the output is not really an option as it varies between runs.
Rather, the extra output should be disabled, again.
Unfortunately, I failed to do so myself ...

Comment 17608

Date: 2012-08-03 10:14:57 +0200
From: @grobian

the extra output is simply illegal, it's in a SQL/MAPI session, so whatever did this, should revert it again

If any extra output is necessary, it should be prefixed with , and NOT positioned between the SQL header and the data (but before or after).

Comment 17609

Date: 2012-08-03 14:03:29 +0200
From: @grobian

This seems to be bigfoot stuff, but I don't know why it is enabled now, nothing seems to do it.

Comment 17696

Date: 2012-08-24 14:56:08 +0200
From: @sjoerdmullender

Jul2012-SP1 has been released.

Comment 17702

Date: 2012-08-24 16:09:19 +0200
From: @sjoerdmullender

Changeset 0901d759538f made by Sjoerd Mullender sjoerd@acm.org in the MonetDB repo, refers to this bug.

For complete details, see http//devmonetdborg/hg/MonetDB?cmd=changeset;node=0901d759538f

Changeset description:

Only print runtimeTiming information when in the (mdb) debugger.
This fixes bugs 3117 and 3145.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working normal SQL
Projects
None yet
Development

No branches or pull requests

2 participants