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

Merge table point to wrong columns if columns in partition tables are deleted #6084

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

Comments

@monetdb-team
Copy link

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

Date: 2016-10-24 18:24:43 +0200
From: @rkoopmanschap
To: SQL devs <>
Version: 11.23.13 (Jun2016-SP2)
CC: @njnes

Last updated: 2016-12-21 13:08:25 +0100

Comment 24591

Date: 2016-10-24 18:24:43 +0200
From: @rkoopmanschap

User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.12; rv:49.0) Gecko/20100101 Firefox/49.0
Build Identifier:

richard.koopmanschap@monetdbsolutions.com

Reproducible: Always

Steps to Reproduce:

Run the following SQL code in mclient:

start transaction;

CREATE TABLE test_table (d int);
INSERT INTO test_table VALUES (6),(6);
CREATE MERGE TABLE test_merge_table (t int);
ALTER TABLE test_merge_table ADD TABLE test_table;
SELECT * FROM test_merge_table;

ALTER TABLE test_table ADD COLUMN u int;
UPDATE test_table SET u = 2;
SELECT * FROM test_table;

ALTER TABLE test_table DROP COLUMN d;
SELECT * FROM test_merge_table;

rollback;

Actual Results:

SELECT * FROM mt3;
+------+
| t |
+======+
| 6 |
| 6 |
+------+

SELECT * FROM t5;
+------+------+
| d | u |
+======+======+
| 6 | 2 |
| 6 | 2 |
+------+------+

SELECT * FROM mt3;
+------+
| t |
+======+
| 2 |
| 2 |
+------+

Expected Results:

Altering the partition table should throw an error. An alternative solution would be to build in support for this in some way.

There is a test for this bug report called: merge-table-edit

Comment 24592

Date: 2016-10-24 18:32:36 +0200
From: MonetDB Mercurial Repository <>

Changeset 6f78ed3dedde made by Richard Koopmanschap richard.koopmanschap@monetdbsolutions.com in the MonetDB repo, refers to this bug.

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

Changeset description:

Added test for bug #6084: Altering partition tables of merge tables

Comment 24621

Date: 2016-10-25 09:13:12 +0200
From: MonetDB Mercurial Repository <>

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

Changeset description:

fixes for bug #6084 (ie do not allow changes to tables which
are a part of a merge or replica table).

Comment 24631

Date: 2016-10-26 09:54:31 +0200
From: @njnes

for now we disallow alter add/drop column/constraint on partitions of a merge or replica table.

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