kiln / flourish-sdk

The SDK for developing Flourish templates
Other
81 stars 16 forks source link

Race condition on change #2

Open robinhouston opened 7 years ago

robinhouston commented 7 years ago

I just saw a case where, as the template was being rebuilt (not by the SDK server), the SDK reloaded with a blank template.js and didn’t pick up the subsequent change.

robinhouston commented 7 years ago

The first step in addressing this would be to create a reliable way of reproducing it.

theduncanclark commented 7 years ago

Has anyone seen this since May?

robinhouston commented 7 years ago

I haven’t seen it recently, but I haven’t fixed it either. I’d bet it’s still a problem. Should be easy enough to repro with an explicit stress test.