OpenSim
/
OpenSimMirror
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

* If an individual scene object throws an exception while storing, deal with this locally rather than letting it propogate up the stack 

* This will allow other scene objects to persist and stop the exception taking down the whole region server
0.6.0-stable
Justin Clarke Casey 2008-09-16 17:30:30 +00:00
parent 0b52453762
commit ec4189b722
2 changed files with 33 additions and 23 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
OpenSim/Region/Environment/Scenes/Scene.cs
View File

@ -885,7 +885,6 @@ namespace OpenSim.Region.Environment.Scenes
{ {
EventManager.TriggerOnBackup(m_storageManager.DataStore); EventManager.TriggerOnBackup(m_storageManager.DataStore);
m_backingup = false; m_backingup = false;
//return true;
} }
#endregion #endregion

27
OpenSim/Region/Environment/Scenes/SceneObjectGroup.cs
View File

@ -1124,10 +1124,14 @@ namespace OpenSim.Region.Environment.Scenes
#region Events #region Events
/// <summary> /// <summary>
/// Processes backup /// Processes backup.
/// </summary> /// </summary>
/// <param name="datastore"></param> /// <param name="datastore"></param>
public void ProcessBackup(IRegionDataStore datastore) public void ProcessBackup(IRegionDataStore datastore)
{
// Since this is the top of the section of call stack for backing up a particular scene object, don't let
// any exception propogate upwards.
try
{ {
if (HasGroupChanged) if (HasGroupChanged)
{ {
@ -1135,7 +1139,7 @@ namespace OpenSim.Region.Environment.Scenes
if ((!IsSelected) && (RootPart != null)) if ((!IsSelected) && (RootPart != null))
{ {
m_log.InfoFormat( m_log.InfoFormat(
"[SCENE]: Storing object {0}, {1} in {2}", "[SCENE]: Storing {0}, {1} in {2}",
Name, UUID, m_scene.RegionInfo.RegionName); Name, UUID, m_scene.RegionInfo.RegionName);
SceneObjectGroup backup_group = Copy(OwnerID, GroupID, false); SceneObjectGroup backup_group = Copy(OwnerID, GroupID, false);
@ -1151,12 +1155,19 @@ namespace OpenSim.Region.Environment.Scenes
backup_group = null; backup_group = null;
} }
// else // else
// { // {
// m_log.DebugFormat( // m_log.DebugFormat(
// "[SCENE]: Did not update persistence of object {0} {1}, selected = {2}", // "[SCENE]: Did not update persistence of object {0} {1}, selected = {2}",
// Name, UUID, IsSelected); // Name, UUID, IsSelected);
// } // }
}
}
catch (Exception e)
{
m_log.ErrorFormat(
"[SCENE]: Storing of {0}, {1} in {2} failed with exception {3}",
Name, UUID, m_scene.RegionInfo.RegionName, e);
} }
} }