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

Niels Basjes commented on DRILL-3423:
-------------------------------------

I'll have a closer look tonight but my first question is why your code 
explicitly names many of the possible fields. In none of the plugins/udfs I've 
written so far (PIG, Hive, Java) do I assume anything about the output fields 
of the parser.
Why do you do that in this situation?
In addition: The "casts" the parser exposes specify if the field can be a 
string, long or double. No need to hardcode that part too.

> Add New HTTPD format plugin
> ---------------------------
>
>                 Key: DRILL-3423
>                 URL: https://issues.apache.org/jira/browse/DRILL-3423
>             Project: Apache Drill
>          Issue Type: New Feature
>          Components: Storage - Other
>            Reporter: Jacques Nadeau
>            Assignee: Jacques Nadeau
>             Fix For: 1.2.0
>
>
> Add an HTTPD logparser based format plugin.  The author has been kind enough 
> to move the logparser project to be released under the Apache License.  Can 
> find it here:
> <dependency>
>     <groupId>nl.basjes.parse.httpdlog</groupId>
>     <artifactId>httpdlog-parser</artifactId>
>     <version>2.0</version>
> </dependency>
>  



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to