[Opensim-dev] [Opensim-commits] r8236 - in trunk/OpenSim: ApplicationPlugins/Rest/Inventory Data/MSSQL Data/MySQL Data/NHibernate Data/SQLite Data/Tests Framework Framework/AssetLoader/Filesystem Framework/Communications/Cache Framework/Communications/Capabilities Grid/AssetServer Region/ClientStack/LindenUDP Region/Communications/Hypergrid Region/Environment/Modules/Agent/AssetTransaction Region/Environment/Modules/Agent/TextureSender Region/Environment/Modules/Avatar/Inventory/Archiver Region/Environment/Modules/Scripting/DynamicTexture Region/Environment/Modules/World/Archiver Region/Environment/Modules/World/Estate Region/Environment/Scenes Region/Environment/Scenes/Hypergrid Region/ScriptEngine/Shared/Api/Implementation

[Sean Dague]

mikem at opensimulator.org wrote:
> Author: mikem
> Date: 2009-02-03 16:01:36 -0800 (Tue, 03 Feb 2009)
> New Revision: 8236
> 
> Modified:
>    trunk/OpenSim/ApplicationPlugins/Rest/Inventory/RestAssetServices.cs
>    trunk/OpenSim/ApplicationPlugins/Rest/Inventory/RestInventoryServices.cs
>    trunk/OpenSim/Data/MSSQL/MSSQLAssetData.cs
>    trunk/OpenSim/Data/MySQL/MySQLAssetData.cs
>    trunk/OpenSim/Data/NHibernate/NHibernateAssetData.cs
>    trunk/OpenSim/Data/SQLite/SQLiteAssetData.cs
>    trunk/OpenSim/Data/Tests/BasicAssetTest.cs
>    trunk/OpenSim/Framework/AssetBase.cs
>    trunk/OpenSim/Framework/AssetLandmark.cs
>    trunk/OpenSim/Framework/AssetLoader/Filesystem/AssetLoaderFileSystem.cs
>    trunk/OpenSim/Framework/Communications/Cache/AssetCache.cs
>    trunk/OpenSim/Framework/Communications/Cache/CryptoGridAssetClient.cs
>    trunk/OpenSim/Framework/Communications/Cache/FileAssetClient.cs
>    trunk/OpenSim/Framework/Communications/Cache/GridAssetClient.cs
>    trunk/OpenSim/Framework/Communications/Capabilities/Caps.cs
>    trunk/OpenSim/Grid/AssetServer/RestService.cs
>    trunk/OpenSim/Region/ClientStack/LindenUDP/LLClientView.cs
>    trunk/OpenSim/Region/ClientStack/LindenUDP/LLFileTransfer.cs
>    trunk/OpenSim/Region/ClientStack/LindenUDP/LLImageManager.cs
>    trunk/OpenSim/Region/Communications/Hypergrid/HGGridServices.cs
>    trunk/OpenSim/Region/Environment/Modules/Agent/AssetTransaction/AgentAssetsTransactions.cs
>    trunk/OpenSim/Region/Environment/Modules/Agent/AssetTransaction/AssetXferUploader.cs
>    trunk/OpenSim/Region/Environment/Modules/Agent/TextureSender/TextureSender.cs
>    trunk/OpenSim/Region/Environment/Modules/Avatar/Inventory/Archiver/InventoryArchiveReadRequest.cs
>    trunk/OpenSim/Region/Environment/Modules/Scripting/DynamicTexture/DynamicTextureModule.cs
>    trunk/OpenSim/Region/Environment/Modules/World/Archiver/ArchiveReadRequest.cs
>    trunk/OpenSim/Region/Environment/Modules/World/Archiver/AssetsArchiver.cs
>    trunk/OpenSim/Region/Environment/Modules/World/Archiver/AssetsDearchiver.cs
>    trunk/OpenSim/Region/Environment/Modules/World/Estate/EstateTerrainXferHandler.cs
>    trunk/OpenSim/Region/Environment/Scenes/Hypergrid/HGAssetMapper.cs
>    trunk/OpenSim/Region/Environment/Scenes/Scene.Inventory.cs
>    trunk/OpenSim/Region/Environment/Scenes/Scene.cs
>    trunk/OpenSim/Region/ScriptEngine/Shared/Api/Implementation/OSSL_Api.cs


Ok, I'm reading through this patch, and it makes everything far more
confusing.  Now we are flattening and unflattening between 1 and 2
objects when we touch xml and the database.

We need to either decide the Metadata is really seperate, in which case
it's a 2nd level entity in the database and the xml (i.e. new table and
nested xml), or it isn't, and we got back.  This midway point is madness.

1 object per database table please.  Anything else is insanity for a
datamodel.

	-Sean

-- 
Sean Dague / Neas Bade
sdague at gmail.com
http://dague.net


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 252 bytes
Desc: OpenPGP digital signature
URL: <http://opensimulator.org/pipermail/opensim-dev/attachments/20090204/6ef1e212/attachment-0001.pgp>