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

comments cause error with return in function #6181

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

comments cause error with return in function #6181

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

Comments

@monetdb-team
Copy link

Date: 2017-01-01 23:52:22 +0100
From: muesli4
To: MonetDB5 devs <>
Version: 11.25.3 (Dec2016)
CC: @mlkersten

Last updated: 2019-04-30 12:36:05 +0200

Comment 24860

Date: 2017-01-01 23:52:22 +0100
From: muesli4

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

Running the snippet results in: ERROR = !SyntaxException:user.f152[0]:RETURN missing

(Couldn't reduce it further.)

Reproducible: Always

Comment 24861

Date: 2017-01-01 23:53:23 +0100
From: muesli4

Created attachment 525
Snippet that doesn't work.

Attached file: missing return.mal (application/octet-stream, 10311 bytes)
Description: Snippet that doesn't work.

Comment 24862

Date: 2017-01-01 23:54:00 +0100
From: muesli4

Created attachment 526
Snippet that works.

Attached file: without comments.mal (application/octet-stream, 7261 bytes)
Description: Snippet that works.

Comment 24983

Date: 2017-02-07 19:46:02 +0100
From: @mlkersten

Works for me on the Dec2016 branch as follows:
mserver5 y
where y contains the function.
It will produce a !SyntaxException:user.f152[0]:RETURN missing
if you do it as follows:
mserver5 <y

It is known to lead to an exception in the Default branch as of mid December 2016, because we removed the superflous 'sub' in the MAL interface.

The exception would complain about the 'algebra.subselect'

Comment 25384

Date: 2017-06-20 15:34:35 +0200
From: @sjoerdmullender

The problem is in MALreader/MCreadClient. For some reason, this stops reading (when reading from mclient) after 10240 bytes. The file is just a little longer, causing the last sql.bind and return statements to be missed in the plan.

@martin: use mclient -lmal to send the file to the server.

Comment 25388

Date: 2017-06-21 10:01:40 +0200
From: @mlkersten

When I run it on Jul2017 branch as follow
mserver5 <x.mal
...

mserver5: /export/scratch1/mk/Jul2017//package/monetdb5/mal/mal_instruction.c:151: resizeMalBlk: Assertion `mb->vsize >= mb->ssize' failed.
Aborted (core dumped)

This assertion uses the fact that each MAL statement has at least one return variable that should be accommodated by the symbol table.

Comment 26926

Date: 2019-03-18 23:54:00 +0100
From: @mlkersten

This seems to work on default Mar 2019

@monetdb-team monetdb-team added bug Something isn't working MAL/M5 normal 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 MAL/M5 normal
Projects
None yet
Development

No branches or pull requests

2 participants