[xwiki-devs] XWikiAllGroup implicit behavior
Hi, As part of the Curriki development, since Curriki is growing big (more than 60000 users), the XWiki.XWikiAllGroup is not scalable and this can lead to bad things. I've proposed a patch to make XWiki.XWikiAllGroup implicit. We have tested it and this seems to work well (see http://jira.xwiki.org/jira/browse/CURRIKI-738 for the patch). The only issue I see is the fact that once you have activated this, the counter of number of users for the group all group in the administration and the ability to edit the group makes no sense. Finally the adding the the AllGroup in registration has not yet been deactivated which would be necessary to make the registration not be impacted by the number of users. The Curriki team would like to see this patch make it in the XWiki Core. WDYT ? Ludovic -- Ludovic Dubost Blog: http://blog.ludovic.org/ XWiki: http://www.xwiki.com Skype: ldubost GTalk: ldubost
What sense does it make to edit the XWikiAllGroup? paul Le 18-mai-09 à 20:05, Ludovic Dubost a écrit :
As part of the Curriki development, since Curriki is growing big (more than 60000 users), the XWiki.XWikiAllGroup is not scalable and this can lead to bad things.
I've proposed a patch to make XWiki.XWikiAllGroup implicit. We have tested it and this seems to work well (see http://jira.xwiki.org/jira/browse/CURRIKI-738 for the patch). The only issue I see is the fact that once you have activated this, the counter of number of users for the group all group in the administration and the ability to edit the group makes no sense. Finally the adding the the AllGroup in registration has not yet been deactivated which would be necessary to make the registration not be impacted by the number of users.
The Curriki team would like to see this patch make it in the XWiki Core.
None.. But we need to remove that feature from the groups management Ludovic Paul Libbrecht a écrit :
What sense does it make to edit the XWikiAllGroup?
paul
Le 18-mai-09 à 20:05, Ludovic Dubost a écrit :
As part of the Curriki development, since Curriki is growing big (more than 60000 users), the XWiki.XWikiAllGroup is not scalable and this can lead to bad things.
I've proposed a patch to make XWiki.XWikiAllGroup implicit. We have tested it and this seems to work well (see http://jira.xwiki.org/jira/browse/CURRIKI-738 for the patch). The only issue I see is the fact that once you have activated this, the counter of number of users for the group all group in the administration and the ability to edit the group makes no sense. Finally the adding the the AllGroup in registration has not yet been deactivated which would be necessary to make the registration not be impacted by the number of users.
The Curriki team would like to see this patch make it in the XWiki Core.
------------------------------------------------------------------------
_______________________________________________ devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs
-- Ludovic Dubost Blog: http://blog.ludovic.org/ XWiki: http://www.xwiki.com Skype: ldubost GTalk: ldubost
Ludovic Dubost wrote:
Hi,
As part of the Curriki development, since Curriki is growing big (more than 60000 users), the XWiki.XWikiAllGroup is not scalable and this can lead to bad things.
I've proposed a patch to make XWiki.XWikiAllGroup implicit. We have tested it and this seems to work well (see http://jira.xwiki.org/jira/browse/CURRIKI-738 for the patch). The only issue I see is the fact that once you have activated this, the counter of number of users for the group all group in the administration and the ability to edit the group makes no sense. Finally the adding the the AllGroup in registration has not yet been deactivated which would be necessary to make the registration not be impacted by the number of users.
The Curriki team would like to see this patch make it in the XWiki Core.
WDYT ?
+1, see http://jira.xwiki.org/jira/browse/XWIKI-2116 But this is a bit too dangerous, so it should be applied starting with 2.0 the earliest. -- Sergiu Dumitriu http://purl.org/net/sergiu/
On Mon, May 18, 2009 at 20:05, Ludovic Dubost <[email protected]> wrote:
Hi,
As part of the Curriki development, since Curriki is growing big (more than 60000 users), the XWiki.XWikiAllGroup is not scalable and this can lead to bad things.
I've proposed a patch to make XWiki.XWikiAllGroup implicit. We have tested it and this seems to work well (see http://jira.xwiki.org/jira/browse/CURRIKI-738 for the patch). The only issue I see is the fact that once you have activated this, the counter of number of users for the group all group in the administration and the ability to edit the group makes no sense. Finally the adding the the AllGroup in registration has not yet been deactivated which would be necessary to make the registration not be impacted by the number of users.
The Curriki team would like to see this patch make it in the XWiki Core.
Yes we need http://jira.xwiki.org/jira/browse/XWIKI-2116. But the patch in http://jira.xwiki.org/jira/browse/CURRIKI-738 is too small IMO, you should add support for this in getAllMatchedMembersNamesForGroup also by returning all the users when the group is XWiki.XWikiAllGroup. That way the admin UI would be right and anyway if we want to make XWiki.XWikiAllGroup a "virtual group" we should do it fully and not half of the APIs.
WDYT ?
Ludovic
-- Ludovic Dubost Blog: http://blog.ludovic.org/ XWiki: http://www.xwiki.com Skype: ldubost GTalk: ldubost
_______________________________________________ devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs
-- Thomas Mortagne
participants (5)
-
Ludovic Dubost -
Ludovic Dubost -
Paul Libbrecht -
Sergiu Dumitriu -
Thomas Mortagne