Need to look into making Pig support Accumulo
Work on this is finished so I am resolving this issue.
Looks great, thanks!
SVN revision number 1339260. Sorry all.
When I forget to put the ticket number in the commit message, I usually just add comment to the ticket pointing to the svn revision number.
Jason, I just noticed the pig classes and tests aren't following our code style (mainly a 2-space indent, no tabs). The details of the code style as well as how to import it into Eclipse are described here: http://accumulo.apache.org/source.html
I think the log message didn't have the ticket number, ACCUMULO-142. It is possible to change the log message after the fact, but JIRA won't be able to pick up the change and show it on the ticket.
Just noticed the commits didn't show up here in JIRA. Weird, because they show up in the SVN repo: https://svn.apache.org/repos/asf/accumulo/contrib/pig/trunk/src/main/java/org/apache/accumulo/pig/
I just committed test code and cleaned up the Storage classes. I think you can close out now.
A heads up: I restructured the contrib so that the modules can be versioned separately and so accumulo-pig can rely on version 1.4.0 instead of 1.5.0-SNAPSHOT. The new location is
This code is functional, but there are no test cases so I would say not yet. I will try to commit some soon.
can this ticket be closed?
Jason, I checked in the patch as a module of a new accumulo-contrib project located at contrib/trunk/. Feel free to test and add things to this module.
functional accumulo/pig Storage implementation.