[ 
https://issues.apache.org/jira/browse/PIG-501?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Daniel Dai updated PIG-501:
---------------------------

    Attachment: PIG_cygwin2.Patch

I changed the patch to improve the performance of PigStorage a little bit. I 
did a performance test on PigStorage on two files with '\r' and without '\r', 
each file consists of 1m lines, every line has two fields, average field size 
is 10. Here is the result:

Unix:
|| ||before patch||after patch||
|contain '\r'|36.554|36.812|
|not contain '\r'|35.497|35.241|

Windows:
|| ||before patch||after patch||
|contain '\r'|36.637|37.001|
|not contain '\r'|35.311|35.318|

The overhead seems to be ok.

For javacc issue I can keep track of javacc release plan. And currently we can 
point pig user to here if they do need cygwin.

> Make branches/types work under cygwin
> -------------------------------------
>
>                 Key: PIG-501
>                 URL: https://issues.apache.org/jira/browse/PIG-501
>             Project: Pig
>          Issue Type: Bug
>          Components: impl
>    Affects Versions: types_branch
>         Environment: cygwin
>            Reporter: Daniel Dai
>            Assignee: Daniel Dai
>             Fix For: types_branch
>
>         Attachments: javacc.jar, PIG_cygwin.Patch, PIG_cygwin2.Patch
>
>
> We've already make all unit tests pass under cygwin for trunk (See 
> [PIG-243|https://issues.apache.org/jira/browse/PIG-243]). We need to do the 
> same for branches/types. 

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

Reply via email to