search

simlaudato / asterixdb

Automatically exported from code.google.com/p/asterixdb
0 stars 0 forks source link

Grammar change needed for CREATE INDEX (no-op but less confusing) #911

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago 
Right now the IndexSpecification is:

    IndexSpecification ::= "index" Identifier IfNotExists "on" QualifiedName
                           "(" ( OpenField ) ( "," OpenField )* ")" ( "type" IndexType )? ( "enforced" )?
    IndexType          ::= "btree"
                         | "rtree"
                         | "keyword"
                         | "ngram" "(" IntegerLiteral ")"

There is no reason (and it's confusing IMO) to say OpenField - this is actually 
ANY field, not just any OPEN field.  The grammar probably said Field before and 
should be reverted to that - and the documentation should also be updated 
accordingly.

Original issue reported on code.google.com by dtab...@gmail.com on 24 Jul 2015 at 11:12

GoogleCodeExporter commented 8 years ago 
I fixed this in my channels branch. I can make this a separate patch if it 
needs to get into master quickly

Original comment by sjaco...@ucr.edu on 4 Aug 2015 at 5:18

GoogleCodeExporter commented 8 years ago 
Sure - thanks!  I changed the docs already, so this won't surface for 
users either way - Ian's call.  :-)

Original comment by dtab...@gmail.com on 4 Aug 2015 at 6:06