Unnecessary code in createIndex, dropIndex and indexes 

I don't think these are necessary. I could not find any code that uses the names, columns, or valuesets attributes of these particular nodes. The constructor looks awfully similar to the Insert node constructor, so I suspect this was a the result of a copy paste.

Please review this change carefully. If these attributes are in fact useful, then we need to add tests for them.