You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@jackrabbit.apache.org by "fabrizio giustina (JIRA)" <ji...@apache.org> on 2010/02/21 15:35:27 UTC
[jira] Created: (JCR-2504) Allow indexingConfiguration to be loaded
from the classpath
Allow indexingConfiguration to be loaded from the classpath
-----------------------------------------------------------
Key: JCR-2504
URL: https://issues.apache.org/jira/browse/JCR-2504
Project: Jackrabbit Content Repository
Issue Type: Improvement
Components: indexing
Affects Versions: 2.0.0, 1.6.1
Reporter: fabrizio giustina
Attachments: indexingConfiguration_classpath.diff
The "indexingConfiguration" attribute in the SearchIndex configuration (http://wiki.apache.org/jackrabbit/IndexingConfiguration) actually requires an absolute filesystem path.
It would be nice if SearchIndex would also accept a file available in the classpath... although you can use variables like ${wsp.home} or similar there are many scenarios where a classpath resource would help (for example when creating a new workspace the directory structure is automatically created by jackrabbit and doesn't need to be already available but the indexing configuration file does).
I am attaching a simple patch to SearchIndex that tries to load the file from the classpath if it has not been found. Since priority is given to the old behavior (file before classpath) so it's fully backward compatible.
Diff has been generated against trunk, it would be nice to have this patch also on the 2.0 branch.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (JCR-2504) Allow indexingConfiguration to be loaded
from the classpath
Posted by "fabrizio giustina (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/JCR-2504?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
fabrizio giustina updated JCR-2504:
-----------------------------------
Status: Patch Available (was: Open)
see attached indexingConfiguration_classpath.diff
> Allow indexingConfiguration to be loaded from the classpath
> -----------------------------------------------------------
>
> Key: JCR-2504
> URL: https://issues.apache.org/jira/browse/JCR-2504
> Project: Jackrabbit Content Repository
> Issue Type: Improvement
> Components: indexing
> Affects Versions: 2.0.0, 1.6.1
> Reporter: fabrizio giustina
> Attachments: indexingConfiguration_classpath.diff
>
>
> The "indexingConfiguration" attribute in the SearchIndex configuration (http://wiki.apache.org/jackrabbit/IndexingConfiguration) actually requires an absolute filesystem path.
> It would be nice if SearchIndex would also accept a file available in the classpath... although you can use variables like ${wsp.home} or similar there are many scenarios where a classpath resource would help (for example when creating a new workspace the directory structure is automatically created by jackrabbit and doesn't need to be already available but the indexing configuration file does).
> I am attaching a simple patch to SearchIndex that tries to load the file from the classpath if it has not been found. Since priority is given to the old behavior (file before classpath) so it's fully backward compatible.
> Diff has been generated against trunk, it would be nice to have this patch also on the 2.0 branch.
>
>
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (JCR-2504) Allow indexingConfiguration to be loaded
from the classpath
Posted by "fabrizio giustina (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/JCR-2504?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
fabrizio giustina updated JCR-2504:
-----------------------------------
Attachment: indexingConfiguration_classpath.diff
SearchIndex patch
> Allow indexingConfiguration to be loaded from the classpath
> -----------------------------------------------------------
>
> Key: JCR-2504
> URL: https://issues.apache.org/jira/browse/JCR-2504
> Project: Jackrabbit Content Repository
> Issue Type: Improvement
> Components: indexing
> Affects Versions: 1.6.1, 2.0.0
> Reporter: fabrizio giustina
> Attachments: indexingConfiguration_classpath.diff
>
>
> The "indexingConfiguration" attribute in the SearchIndex configuration (http://wiki.apache.org/jackrabbit/IndexingConfiguration) actually requires an absolute filesystem path.
> It would be nice if SearchIndex would also accept a file available in the classpath... although you can use variables like ${wsp.home} or similar there are many scenarios where a classpath resource would help (for example when creating a new workspace the directory structure is automatically created by jackrabbit and doesn't need to be already available but the indexing configuration file does).
> I am attaching a simple patch to SearchIndex that tries to load the file from the classpath if it has not been found. Since priority is given to the old behavior (file before classpath) so it's fully backward compatible.
> Diff has been generated against trunk, it would be nice to have this patch also on the 2.0 branch.
>
>
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (JCR-2504) Allow indexingConfiguration to be loaded
from the classpath
Posted by "Jukka Zitting (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/JCR-2504?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Jukka Zitting updated JCR-2504:
-------------------------------
Resolution: Fixed
Assignee: Jukka Zitting
Status: Resolved (was: Patch Available)
Committed to trunk in revision 932319 and merged to the 2.1 branch in revision 932348. Thanks!
> Allow indexingConfiguration to be loaded from the classpath
> -----------------------------------------------------------
>
> Key: JCR-2504
> URL: https://issues.apache.org/jira/browse/JCR-2504
> Project: Jackrabbit Content Repository
> Issue Type: Improvement
> Components: indexing
> Affects Versions: 1.6.1, 2.0.0
> Reporter: fabrizio giustina
> Assignee: Jukka Zitting
> Fix For: 2.1.0
>
> Attachments: indexingConfiguration_classpath.diff
>
>
> The "indexingConfiguration" attribute in the SearchIndex configuration (http://wiki.apache.org/jackrabbit/IndexingConfiguration) actually requires an absolute filesystem path.
> It would be nice if SearchIndex would also accept a file available in the classpath... although you can use variables like ${wsp.home} or similar there are many scenarios where a classpath resource would help (for example when creating a new workspace the directory structure is automatically created by jackrabbit and doesn't need to be already available but the indexing configuration file does).
> I am attaching a simple patch to SearchIndex that tries to load the file from the classpath if it has not been found. Since priority is given to the old behavior (file before classpath) so it's fully backward compatible.
> Diff has been generated against trunk, it would be nice to have this patch also on the 2.0 branch.
>
>
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (JCR-2504) Allow indexingConfiguration to be loaded
from the classpath
Posted by "fabrizio giustina (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/JCR-2504?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
fabrizio giustina updated JCR-2504:
-----------------------------------
Fix Version/s: 2.1.0
tagging with 2.1.0 as fix version, to not forget the patch for the 2.1 release
> Allow indexingConfiguration to be loaded from the classpath
> -----------------------------------------------------------
>
> Key: JCR-2504
> URL: https://issues.apache.org/jira/browse/JCR-2504
> Project: Jackrabbit Content Repository
> Issue Type: Improvement
> Components: indexing
> Affects Versions: 1.6.1, 2.0.0
> Reporter: fabrizio giustina
> Fix For: 2.1.0
>
> Attachments: indexingConfiguration_classpath.diff
>
>
> The "indexingConfiguration" attribute in the SearchIndex configuration (http://wiki.apache.org/jackrabbit/IndexingConfiguration) actually requires an absolute filesystem path.
> It would be nice if SearchIndex would also accept a file available in the classpath... although you can use variables like ${wsp.home} or similar there are many scenarios where a classpath resource would help (for example when creating a new workspace the directory structure is automatically created by jackrabbit and doesn't need to be already available but the indexing configuration file does).
> I am attaching a simple patch to SearchIndex that tries to load the file from the classpath if it has not been found. Since priority is given to the old behavior (file before classpath) so it's fully backward compatible.
> Diff has been generated against trunk, it would be nice to have this patch also on the 2.0 branch.
>
>
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.