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

boolean attribute in where clause #379

Merged
merged 1 commit into from
Mar 31, 2014

Conversation

jvshahid
Copy link
Contributor

This query never returns for us:

select identifier, signed_in from events where signed_in limit 10;

Removing the WHERE clause we get an immediate result:

select identifier, signed_in from events limit 10;

I get this in the log file (log level: info) of the server receiving the query (we have a cluster of three machines):

********************************BUG********************************
Database: tonne
Query: [select identifier, signed_in from events where signed_in limit 10;]
Error: runtime error: index out of range. Stacktrace: goroutine 45 [running]:
coordinator.recoverFunc(0xc210047c66, 0x5, 0xc21053f5f0, 0x42)
    /home/vagrant/influxdb/src/coordinator/coordinator.go:342 +0x106
runtime.panic(0x883480, 0x1001917)
    /home/vagrant/bin/go/src/pkg/runtime/panic.c:248 +0x106
parser.getTime(0xc2117903c0, 0xc211790300, 0x0, 0x0, 0x101a500, ...)
    /home/vagrant/influxdb/src/parser/query_api.go:384 +0xa7c
parser.parseSelectDeleteCommonQuery(0xc21053f5f0, 0x42, 0x7ff1e8015650, 0x7ff1e8015380, 0xc20fe86fc0, ...)
    /home/vagrant/influxdb/src/parser/parser.go:591 +0x1cf
parser.parseSelectQuery(0xc21053f5f0, 0x42, 0x7ff1e8013640, 0x0, 0x0, ...)
    /home/vagrant/influxdb/src/parser/parser.go:620 +0x73
parser.ParseQuery(0xc21053f5f0, 0x42, 0x0, 0x0, 0x0, ...)
    /home/vagrant/influxdb/src/parser/parser.go:529 +0x579
coordinator.(*CoordinatorImpl).RunQuery(0xc2100a4900, 0x7ff1ed6e96e0, 0xc210071b00, 0xc210047c66, 0x5, ...)
    /home/vagrant/influxdb/src/coordinator/coordinator.go:85 +0x196
api/http.func·004(0x7ff1ed6e96e0, 0xc210071b00, 0x7c66e0, 0xc211265730

I use the web interface, InfluxDB 0.5.1, Debian Wheezy, Amd64 architecture.

@pauldix pauldix added this to the 0.5.3 milestone Mar 31, 2014
@pauldix
Copy link
Member

pauldix commented Mar 31, 2014

So this would be the right way to do that query: select identifier, signed_in from events where signed_in = true limit 10

However, we don't support that yet. We'll add it in for a release later this week.

jvshahid added a commit that referenced this pull request Mar 31, 2014
…onditions

boolean attribute in where clause
@jvshahid jvshahid merged commit 8c5bb2a into master Mar 31, 2014
@jvshahid jvshahid deleted the fix-379-boolean-columns-where-conditions branch April 1, 2014 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants