nutch-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "tcurran@approachingpi.com (JIRA)" <j...@apache.org>
Subject [jira] Commented: (NUTCH-764) Add support for vfsfile:// loading of plugins for JBoss
Date Tue, 10 Nov 2009 14:31:27 GMT

    [ https://issues.apache.org/jira/browse/NUTCH-764?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12775434#action_12775434
] 

tcurran@approachingpi.com commented on NUTCH-764:
-------------------------------------------------

I should have said, by default, it will always be vfsfile. Jboss 5.0+ uses this virtual file
system for any files access performed via the context class loaders.  For files outside the
deployment directory can still be loaded with file:.  It is just a cache and abstracts that
sometimes the file will be in memory or a different physical location during application runtime.

for your second question: yes, this does fix Nutch and it is then able to load all of the
plugins properly.

I understand your concerns and it was hasty of me to assume that this was a trivial issue
to implement.  I will spend some time investigating and see if there are any other aspects
of Nutch that are impacted by this protocol.



> Add support for vfsfile:// loading of plugins for JBoss
> -------------------------------------------------------
>
>                 Key: NUTCH-764
>                 URL: https://issues.apache.org/jira/browse/NUTCH-764
>             Project: Nutch
>          Issue Type: Improvement
>          Components: searcher
>    Affects Versions: 1.0.0
>         Environment: JBoss AS 5.1.0
>            Reporter: tcurran@approachingpi.com
>            Priority: Trivial
>
> In the file:
> /src/java/org/apache/nutch/plugin/PluginManifestParser.java
> There is a check to make sure that the plugin file location is a url formatted like "file://path/plugins".
> When deployed on Jboss, the file protocol will sometimes be: "vfsfile://path/plugins".
 The code with vfsfile can operate the same so I propose a change to the check to also allow
this protocol.  This would allow Nutch to be deployed on the newer versions of JBoss without
any modification.
> Here is a simple patch:
> Index: src/java/org/apache/nutch/plugin/PluginManifestParser.java
> ===================================================================
> --- src/java/org/apache/nutch/plugin/PluginManifestParser.java	Mon Nov 09 20:20:51 EST
2009
> +++ src/java/org/apache/nutch/plugin/PluginManifestParser.java	Mon Nov 09 20:20:51 EST
2009
> @@ -121,7 +121,8 @@
>        } else if (url == null) {
>          LOG.warn("Plugins: directory not found: " + name);
>          return null;
> -      } else if (!"file".equals(url.getProtocol())) {
> +      } else if (!"file".equals(url.getProtocol()) &&
> +        !"vfsfile".equals(url.getProtocol())) {
>          LOG.warn("Plugins: not a file: url. Can't load plugins from: " + url);
>          return null;
>        }

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message