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

implicit NULL value not propagated in distributed/remote query #6353

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

implicit NULL value not propagated in distributed/remote query #6353

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

Comments

@monetdb-team
Copy link

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

Date: 2017-07-14 14:23:25 +0200
From: @bartscheers
To: SQL devs <>
Version: -- development
CC: @njnes

Last updated: 2017-10-26 14:01:35 +0200

Comment 25466

Date: 2017-07-14 14:23:25 +0200
From: @bartscheers

User-Agent: Mozilla/5.0 (X11; Fedora; Linux x86_64; rv:54.0) Gecko/20100101 Firefox/54.0
Build Identifier:

Statement is "insert into tab1 (col1, col2) select col1, col2 from tab2". Here tab2 is defined as a remote table and has a third column col3 which is nullable. If not explicitly written in the statement the query cannot be registered.

Reproducible: Always

Steps to Reproduce:

  1. run monetdb on master and worker node
  2. execute the run.sh (first part on worker, then second part on master)

Actual Results:

(mapi:monetdb://monetdb@rocks078/worker) Cannot register
project (
table(sys.tmpextrsrc) [ "tmpextrsrc"."id" NOT NULL, "tmpextrsrc"."total_flux" NOT NULL, "tmpextrsrc"."x" NOT NULL, "tmpextrsrc"."y" NOT NULL, "tmpextrsrc"."z" NOT NULL, "tmpextrsrc"."%TID%" NOT NULL ] COUNT
) [ "tmpextrsrc"."id" NOT NULL, "tmpextrsrc"."total_flux" NOT NULL, double "NULL" as "L7"."L7", "tmpextrsrc"."x" NOT NULL, "tmpextrsrc"."y" NOT NULL, "tmpextrsrc"."z" NOT NULL ] REMOTE mapi:monetdb://rocks078:50078/worker

Expected Results:

operation successful
2 affected rows

This was tested on the Jul2017 branch (version d0720566e7f8)

Comment 25467

Date: 2017-07-14 14:24:27 +0200
From: @bartscheers

Created attachment 558
init db

Attached file: init_db.sh (application/x-shellscript, 243 bytes)
Description: init db

Comment 25468

Date: 2017-07-14 14:27:03 +0200
From: @bartscheers

Created attachment 559
worker stuff

Attached file: worker.sql (application/sql, 327 bytes)
Description: worker stuff

Comment 25469

Date: 2017-07-14 14:27:30 +0200
From: @bartscheers

Created attachment 560
master stuff

Attached file: master.sql (application/sql, 702 bytes)
Description: master stuff

Comment 25470

Date: 2017-07-14 14:27:47 +0200
From: @bartscheers

Created attachment 561
master buggy

Attached file: master_bug.sql (application/sql, 678 bytes)
Description: master buggy

Comment 25471

Date: 2017-07-14 14:28:53 +0200
From: @bartscheers

Created attachment 562
run script

Attached file: run.sh (application/x-shellscript, 186 bytes)
Description: run script

Comment 25512

Date: 2017-07-28 13:01:25 +0200
From: MonetDB Mercurial Repository <>

Changeset d876a2cc5823 made by Niels Nes niels@cwi.nl in the MonetDB repo, refers to this bug.

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

Changeset description:

fixes for bug #6353, ie handle NULL in rel_read better

Comment 25513

Date: 2017-07-28 13:01:50 +0200
From: @njnes

fixed, ie handle the NULL when parsing atoms in the rel_read code

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