Sun, 18 May 2008 00:21:58 -0400 http://bugs.horde.org/ticket/6109 optimized sql Share driver - flat attribute table - data splited per tables - split read support Mon, 14 Jan 2008 06:42:42 -0500 http://bugs.horde.org/ticket/6109#t41120 Genie DT to SQL conversion sql subselect script. With little modification it should work for all shares groups. Mon, 14 Jan 2008 06:44:22 -0500 http://bugs.horde.org/ticket/6109#t41121 Fix _getShareCriteria(). Seems to work with kronolith, nag, mnemo and genie. Mon, 14 Jan 2008 08:56:37 -0500 http://bugs.horde.org/ticket/6109#t41123 Update transition script works for kronolith, nag, mnemo and genie. Create tables, transfers data, and delete old datatree entries. Just replace the string genie with appropriate share name. Unfortunately uses subselect so it will probably work with mysql only. Mon, 14 Jan 2008 10:09:34 -0500 http://bugs.horde.org/ticket/6109#t41134 Make a local copy of listShares() to do a query less - do not query the db for ids and then select the data, but pick the data directly Mon, 14 Jan 2008 12:52:17 -0500 http://bugs.horde.org/ticket/6109#t41135 - Added __sleep and __wakeup methods - Add perm_extra columns to mark if the share has any additional user or group definition. So we avoid queries for shares that don't have them set. In a lot of occasions we need only one query to get the entire share list. Mon, 14 Jan 2008 13:55:22 -0500 http://bugs.horde.org/ticket/6109#t41139 Do not sort shares by name inside PHP as SQL can do it so. Now I finaly can list all 5.200 wish-list without timeout. With this I think is the last thing I can do without share api changes. Now I am just wainting mysql to use indexes for bitwise operators. Duck Tue, 15 Jan 2008 13:36:31 -0500 http://bugs.horde.org/ticket/6109#t41168 This is maybe a hack. But introducing an dedicated object with minimal data (id,name,owner) to work with share listing in panels etc lower the ram usage with 5.200 entries from 18M to 7M. Tried with indexed arrays and noticed that uses even 1M less. Sat, 19 Jan 2008 06:31:41 -0500 http://bugs.horde.org/ticket/6109#t41309 fix harcoded DT permissons and param_extra query Fri, 25 Jan 2008 14:37:52 -0500 http://bugs.horde.org/ticket/6109#t41496 Just to be clear, shareimport-genie.sql is replaced by horde.shares.genie.sql, right? Mon, 18 Feb 2008 22:16:04 -0500 http://bugs.horde.org/ticket/6109#t42564 > Just to be clear, shareimport-genie.sql is replaced by > horde.shares.genie.sql, right? yes Tue, 19 Feb 2008 05:38:23 -0500 http://bugs.horde.org/ticket/6109#t42598 I am uploading a new version that fixes a little bug in _getShareCriteria I discovered while trying to port it to Ansel. Horde_Share_Object_sqlmini is us just a hack, and can be completely omitted. Is a fact that all applications fetches the list only to get the share id, name and owner. All other actions are done on a single share object. Only Kronolith uses the results from listShares for permission checking. So it would be better to remove Horde_Share_Object_sqlmini and add a method to return an indexed array for share names/owners as we already discussed some time ago in the list. Tue, 19 Feb 2008 13:34:49 -0500 http://bugs.horde.org/ticket/6109#t42611 Should go into 3.2 as an experimental driver. Wed, 05 Mar 2008 19:24:01 -0500 http://bugs.horde.org/ticket/6109#t43431 I've committed the base driver code. Can you post what the config settings look like for this driver? Thu, 06 Mar 2008 23:32:05 -0500 http://bugs.horde.org/ticket/6109#t43523 > I've committed the base driver code. Can you post what the config > settings look like for this driver? No configuration options is needed. It uses the global horde sql configuration (with split write support). All you need is to create additional tables per very share group. Like is already attached here in horde.shares.genie.sql. You must just change the prefix by scope name and add attributes columns if needed. I am attaching another example for Kronolith. Structures for each application can be bundled with application sql structure. Fri, 07 Mar 2008 04:50:21 -0500 http://bugs.horde.org/ticket/6109#t43526 Okay, so we just need a 'sql' option with no extra params in the $conf['shares'] array? Fri, 07 Mar 2008 15:45:42 -0500 http://bugs.horde.org/ticket/6109#t43555 > Okay, so we just need a 'sql' option with no extra params in the > $conf['shares'] array? yes Sat, 08 Mar 2008 04:27:39 -0500 http://bugs.horde.org/ticket/6109#t43570 After framework update I noticed that I didn't update the code here. - Fix unsetting the permissions for guest/default/creator. - Don't avoid permission check with attributes. Sun, 06 Apr 2008 14:05:14 -0400 http://bugs.horde.org/ticket/6109#t44452 Committed along with making the code parse under PHP 4. Still need to remove the REPLACE INTO statement so it works in more than MySQL. Sun, 06 Apr 2008 22:14:42 -0400 http://bugs.horde.org/ticket/6109#t44466 Code is working for me on SQLite now as well as MySQL. Still to do: - sql scripts for Ingo, Kronolith (done), Mnemo, Nag, Turba - datatree-to-sql upgrade scripts for Ingo, Kronolith, Mnemo, Nag, Turba - conf.xml entry (done, just waiting for full functionality) - docs/UPGRADING entries I'm working at this, but help is certainly welcome. If anyone else wants to/can jump in here, please just add a note here so we don't overlap. Sat, 12 Apr 2008 22:59:35 -0400 http://bugs.horde.org/ticket/6109#t44608 table defs have been added for Kronolith; still need the upgrade scripts (both for creating the tables and migration). Sun, 13 Apr 2008 01:01:06 -0400 http://bugs.horde.org/ticket/6109#t44609 I'll try to work on what's required for Turba... Sun, 13 Apr 2008 13:18:58 -0400 http://bugs.horde.org/ticket/6109#t44613 Table defs are done for Turba - still need to work on the conversion script Sun, 13 Apr 2008 15:01:45 -0400 http://bugs.horde.org/ticket/6109#t44614 Table defs and table def upgrades are done for Nag and Mnemo. Mon, 14 Apr 2008 14:11:43 -0400 http://bugs.horde.org/ticket/6109#t44635 Ingo table defs are done. That leaves, to summarize: - datatree-to-sql upgrade scripts for Ingo, Kronolith, Mnemo, Nag, Turba - conf.xml entry (done, just waiting for full functionality) - docs/UPGRADING entries Mon, 14 Apr 2008 16:20:41 -0400 http://bugs.horde.org/ticket/6109#t44637 Turba migrations and UPGRADING is done (that should be it for Turba) So..to update the summary: > Ingo table defs are done. That leaves, to summarize: > > - datatree-to-sql upgrade scripts for Ingo, Kronolith, Mnemo, Nag > - conf.xml entry (done, just waiting for full functionality) > - docs/UPGRADING entries Ingo, Kronolith, Mnemo, Nag I'll start moving the migration script to the other apps and post as I complete them. Wed, 16 Apr 2008 18:21:40 -0400 http://bugs.horde.org/ticket/6109#t44685 Kronolith migration and UPGRADING done Wed, 16 Apr 2008 19:07:01 -0400 http://bugs.horde.org/ticket/6109#t44686 Ingo migration and UPGRADING done that leaves: >> - datatree-to-sql upgrade scripts for Mnemo, Nag >> - conf.xml entry (done, just waiting for full functionality) >> - docs/UPGRADING entries Mnemo, Nag Wed, 16 Apr 2008 22:34:29 -0400 http://bugs.horde.org/ticket/6109#t44689 Duck- can you explain what the check in Horde_Share_sql::_getShareCriteria against $conf['share']['no_sharing'] is for? It doesn't appear in the other backends, and I'm inclined to remove it here also unless there's a reason I'm missing. Thu, 17 Apr 2008 10:44:32 -0400 http://bugs.horde.org/ticket/6109#t44705 > Duck- > can you explain what the check in Horde_Share_sql::_getShareCriteria > against $conf['share']['no_sharing'] is for? It doesn't appear in the > other backends, and I'm inclined to remove it here also unless > there's a reason I'm missing. As I remember, even when shares are disabled, the default user's share is still created. Like the default user calendar. Performing the select joins the users/group table. But those two tables will be always empty. So I add a check to avoid the JOIN. Thu, 17 Apr 2008 11:27:01 -0400 http://bugs.horde.org/ticket/6109#t44706 > As I remember, even when shares are disabled, the default user's > share is still created. Like the default user calendar. Performing > the select joins the users/group table. But those two tables will be > always empty. So I add a check to avoid the JOIN. Shouldn't the check be the other way then? Right now it only joins in those tables if no_sharing is enabled? Actually I think it should be removed, since even if users aren't allowed to control who sees their shares, someone might create system shares that everyone sees. Thu, 17 Apr 2008 12:03:47 -0400 http://bugs.horde.org/ticket/6109#t44711 Looks like all we have left now is the conf.xml entry. Chuck, you said you have it done, right? Thu, 17 Apr 2008 15:36:50 -0400 http://bugs.horde.org/ticket/6109#t44713 No great shakes, but yup, it's done. Now we test :) Thu, 17 Apr 2008 16:05:29 -0400 http://bugs.horde.org/ticket/6109#t44714 Also added upgrade files to Genie as well... Sat, 19 Apr 2008 14:39:25 -0400 http://bugs.horde.org/ticket/6109#t44739 This is done and a pretty darn solid beta if I say so myself. Huge thanks to Duck for getting this started and to Michael R. for all the help testing and for writing the migration scripts. Tue, 29 Apr 2008 23:34:27 -0400 http://bugs.horde.org/ticket/6109#t44999