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

Sqlitelogictest: Wrong MAL plan generation for column product #6513

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

Sqlitelogictest: Wrong MAL plan generation for column product #6513

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

Comments

@monetdb-team
Copy link

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

Date: 2018-01-12 14:52:00 +0100
From: @PedroTadim
To: SQL devs <>
Version: -- development

Last updated: 2018-03-29 15:39:22 +0200

Comment 26076

Date: 2018-01-12 14:52:00 +0100
From: @PedroTadim

User-Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/63.0.3239.132 Safari/537.36
Build Identifier:

MonetDB generates a nonexistent MAL function call in select query, followed by an overflow. It is not possible to obtain a long integer with a product of two byte atoms, so the correct MAL call should be generated.

Reproducible: Always

Steps to Reproduce:

  1. CREATE TABLE tab1(col0 INTEGER, col1 INTEGER, col2 INTEGER);
  2. INSERT INTO tab0 VALUES(97,1,99);
  3. INSERT INTO tab0 VALUES(15,81,47);
  4. INSERT INTO tab0 VALUES(87,21,10);
  5. SELECT DISTINCT - + col2 * 68 * - + 22 FROM tab0 AS cor0;

Actual Results:

The following error, followed by an overflow:
TypeException:user.main[14]:'calc.' undefined in: X_1008:lng := calc.(X_1006:bte, X_1007:bte);
ERROR: 22003!overflow in calculation 68*-22.

Expected Results:

A single column result set with values 148104, 14960, 70312.

Comment 26077

Date: 2018-01-12 14:56:49 +0100
From: @PedroTadim

First step is wrong, sorry:

  1. CREATE TABLE tab0(col0 INTEGER, col1 INTEGER, col2 INTEGER);

Comment 26079

Date: 2018-01-12 15:12:42 +0100
From: MonetDB Mercurial Repository <>

Changeset 9a05add39d7b made by Pedro Ferreira pedro.ferreira@monetdbsolutions.com in the MonetDB repo, refers to this bug.

For complete details, see https//devmonetdborg/hg/MonetDB?cmd=changeset;node=9a05add39d7b

Changeset description:

Added tests for bugs 6513 and 6514

Comment 26083

Date: 2018-01-16 15:17:31 +0100
From: MonetDB Mercurial Repository <>

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

For complete details, see https//devmonetdborg/hg/MonetDB?cmd=changeset;node=ff833040fa32

Changeset description:

Enabled the symbol FULL_IMPLEMENTATION in gdk_calc.c.
And made the newly available interfaces accessible through MAL.

This fixes bug #6513.

Comment 26154

Date: 2018-02-01 10:30:53 +0100
From: @sjoerdmullender

Works in default branch.

Comment 26322

Date: 2018-03-29 15:39:22 +0200
From: @sjoerdmullender

The Mar2018 version has been released.

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