From aea5d3a84212a236fe1a766131b8e3513f3705a8 Mon Sep 17 00:00:00 2001
From: Robert Adams
Date: Tue, 24 Sep 2013 13:41:44 -0700
Subject: Remove time based terrain storage in SQLite so revision number can be
 used to denote terrain format revision. Add terrain DB format revision codes
 to ISimulationDataStore.cs. Setup so legacy compatible terrain storage and
 fetch is possible while allowing future format extensions.

---
 OpenSim/Data/MSSQL/MSSQLSimulationData.cs   |  2 +-
 OpenSim/Data/MySQL/MySQLSimulationData.cs   |  7 ++++---
 OpenSim/Data/SQLite/SQLiteSimulationData.cs | 14 ++------------
 3 files changed, 7 insertions(+), 16 deletions(-)

(limited to 'OpenSim/Data')

diff --git a/OpenSim/Data/MSSQL/MSSQLSimulationData.cs b/OpenSim/Data/MSSQL/MSSQLSimulationData.cs
index f41f60c..8adddc9 100644
--- a/OpenSim/Data/MSSQL/MSSQLSimulationData.cs
+++ b/OpenSim/Data/MSSQL/MSSQLSimulationData.cs
@@ -576,7 +576,7 @@ ELSE
         /// <param name="regionID">regionID.</param>
         public void StoreTerrain(double[,] terrain, UUID regionID)
         {
-            int revision = Util.UnixTimeSinceEpoch();
+            int revision = (int)DBTerrainRevision.Legacy256;
 
             //Delete old terrain map
             string sql = "delete from terrain where RegionUUID=@RegionUUID";
diff --git a/OpenSim/Data/MySQL/MySQLSimulationData.cs b/OpenSim/Data/MySQL/MySQLSimulationData.cs
index b03a904..5751dc8 100644
--- a/OpenSim/Data/MySQL/MySQLSimulationData.cs
+++ b/OpenSim/Data/MySQL/MySQLSimulationData.cs
@@ -575,6 +575,7 @@ namespace OpenSim.Data.MySQL
         public void StoreTerrain(double[,] ter, UUID regionID)
         {
             m_log.Info("[REGION DB]: Storing terrain");
+            int revision = (int)DBTerrainRevision.Legacy256;
 
             lock (m_dbLock)
             {
@@ -589,10 +590,10 @@ namespace OpenSim.Data.MySQL
 
                         ExecuteNonQuery(cmd);
 
-                        cmd.CommandText = "insert into terrain (RegionUUID, " +
-                            "Revision, Heightfield) values (?RegionUUID, " +
-                            "1, ?Heightfield)";
+                        cmd.CommandText = "insert into terrain (RegionUUID, Revision, Heightfield)"
+                        +   "values (?RegionUUID, ?Revision, ?Heightfield)";
  
+                        cmd.Parameters.AddWithValue("Revision", revision);
                         cmd.Parameters.AddWithValue("Heightfield", SerializeTerrain(ter));
 
                         ExecuteNonQuery(cmd);
diff --git a/OpenSim/Data/SQLite/SQLiteSimulationData.cs b/OpenSim/Data/SQLite/SQLiteSimulationData.cs
index d938b6b..b70af6b 100644
--- a/OpenSim/Data/SQLite/SQLiteSimulationData.cs
+++ b/OpenSim/Data/SQLite/SQLiteSimulationData.cs
@@ -828,22 +828,12 @@ namespace OpenSim.Data.SQLite
         {
             lock (ds)
             {
-                int revision = Util.UnixTimeSinceEpoch();
-
-                // This is added to get rid of the infinitely growing
-                // terrain databases which negatively impact on SQLite
-                // over time.  Before reenabling this feature there
-                // needs to be a limitter put on the number of
-                // revisions in the database, as this old
-                // implementation is a DOS attack waiting to happen.
+                int revision = (int)DBTerrainRevision.Legacy256;
 
                 using (
-                    SqliteCommand cmd =
-                        new SqliteCommand("delete from terrain where RegionUUID=:RegionUUID and Revision <= :Revision",
-                                          m_conn))
+                    SqliteCommand cmd = new SqliteCommand("delete from terrain where RegionUUID=:RegionUUID", m_conn))
                 {
                     cmd.Parameters.Add(new SqliteParameter(":RegionUUID", regionID.ToString()));
-                    cmd.Parameters.Add(new SqliteParameter(":Revision", revision));
                     cmd.ExecuteNonQuery();
                 }
 
-- 
cgit v1.1