algebra.join undefined (caused by non-existing variables in the plan)

[monetdb-team]

Date: 2015-03-30 17:53:39 +0200
From: @swingbit
To: SQL devs <>
Version: 11.19.9 (Oct2014-SP2)
CC: @njnes

Last updated: 2015-05-07 12:37:24 +0200

Comment 20749

Date: 2015-03-30 17:53:39 +0200
From: @swingbit

Created attachment 330
SQL script to reproduce the bug

I get this error when executing the query in attachment:

TypeException:user.s14_1[131]:'algebra.join' undefined in: (_210:any, r1_210:any) := algebra.join(_10:bat[:oid,:int], _209:bat[:oid,:str])

The MAL plan in the explain for the same query shows this instruction:

  (X_210,r1_210) := algebra.join(X_10,X_209);

but variable X_10 was NEVER DEFINED.

I thought some of the optimizers could have performed a wrong rewrite, but the same error happens when using 'minimal_pipe'.

Attached file: test.sql (application/sql, 5093 bytes)
Description: SQL script to reproduce the bug

Comment 20750

Date: 2015-03-30 18:06:38 +0200
From: @swingbit

Additional info: I'm running Oct2014 updated to changeset 55186:2f3c782345a2, which is the latest at this point in time.

Comment 20777

Date: 2015-04-10 15:21:01 +0200
From: @njnes

fixed a problem in project rewrites

Comment 20778

Date: 2015-04-10 15:26:49 +0200
From: MonetDB Mercurial Repository <>

Changeset ed162e68df7f made by Niels Nes niels@cwi.nl in the MonetDB repo, refers to this bug.

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

Changeset description:

fixed bug #3693 (fixed problem with project rewrites)

Comment 20789

Date: 2015-04-13 15:34:20 +0200
From: @swingbit

Created attachment 332
Second test, it still fails after the first fix

Attached file: BUG-3693-_2.sql (application/sql, 19535 bytes)
Description: Second test, it still fails after the first fix

Comment 20790

Date: 2015-04-13 15:35:19 +0200
From: @swingbit

The first fix does make my first test succeed, but it still fails on my second test (see second attachment)

Comment 20819

Date: 2015-04-19 21:29:36 +0200
From: @njnes

now even more protection is added, added also the second part of the test.