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

Add null keys support for MVRBTree indexes [moved] #764

Closed
lvca opened this issue Dec 10, 2012 · 2 comments
Closed

Add null keys support for MVRBTree indexes [moved] #764

lvca opened this issue Dec 10, 2012 · 2 comments

Comments

@lvca
Copy link
Member

lvca commented Dec 10, 2012

This is Issue 764 moved from a Google Code project.
Added by 2012-03-24T19:32:38.000Z by lomakin....@gmail.com.
Please review that bug for more context and additional comments, but update this bug.

Original labels: Type-Enhancement, Priority-Medium, v2.0, uateam

Original description

Currently null keys are not allowed in indexes, but I found that newcomers , still use indexes with null keys, that is not big problem to use null marker value, but it will be more convenient to support null keys in simple and composite keys.

It can be implemented by using NullWrapper object  that will be less than any provided value, but will equals to itself.

Also “is null”  query in  SQL queries will be optimized..

NullWrapper could have a static INSTANCE to avoid to create multiple objects but just 1 per JVM
@ghost ghost assigned andrii0lomakin Feb 2, 2013
@lvca
Copy link
Member Author

lvca commented Mar 28, 2014

Fixed in SBTree

@lvca lvca closed this as completed Mar 28, 2014
@lvca lvca removed this from the 2.0 milestone Mar 28, 2014
@TeamXceleratorDev
Copy link

So what is the recommended way to leverage an index where part of a composite key is NULL. For example, select expand(rid) from INDEX:MHEntity.entityType_owner_id_status where key = [#23:4,NULL,'G3P8TYQ1E7EXQ02Z',#18:0]. This statement returns no rows though it should. Passing "NULL" is OK for syntax, but doesn't yield correct results using 2.0.12.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants