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
_______________________________________________
devs mailing list
[email protected]
http://lists.xwiki.org/mailman/listinfo/devs

Reply via email to