You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
return sql_message("DROP VIEW: unable to drop view '%s'", tname);
return sql_message("DROP VIEW: unable to drop view '%s': is a table", tname);
} else if (t->system) {
return sql_message("DROP VIEW: cannot drop system view '%s'", tname);
} else if (! drop_action && mvc_check_dependency(sql, t->base.id, VIEW_DEPENDENCY, NULL)) {
Date: 2010-11-04 17:25:17 +0100
From: @romulogoncalves
To: SQL devs <>
Version: 2.40.1 (Oct2010) [obsolete]
CC: @njnes
Last updated: 2011-03-28 17:31:29 +0200
Comment 15171
Date: 2010-11-04 17:25:17 +0100
From: @romulogoncalves
User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-GB; rv:1.9.1.15) Gecko/20101027 Fedora/3.5.15-1.fc12 Firefox/3.5.15
Build Identifier:
The following SQL statement gives a wrong error message:
create table t1(id int);
drop view t1;
drop table t1;
MAPI = monetdb@localhost:55000
QUERY = drop view t1;
ERROR = !SQLException:sql.catalog:DROP VIEW: unable to drop view 't1'
It should be:
QUERY = drop view t1;
ERROR = !SQLException:sql.catalog:DROP VIEW: unknown view 't1'
The right error happens once the table t1 is dropped.
Reproducible: Always
Steps to Reproduce:
1.Follow the instructions in "Details"
2.
3.
Actual Results:
QUERY = drop view t1;
ERROR = !SQLException:sql.catalog:DROP VIEW: unable to drop view 't1'
Expected Results:
QUERY = drop view t1;
ERROR = !SQLException:sql.catalog:DROP VIEW: unknown view 't1'
Comment 15172
Date: 2010-11-04 18:10:25 +0100
From: @grobian
would
DROP VIEW: unable to drop view 't1': is a table
be sufficient for you, too?
If so, I think this patch will do it:
diff --git a/sql/src/backends/monet5/sql.mx b/sql/src/backends/monet5/sql.mx
--- a/sql/src/backends/monet5/sql.mx
+++ b/sql/src/backends/monet5/sql.mx
@@ -1817,7 +1817,7 @@
} else if (!t) {
return sql_message("DROP VIEW: unknown view '%s'", tname);
} else if (!isView(t)) {
Comment 15284
Date: 2010-12-05 13:35:17 +0100
From: @njnes
sounds fine to me..
Comment 15335
Date: 2010-12-10 22:27:27 +0100
From: @njnes
applied patch
Comment 15336
Date: 2010-12-10 22:29:10 +0100
From: @njnes
Changeset f022c1b3925f 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=f022c1b3925f
Changeset description:
Comment 15627
Date: 2011-03-28 17:31:29 +0200
From: @sjoerdmullender
The Mar2011 version has been released.
The text was updated successfully, but these errors were encountered: