fastmail / Ix

automatic generation of JMAP-style APIs
GNU General Public License v2.0
10 stars 5 forks source link

Add ix_bump_state method #100

Closed mmcclimon closed 6 years ago

mmcclimon commented 6 years ago

A situation arose where we needed to artificially bump the state on a result to cause some side effects to happen properly.

This might need tests, but I wasn't sure how best to do that.

rjbs commented 6 years ago

I think this method needs to get a collection lock, to avoid conflicting updates to the state. (ix_set does this, but I assume this is meant to be used outside the ix_set for the collection in question)

wolfsage commented 6 years ago

agreed, a collection lock is probably necessary

mmcclimon commented 6 years ago

Closing this in favor of #101.