[ 
https://issues.apache.org/jira/browse/METRON-744?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15888867#comment-15888867
 ] 

ASF GitHub Bot commented on METRON-744:
---------------------------------------

Github user nickwallen commented on a diff in the pull request:

    https://github.com/apache/incubator-metron/pull/468#discussion_r103550096
  
    --- Diff: 
metron-platform/metron-common/src/main/java/org/apache/metron/common/utils/ClassloaderUtil.java
 ---
    @@ -0,0 +1,146 @@
    +/**
    + * Licensed to the Apache Software Foundation (ASF) under one
    + * or more contributor license agreements.  See the NOTICE file
    + * distributed with this work for additional information
    + * regarding copyright ownership.  The ASF licenses this file
    + * to you under the Apache License, Version 2.0 (the
    + * "License"); you may not use this file except in compliance
    + * with the License.  You may obtain a copy of the License at
    + *
    + *     http://www.apache.org/licenses/LICENSE-2.0
    + *
    + * Unless required by applicable law or agreed to in writing, software
    + * distributed under the License is distributed on an "AS IS" BASIS,
    + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
    + * See the License for the specific language governing permissions and
    + * limitations under the License.
    + */
    +package org.apache.metron.common.utils;
    +
    +import org.apache.accumulo.start.classloader.vfs.UniqueFileReplicator;
    +import org.apache.commons.logging.Log;
    +import org.apache.commons.vfs2.*;
    +import org.apache.commons.vfs2.cache.SoftRefFilesCache;
    +import org.apache.commons.vfs2.impl.DefaultFileSystemManager;
    +import org.apache.commons.vfs2.impl.FileContentInfoFilenameFactory;
    +import org.apache.commons.vfs2.impl.VFSClassLoader;
    +import org.apache.commons.vfs2.provider.FileReplicator;
    +import org.apache.commons.vfs2.provider.UriParser;
    +import org.apache.commons.vfs2.provider.VfsComponent;
    +import org.apache.commons.vfs2.provider.VfsComponentContext;
    +import org.apache.commons.vfs2.provider.hdfs.HdfsFileProvider;
    +import org.slf4j.Logger;
    +import org.slf4j.LoggerFactory;
    +
    +import java.io.File;
    +import java.io.IOException;
    +import java.util.*;
    +
    +public class ClassloaderUtil {
    --- End diff --
    
    Javadoc would be awesome.  I don't know how I would use these functions.  
Seems like you put them here because you think there are opportunities for 
reuse.
    
    Also, whitespace and formatting is a little screwy in this file.


> Allow Stellar functions to be loaded from HDFS
> ----------------------------------------------
>
>                 Key: METRON-744
>                 URL: https://issues.apache.org/jira/browse/METRON-744
>             Project: Metron
>          Issue Type: New Feature
>            Reporter: Casey Stella
>
> The benefit of Stellar is that adding new functionality is as simple as 
> providing a Jar.  This enables people who want to integrate with Metron to 
> easy add enrichments or other functionality.  The snag currently with this is 
> that we provide a single jar, so all stellar functions that we have available 
> must be dependencies of the main jar that drives the topology plus what local 
> directories we can configure via the storm configs.  This makes the process 
> of adding 3rd party jars not as easy as it could be.
> Adjust the the following to additionally load classes from a location in HDFS 
> /apps/metron/stellar using something like accumulo ( 
> https://accumulo.apache.org/blog/2014/05/03/accumulo-classloader.html)
> * Profiler topology
> * Parser topology
> * Enrichment topology
> * Enrichment Flat file loader
> * Enrichment MR loader



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to