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

Crash when using CONTAINS #7469

Closed
suyZhong opened this issue Mar 13, 2024 · 4 comments
Closed

Crash when using CONTAINS #7469

suyZhong opened this issue Mar 13, 2024 · 4 comments
Labels
bug Something isn't working
Milestone

Comments

@suyZhong
Copy link

Describe the bug
Consider the test cases below. It would crash the monetdb server. The server log (merovingian.log) would show: 2024-03-13 12:47:42 MSG merovingian[165]: database 'monetdb' (2297) has crashed with signal SIGABRT (dumped core)

To Reproduce

DROP TABLE IF EXISTS t0 CASCADE;

CREATE TABLE t0(c0 BOOLEAN, c1 VARCHAR);
INSERT INTO t0(c0, c1) VALUES (true, 'a');
INSERT INTO t0(c0, c1) VALUES (true, 'b');
INSERT INTO t0(c0, c1) VALUES (false, 'c');

SELECT t0.c1, t0.c0 FROM t0 WHERE ((CONTAINS(t0.c1, ''))OR(t0.c0)); -- unexpected end of file

Expected behavior
Should return an error message, or return the correct result.

Screenshots
N/A

Software versions
Build from source:

> docker exec monetdb-test mserver5 --version --dbname=monetdb
MonetDB 5 server 11.50.0 (hg id: 5447912) (64-bit, 128-bit integers)
This is an unreleased version
Copyright (c) 1993 - July 2008 CWI
Copyright (c) August 2008 - 2023 MonetDB B.V., all rights reserved
Visit https://www.monetdb.org/ for further information
Found 503.5GiB available memory, 128 available cpu cores
Libraries:
Compiled by: @305ec6a4ac34 (x86_64-pc-linux-gnu)
Compilation: /usr/bin/cc 
Linking    : /usr/bin/ld

Issue labeling
Bug

Additional context
Kindly inform me if you need more information (e.g. error trace, etc.)

@mvdvm
Copy link
Contributor

mvdvm commented Mar 14, 2024

Issue does not occur on MonetDB 5 server v11.49.6 (hg id: 7449a96702cc) , current Dec2023 branch.
Will test it also on default branch.

monetdb-team pushed a commit that referenced this issue Mar 14, 2024
@mvdvm
Copy link
Contributor

mvdvm commented Mar 14, 2024

Issue does not occur on MonetDB 5 server v11.50.0 (hg id: b492ab0d8ac1), current default branch.

@mvdvm mvdvm closed this as completed Mar 14, 2024
@suyZhong
Copy link
Author

Hi @mvdvm, thanks for your effort.

However, it looks like I could still reproduce this on the latest commit bb8ded6. Not sure if it's I built with assertions enabled, or I made something wrong.

Here's how I build MonetDB (in a docker container). According to this, I think I have enabled it?

mkdir build && \
cd build && \
cmake .. && \
cmake --build . && \
cmake --build . --target install

Here's the most latest two logs in the merovingian.log

2024-03-15 14:52:45 ERR monetdb[176]: mserver5: /app/MonetDB/gdk/gdk_cand.c:438: canditer_init: Assertion `s->ttype == TYPE_msk|| s->tsorted' failed.
2024-03-15 14:52:50 MSG merovingian[159]: database 'monetdb' (176) has crashed with signal SIGABRT (dumped core)

@mvdvm
Copy link
Contributor

mvdvm commented Mar 15, 2024

Correct.
I also noticed that on our nightly testweb however only for 1 platform. See contains-or-7469

It triggers the assertion: gdk/gdk_cand.c:438: canditer_init: Assertion s->ttype == TYPE_msk|| s->tsorted' failed.

@mvdvm mvdvm reopened this Mar 15, 2024
@mvdvm mvdvm added the bug Something isn't working label Mar 15, 2024
monetdb-team pushed a commit that referenced this issue Mar 15, 2024
@mvdvm mvdvm added this to the NEXTRELEASE milestone Mar 20, 2024
@mvdvm mvdvm closed this as completed Mar 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants