19 May
2009
19 May
'09
10:02 a.m.
On Mon, May 18, 2009 at 20:05, Ludovic Dubost <ludovic(a)xwiki.com> 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
devs(a)xwiki.org
http://lists.xwiki.org/mailman/listinfo/devs
--
Thomas Mortagne