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

Cheolsoo Park commented on PIG-3015:
------------------------------------

Hi Joseph,

Thanks for the update. I support what you're proposing. I appreciate your 
effort to clean up the code. Just to be clear, I have the following questions:
{quote}
Test pig scripts will be kept in discrete files, with parameters as file names. 
I'll modify the test runner to set the runtime parameters correctly.
{quote}
Ideally, all Pig unit test should be written this way. Currently, Pig queries 
are hard-coded in the test code, which is not very nice. But changing it is 
going to be a long-term effort. Your changes for this jira will be isolated in 
{{TestAvroStorage}}, won't they? If not, can you please provide more detail? I 
am just trying to understand the scope of your proposal.
{quote}
I'm thinking about modifying the build process to compile human readable files 
(in JSON format) into avro files before running the tests.
{quote}
This will be fully automated in the current framework (ant + junit), so I can 
run {{ant test -Dtestcase=TestAvroStorage}} to run unit test cases, right? One 
exception for this might be a test case for corrupted  Avro files I guess.

Thanks!
                
> Rewrite of AvroStorage
> ----------------------
>
>                 Key: PIG-3015
>                 URL: https://issues.apache.org/jira/browse/PIG-3015
>             Project: Pig
>          Issue Type: Improvement
>          Components: piggybank
>            Reporter: Joseph Adler
>            Assignee: Joseph Adler
>
> The current AvroStorage implementation has a lot of issues: it requires old 
> versions of Avro, it copies data much more than needed, and it's verbose and 
> complicated. (One pet peeve of mine is that old versions of Avro don't 
> support Snappy compression.)
> I rewrote AvroStorage from scratch to fix these issues. In early tests, the 
> new implementation is significantly faster, and the code is a lot simpler. 
> Rewriting AvroStorage also enabled me to implement support for Trevni.
> I'm opening this ticket to facilitate discussion while I figure out the best 
> way to contribute the changes back to Apache.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to