Fix for warning for missing index needs to be updated. #22

Merged
merged 1 commit into from Nov 27, 2012

Conversation

Projects
None yet
2 participants
@baank
Contributor

baank commented Nov 23, 2012

No description provided.

@baank

This comment has been minimized.

Show comment
Hide comment
@baank

baank Nov 23, 2012

Contributor

If you have a NoSQLQuery that references an unindexed attribute it throws the following warning:

Caused by: java.lang.IllegalArgumentException: You cannot have 'result.id' in your sql query since id has no @index annotation on the field in the entity
at com.alvazan.orm.parser.antlr.ScannerSql.processColumnName(ScannerSql.java:541)
at com.alvazan.orm.parser.antlr.ScannerSql.processSide(ScannerSql.java:443)

It should throw the warning:
You cannot have 'result.id' in your sql query since id has no @nosqlindexed annotation on the field in the entity.

Contributor

baank commented Nov 23, 2012

If you have a NoSQLQuery that references an unindexed attribute it throws the following warning:

Caused by: java.lang.IllegalArgumentException: You cannot have 'result.id' in your sql query since id has no @index annotation on the field in the entity
at com.alvazan.orm.parser.antlr.ScannerSql.processColumnName(ScannerSql.java:541)
at com.alvazan.orm.parser.antlr.ScannerSql.processSide(ScannerSql.java:443)

It should throw the warning:
You cannot have 'result.id' in your sql query since id has no @nosqlindexed annotation on the field in the entity.

easility added a commit that referenced this pull request Nov 27, 2012

Merge pull request #22 from taligent/3eacb1f88d88f0096fd0fb4bb86ddf04…
…baf6c81b

Fix for warning for missing index needs to be updated.

@easility easility merged commit 1934450 into deanhiller:master Nov 27, 2012

easility added a commit that referenced this pull request Mar 4, 2013

Merge pull request #22 from deanhiller/master
merging deanhiller with easility
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment