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

combined conditions on declared table in User Defined Function definition crashes monetdb #3770

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: 2015-07-19 22:29:32 +0200
From: 357416268
To: SQL devs <>
Version: 11.19.15 (Oct2014-SP4)
CC: 357416268, @njnes

Last updated: 2015-08-28 13:42:49 +0200

Comment 21025

Date: 2015-07-19 22:29:32 +0200
From: 357416268

User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/43.0.2357.134 Safari/537.36
Build Identifier:

Reproducible: Always

Steps to Reproduce:

create table uniquecatalog11(id bigint PRIMARY KEY
, targetid bigint
, ra_avg double
, decl_avg double
, flux_ref double
, datapoints int
, zone smallint
, x double
, y double
, z double
, INACTIVE BOOLEAN
);

  1. create function neighbor30()
    returns table (ra_avg double)
    begin
    declare table uzone (id bigint, ra_avg double);
    insert into uzone select id, ra_avg from uniquecatalog11;
    return table( select ra_avg
    from uzone as u0
    where id between 10 and 20
    and ra_avg between 10 and 20
    );
    end;

Actual Results:

monetdb crashed.

Expected Results:

a udf defined successfully.

default and July branches are tested on.

Comment 21050

Date: 2015-07-22 22:52:46 +0200
From: @njnes

added more protection against declared tables (as they do not have a schema)

Comment 21051

Date: 2015-07-22 22:54:05 +0200
From: MonetDB Mercurial Repository <>

Changeset a0e705004156 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=a0e705004156

Changeset description:

fix for bug #3770, ie protect agains missing schema names

Comment 21217

Date: 2015-08-28 13:42:49 +0200
From: @sjoerdmullender

Jul2015 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