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

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

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

    https://github.com/apache/metron/pull/785#discussion_r159122619
  
    --- Diff: metron-platform/metron-parsers/3rdPartyParser.md ---
    @@ -0,0 +1,306 @@
    +# Custom Metron Parsers
    --- End diff --
    
    Not sure what the state of this is, but if we're going to merge this as a 
stop-gap to 777, could you add the license header here?
    
    ```
    <!--
    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.
    -->
    ```


> As a stopgap prior to METRON-777, add more simplistic sideloading of custom 
> Parsers
> -----------------------------------------------------------------------------------
>
>                 Key: METRON-1230
>                 URL: https://issues.apache.org/jira/browse/METRON-1230
>             Project: Metron
>          Issue Type: Improvement
>            Reporter: Casey Stella
>
> Until we get 777 in, it'd be nice to have a simple ability using the normal 
> storm functionality to enable users to provide custom parsers without forking 
> Metron.  This should be done via simply creating a jar with their code (and 
> bundled dependencies) and have it picked up and available to the REST Service 
> (and consequently the management UI) as well as the start_parser_topology.sh 
> command.
> This should be minimal movement as we have a more robust solution coming with 
> METRON-777.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to