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 monetdb5/modules/kernel/Tests/vacuum.mal fails since changeset ada288cef37c #2994

Closed
monetdb-team opened this issue Nov 30, 2020 · 0 comments

Comments

@monetdb-team
Copy link

@monetdb-team monetdb-team commented Nov 30, 2020

Date: 2012-02-03 11:04:22 +0100
From: @drstmane
To: MonetDB5 devs <>
Version: -- development
CC: @yzchang

Last updated: 2012-11-27 14:14:52 +0100

Comment 16857

Date: 2012-02-03 11:04:22 +0100
From: @drstmane

The following tests fail since changeset ada288cef37c
http://dev.monetdb.org/hg/MonetDB/rev/ada288cef37c

monetdb5/mal/Tests/tst654.mal
http://monetdb.cwi.nl/testweb/tests/43018:6e534d939e36/GNU-Fedora-x86_64-propcheck-assert/monetdb5/mTests/mal/tst654.out.diff.html

monetdb5/mal/Tests/tst655.mal
http://monetdb.cwi.nl/testweb/tests/43018:6e534d939e36/GNU-Fedora-x86_64-propcheck-assert/monetdb5/mTests/mal/tst655.out.diff.html

IMHO the key property must not be set in both cases, since there is a NULL value (rendered as "unknown")

monetdb5/modules/kernel/Tests/vacuum.mal
http://monetdb.cwi.nl/testweb/tests/43018:6e534d939e36/GNU-Fedora-x86_64-propcheck-assert/monetdb5/mTests/modules/kernel/vacuum.err.diff.html

times out after 100s

Comment 16859

Date: 2012-02-03 12:29:20 +0100
From: @drstmane

Correction for tests monetdb5/mal/Tests/tst654.mal & monetdb5/mal/Tests/tst655.mal

Since there is only one NULL values, the "key" property does hold after all (at least for the first/left of the two BATs is was already approved as correct.

Hence, I approved the new output for tests monetdb5/mal/Tests/tst654.mal & monetdb5/mal/Tests/tst655.mal and changed the subject of this bug report to refer only to test monetdb5/modules/kernel/Tests/vacuum.mal

Comment 17174

Date: 2012-04-27 01:15:50 +0200
From: @drstmane

This appears to be fixed in Apr2012_1, Apr2012 (latest) and default:

http://monetdb.cwi.nl/testweb/web/testgrid.php?serial=43870:4702495ed780&module=monetdb5

http://monetdb.cwi.nl/testweb/web/testgrid.php?serial=43950:702b55cbad26&module=monetdb5

http://monetdb.cwi.nl/testweb/web/testgrid.php?serial=43935:bd7bcdd865f7&module=monetdb5

Comment 18075

Date: 2012-11-27 14:14:52 +0100
From: @yzchang

This is about existing tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant