Another Bug In DataWriter

[samira-t]

We discussed about reordering of records in the log file in issue #1088. But it seems that
the problem is more serious than just reordering the records in the DataWriter. Because of reordering of the messages sent from actions to Terminator and DataWriter, it is possible that the DataWriter receives a scenarioMessage after the Flush message, If that happens, an exception is thrown indicating that the DataWriter cannot accept the scenarioMessage. The reason is that after receiving the Flush message, the DataWriter goes to uninitialized state.

[slandelle]

Hi Samira,

Thanks for reporting.

Could you please explain how you managed to detect this bug? Did you experience it or did you analyze the code? Do you run hundreds of tests? Do you have a particular hardware?

I ask this because I personally have never experienced this one bug.

I will make sure that ScenarioMessages can't be lost. However, I wonder if we can really have a design where we can guarantee that we don't lose RequestMessages.

[samira-t]

Hi Stephane,

I have a testing tool that injects delays for the messages to simulate the real delays that may happen in a concurrent environment (when the program run on different machines or in different environment). However, it respects all the semantics of actor programs and Akka library. This tool is still under development; I will release it when it gets ready.

Regards,
Samira

[slandelle]

I just pushed a fix. Could you try it out?

[samira-t]

Sure. I will try it and let you know if I have the same problem. However,
it might not in the next few days since I am a bit busy now.

On Tue, Apr 30, 2013 at 4:16 PM, Stephane Landelle <notifications@github.com

wrote:

I just pushed a fix. Could you try it out?

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/1116#issuecomment-17254671
.

[slandelle]

np, thanks a lot. I'll try to cut a new release for 1.4 next week and ship it.

[samira-t]

It seems that it has been resolved now. Thanks for fixing.

On Tue, Apr 30, 2013 at 4:26 PM, Stephane Landelle <notifications@github.com

wrote:

np, thanks a lot. I'll try to cut a new release for 1.4 next week and ship
it.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/1116#issuecomment-17255281
.

[slandelle]

Cool, thanks for your prompt feedback!