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 failure when querying a Blob column with order by DESC #6250

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

Assertion failure when querying a Blob column with order by DESC #6250

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-03-23 15:59:15 +0100
From: Martin van Dinther <<martin.van.dinther>>
To: SQL devs <>
Version: 11.25.9 (Dec2016-SP2)
CC: @njnes

Last updated: 2017-03-31 13:28:41 +0200

Comment 25173

Date: 2017-03-23 15:59:15 +0100
From: Martin van Dinther <<martin.van.dinther>>

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

mserver5: ../../dev/gdk/gdk_bat.c:2246: BATassertProps: Assertion `!b->tkey' failed.

Reproducible: Always

Steps to Reproduce:

See test: sql/test/orderidx/Tests/oidx_all_types.sql

QUERY = select "blob100" from all_types order by "blob100" DESC;
ERROR = !write error on stream

Comment 25183

Date: 2017-03-29 11:32:16 +0200
From: @njnes

ordered indices do not have a sql side bat, ie don't access these structures.

Comment 25184

Date: 2017-03-29 12:50:13 +0200
From: @njnes

the sorting of blob's isn't supported by the mal atom functions, currently
only a not equal function is precent. We should deside if we want order for blobs, or (if not) we should not allow order by statements on the sql layer.

Comment 25189

Date: 2017-03-30 10:51:50 +0200
From: @sjoerdmullender

Changesets 4b93dae5fb21 and 675095ddbd82 implemented a sort on BLOBs.

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