pig-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Taku Miyakawa" <miyakawa.t...@gmail.com>
Subject Re: Review Request: PIG-3215 [piggybank] Add LTSVLoader to load LTSV files
Date Thu, 07 Mar 2013 23:31:40 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/9685/
-----------------------------------------------------------

(Updated March 7, 2013, 11:31 p.m.)


Review request for pig.


Changes
-------

Outputs skipped labels to the task log at the first occurrence.


Description
-------

This is a review board for  https://issues.apache.org/jira/browse/PIG-3215

The patch adds LTSVLoader function and its test class.


This addresses bug PIG-3215.
    https://issues.apache.org/jira/browse/PIG-3215


Diffs (updated)
-----

  contrib/piggybank/java/src/main/java/org/apache/pig/piggybank/storage/LTSVLoader.java PRE-CREATION

  contrib/piggybank/java/src/test/java/org/apache/pig/piggybank/test/storage/TestLTSVLoader.java
PRE-CREATION 

Diff: https://reviews.apache.org/r/9685/diff/


Testing
-------

ant compile-test
cd contrib/piggybank/java/
ant -Dtestcase=TestLTSVLoader test


Thanks,

Taku Miyakawa


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message