Secondary Indexes?

[runvnc]

Hello, does this library support Secondary Indexes?

[alexbosworth]

It doesn't - but I posted a method for the new style of query in #44

[runvnc]

Awesome so to use that you would use query_DynamoDB_20120810 and specify keyConditions or something?
Question: how does node-dynamodb currently handle different api versions, or does it only support one version (except for your patch)?

[alexbosworth]

I couldn't really find a good writeup anywhere by Amazon of what they changed exactly (other than the query method)

In my method I tried to make it backwards compatible so that it wouldn't break existing code. If you specify keyConditions in your query, it switches to the new query API, if you leave them out, it goes to the legacy query API

I didn't change any other methods, and if you don't specify keyConditions everything should be the same as the existing module.

[runvnc]

I see now. Nice! So the tests all run fine then?

[alexbosworth]

Everything should be good but re issue #19 there didn't seem to be integration tests for querying

Theoretically though I just changed the basic syntax of the query and kept the same logic as the other query method

One tricky thing I noticed is that it's harder to cause a user error in the original query function. Now that the index restrictions are looser and you have to make sure you specify your hash and range keys by name and not just 'range key', 'hash key', you need to pay more attention to well formed queries than in the original function.