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

assertion failed when creating table with default interval #2877

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

assertion failed when creating table with default interval #2877

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

Comments

@monetdb-team
Copy link

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

Date: 2011-09-09 14:41:18 +0200
From: @sjoerdmullender
To: SQL devs <>
Version: 11.5.1 (Aug2011) [obsolete]
CC: @njnes

Last updated: 2011-09-30 10:58:47 +0200

Comment 16202

Date: 2011-09-09 14:41:18 +0200
From: @sjoerdmullender

This table definition causes a failed assertion:

create table t (
c interval hour to second default interval '1:00:00' hour to second
);

mserver5: sql/server/sql_atom.c:329: atom2sql: Assertion `a->data.vtype == 13' failed.

Comment 16203

Date: 2011-09-09 14:48:38 +0200
From: @sjoerdmullender

This works:
create table t (
c interval hour to second default 3600
);
and that is also basically what msqldump produces.

Comment 16204

Date: 2011-09-11 13:04:24 +0200
From: @njnes

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

Changeset description:

fixed crash in atom2sql, see bug #2877

Comment 16205

Date: 2011-09-11 13:05:45 +0200
From: @njnes

crash is fixed. But both versions will now insert incorrect intervals into the c column. (3600 * the required value)

Comment 16316

Date: 2011-09-21 12:43:57 +0200
From: @sjoerdmullender

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

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

Changeset description:

Added test for bug #2877.

Comment 16317

Date: 2011-09-21 16:16:59 +0200
From: @sjoerdmullender

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

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

Changeset description:

Fixed a bunch of interrelated problems with default interval values.
The value that is stored in the default column of the sys.columns
table for an interval column is the number of units in the right-most
field of the interval (i.e. minutes in an INTERVAL DAY TO MINUTE).  So
make sure we multiply the value with the factor for that right-most
field when inserting into a table.
Also, we now declare the global SQL variable current_timezone as an
INTERVAL HOUR TO SECOND.
This fixes bug #2877.

Comment 16356

Date: 2011-09-30 10:58:47 +0200
From: @grobian

Released in Aug2011-SP1

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