derbyjs / racer

Realtime model synchronization engine for Node.js
1.18k stars 117 forks source link

enable updateIndices by default #214

Closed wenzowski closed 9 years ago

wenzowski commented 9 years ago

Why is updateIndices disabled by default? Performance?

All tests pass but do not merge this as-is: if merging tests should be updated to reflect change in defaults.

closes derbyjs/derby#483

enjalot commented 9 years ago

we don't want to change the default, using it in derby is where we want to turn it on. most refs dont need it, so it would be unnecessary overhead.