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

Ashutosh Chauhan commented on PIG-2103:
---------------------------------------

Although this approach also work, I wonder if it would have been better that we 
make RegExLoader take in a regex as a ctor argument and use it to compile 
pattern. That way Regex loader can work with any regular expression and then 
there wouldn't be a need for a different loader for every regex. In this patch, 
you already have three different loaders which just differ on the regular 
expression they are working on.

> Support for Squid logs parsing/loading (Loader UDF)
> ---------------------------------------------------
>
>                 Key: PIG-2103
>                 URL: https://issues.apache.org/jira/browse/PIG-2103
>             Project: Pig
>          Issue Type: New Feature
>            Reporter: Julian Gutierrez Oschmann
>            Priority: Minor
>         Attachments: squid-fixed.patch, squid.patch
>
>
> As proposed in the development list, LoadFunc UDF for parsing/loading of 
> Squid logs (default, common, squidmime and combined log levels)

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to