Opened 4 years ago

Closed 4 years ago

#1246 closed defect (fixed)

Race condition when displaying progress message

Reported by: Nicklas Nordborg Owned by: Nicklas Nordborg
Priority: major Milestone: Reggie v4.27.1
Component: net.sf.basedb.reggie Keywords:


There is a possible race condition in all(?) wizards that use progress bar. If the final progress update just before the work is complete is delayed it seems like the messages from the actual work can be overwritten by progress message. I have never seen it on the development system, but it happened on the production server when testing the new batch importer for follow-up blood (see #1234).

The situation can be triggered by adding a Thread.sleep() call to the GetProgress command.

To avoid the problem I think the progress messages should not use the same element as the final messages from the wizard is using.

Change History (1)

comment:1 by Nicklas Nordborg, 4 years ago

Resolution: fixed
Status: newclosed

In 5951:

Fixes #1246: Race condition when displaying progress message

The progress bar now no longer uses the 'wizard-status' element to report progress messages so it should never overwrite any text that is there.

Note: See TracTickets for help on using tickets.