Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Plain View
Pig, mail # dev - Review Request: PIG-3215 [piggybank] Add LTSVLoader to load LTSV files


+
Taku Miyakawa 2013-02-28, 22:53
+
Jonathan Coveney 2013-03-01, 14:54
+
Cheolsoo Park 2013-03-04, 01:46
+
Taku Miyakawa 2013-03-03, 06:35
+
Jonathan Coveney 2013-03-04, 15:05
+
Taku Miyakawa 2013-03-05, 23:10
+
Jonathan Coveney 2013-03-19, 16:45
+
Taku Miyakawa 2013-03-05, 23:38
+
Taku Miyakawa 2013-03-05, 23:42
+
Taku Miyakawa 2013-03-07, 22:46
Copy link to this message
-
Re: Review Request: PIG-3215 [piggybank] Add LTSVLoader to load LTSV files
Taku Miyakawa 2013-03-07, 23:31

-----------------------------------------------------------
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

+
Taku Miyakawa 2013-03-09, 08:18