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

create procedure fails silently #6604

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

create procedure fails silently #6604

monetdb-team opened this issue Nov 30, 2020 · 0 comments
Labels
bug Something isn't working normal SQL

Comments

@monetdb-team
Copy link

Date: 2018-05-17 12:57:57 +0200
From: @swingbit
To: SQL devs <>
Version: 11.27.13 (Jul2017-SP4)
CC: @kutsurak, @njnes

Last updated: 2018-11-14 14:53:32 +0100

Comment 26454

Date: 2018-05-17 12:57:57 +0200
From: @swingbit

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

This succeeds:

sql>create procedure test()
more>begin
more> create table x(i int);
more>end;
operation successful (4.570ms)

This gives no error, but also no success (indeed, it can be repeated and nothing happens):

sql>create procedure test()
more>begin
more> create table x as select * from sys.functions with data;
more>end;
sql>create procedure test()
more>begin
more> create table x as select * from sys.functions with data;
more>end;
sql>

Reproducible: Always

Comment 26455

Date: 2018-05-18 10:48:37 +0200
From: MonetDB Mercurial Repository <>

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

Changeset description:

Steps to fix bug #6604. When creating a table inside a sequential block (UDF) check for the rel_insert case when the table is created with data.

However even in the scenario without data, the table doesn't get created. Another bug :(

Comment 26456

Date: 2018-05-18 11:29:11 +0200
From: @kutsurak

Hi Roberto,

This seems to be fixed in the Mar2018 version:

$ mclient -d supervisor
Welcome to mclient, the MonetDB/SQL interactive terminal (Mar2018)
Database: MonetDB v11.29.3 (Mar2018), 'mapi:monetdb://localhost:50000/supervisor'
Type \q to quit, ? for a list of available commands
auto commit mode: on
sql>create procedure test()
more>begin
more> create table foo (i int);
more>end;
operation successful
sql>create procedure test()
more>begin
more> create table x as select * from sys.functions with data;
more>end;
CREATE PROCEDURE: name 'test' already in use
sql>

Apparently Pedro stumbled upon a different issue while trying to reproduce the problem you are having.

Panos.

Comment 26458

Date: 2018-05-21 13:14:46 +0200
From: @njnes

currently the 'create' within a procedure (or function) is handled as a
declare table.

Comment 26462

Date: 2018-05-22 11:00:19 +0200
From: @swingbit

Niels, I see the status changed to NEEDINFO.
Do you need info from me?

Comment 26677

Date: 2018-11-14 14:53:32 +0100
From: MonetDB Mercurial Repository <>

Changeset 2048aab70e74 made by Martin van Dinther martin.van.dinther@monetdbsolutions.com in the MonetDB repo, refers to this bug.

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

Changeset description:

Adapt test to match issue reported in bug #6604
Added stable output and error files
The original issue reported in 6604 (multiple create proceudre test() succeed) is not reproducable in Aug2018 release
@monetdb-team monetdb-team added bug Something isn't working normal SQL labels Nov 30, 2020
@sjoerdmullender sjoerdmullender added this to the Ancient Release milestone Feb 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working normal SQL
Projects
None yet
Development

No branches or pull requests

2 participants