WSDOT / HATS

Highway Activity Tracking System Issue Tracker
6 stars 0 forks source link

Redownload of changed schema basemap crashes Hats #1205

Closed ReedHunter-Wsdot closed 5 years ago

ReedHunter-Wsdot commented 5 years ago

When a basemap with one layer removed and an identical version number was redownloaded, it crashed Hats when the map tried to load from this source. A restart of Hats worked around this problem.

Redownloads of Hats basemaps with new layers added didn't seem to trigger this failure.

ReedHunter-Wsdot commented 5 years ago

Redownload of any new basemap seems to crash Hats, and I'm now wondering if this isn't a variation on what was blocking registration of offline feature maps -- that once again if any reference to the data is left over after a redownload then this will happen.

I should be able to confirm this Monday. A redownload test starting from a more reduced basemap still triggered a crash, so I'm now thinking this is going to happen regardless of the new basemap.

ReedHunter-Wsdot commented 5 years ago

Found the needle in the haystack. An array of delayed load milepost map layers wasn't being cleared when downloading a new basemap file, so a memory lock on the old basemap database wasn't being released. The line of code trying to load the new file then would crash Hats, and the new file would load fine after a restart.

After adding this to the layer references being released, downloading a different basemap went fine. I'll merge and check-in my fix on Monday.

ReedHunter-Wsdot commented 5 years ago

Make that Tuesday since Monday is MLK day. =)

ReedHunter-Wsdot commented 5 years ago

The fix is checked in.