---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11355/
---
(Updated June 4, 2013, 9:50 p.m.)
Review request for pig and Rohini
On June 3, 2013, 1:20 p.m., Rohini Palaniswamy wrote:
http://svn.apache.org/repos/asf/pig/trunk/contrib/piggybank/java/src/test/java/org/apache/pig/piggybank/test/storage/avro/TestAvroStorage.java,
lines 629-636
https://reviews.apache.org/r/11355/diff/2/?file=295976#file295976line629
On June 2, 2013, 8:55 p.m., Cheolsoo Park wrote:
Hi Viraj,
I have a couple of comments:
- 5k records seems unnecessary for a unit test case. You need just a few
records to verify your fix, don't you?
- In you test case, can't you use mock.Storage instead of PigStorage? Then,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11355/
---
(Updated June 4, 2013, 11:23 p.m.)
Review request for pig and Rohini
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11355/#review21455
---
Ship it!
Ship It!
- Rohini Palaniswamy
On June 4, 2013, 11:23
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11355/#review21315
---
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11355/#review21304
---
Hi Viraj,
I have a couple of comments:
- 5k records seems
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11355/
---
(Updated May 30, 2013, 2:29 a.m.)
Review request for pig and Rohini