[
http://jira.magnolia-cms.com/browse/MAGNOLIA-3064?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=26451#action_26451
]
Hudson CI server commented on MAGNOLIA-3064:
--------------------------------------------
Integrated in !http://hudson.magnolia-cms.com/nocacheImages/16x16/blue.gif!
[magnolia_main-branch-utf8
#2|http://hudson.magnolia-cms.com/job/magnolia_main-branch-utf8/2/]
Merged revisions 31745-31749,31760 via svnmerge from
http://svn.magnolia-cms.com/svn/community/magnolia/trunk
........
r31745 | gjoseph | 2010-02-10 18:28:51 +0100 (Wed, 10 Feb 2010) | 1 line
copied getCurrentTestMethodName from AbstractImagingTest
........
r31746 | gjoseph | 2010-02-10 18:30:08 +0100 (Wed, 10 Feb 2010) | 1 line
extracted AbstractFreemarkerTestCase
........
r31747 | gjoseph | 2010-02-10 18:31:39 +0100 (Wed, 10 Feb 2010) | 1 line
comment
........
r31748 | gjoseph | 2010-02-10 18:39:43 +0100 (Wed, 10 Feb 2010) | 1 line
extracted AbsractFreemarkerTestCase
........
r31749 | gjoseph | 2010-02-10 18:47:42 +0100 (Wed, 10 Feb 2010) | 1 line
renamed fmTemplateLoader to fmConfig - name was set before the
FreemarkerConfig class was introduced
........
r31760 | fgiust | 2010-02-10 23:05:01 +0100 (Wed, 10 Feb 2010) | 1 line
MAGNOLIA-3064 check for an existing repository mapping before adding a new
workspace for a module
........
> check for an existing repository mapping before adding a new workspace for a
> module
> ------------------------------------------------------------------------------------
>
> Key: MAGNOLIA-3064
> URL: http://jira.magnolia-cms.com/browse/MAGNOLIA-3064
> Project: Magnolia
> Issue Type: Improvement
> Reporter: Fabrizio Giustina
> Assignee: Fabrizio Giustina
> Fix For: 4.3
>
>
> This change is needed in order to be able to setup a separate clustered
> repository for specific workspaces.
> Although we have the repository mapping in place, which converts between a
> logical workspace name to a repository/workspace name pair for jackrabbit,
> the jackrabbit repository name is specified also in each module descriptor.
> So if we have a module with a repository declared as:
> {code}
> <repositories>
> <repository>
> <name>magnolia</name>
> <workspaces>
> <workspace>commenting</workspace>
> </workspaces>
>
> <nodeTypeFile>/mgnl-nodetypes/commenting/magnolia-nodetypes.xml</nodeTypeFile>
> </repository>
> </repositories>
> {code}
> the only way to move this workspace to a shared repository is to patch the
> module descriptor, changing the repository name to a different value, e.g.
> "shared".
> A good solution would have been avoiding the repository name in the module
> descriptor at all, defaulting to Magnolia if there is no mapping already
> defined in repositories.xml (but it's too late for a similar change, module
> descriptors needs to be compatible with previous versions).
> The proposed solution is to fix ModuleManagerImpl.loadRepository() in order
> to look for an existing repo mapping using the workspace name. If the mapping
> is found just use such repository instead of creating a new one, whatever
> repository name is set in the module descriptor.
> For the previous scenario, we will look for a mapping for the "commenting"
> workspace. If found, we proceed using the jackrabbit repository name
> specified in the mapping. (side note: ModuleManagerImpl actually always
> creates an additional workspace with the same name of the repository - this
> should not be done, the actual jackrabbit repository name could be any
> user-invented value)
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.magnolia-cms.com/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
----------------------------------------------------------------
For list details see
http://www.magnolia-cms.com/home/community/mailing-lists.html
To unsubscribe, E-mail to: <[email protected]>
----------------------------------------------------------------
