Details

    • Type: Task Task
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.5.0
    • Component/s: None
    • Labels:

      Description

      Need to look into making Pig support Accumulo

        Activity

        Hide
        Jason Trost added a comment -

        Work on this is finished so I am resolving this issue.

        Show
        Jason Trost added a comment - Work on this is finished so I am resolving this issue.
        Hide
        Billie Rinaldi added a comment -

        Looks great, thanks!

        Show
        Billie Rinaldi added a comment - Looks great, thanks!
        Hide
        Jason Trost added a comment -

        code re-formatted...

        Show
        Jason Trost added a comment - code re-formatted...
        Hide
        Jason Trost added a comment -

        SVN revision number 1339260. Sorry all.

        Show
        Jason Trost added a comment - SVN revision number 1339260. Sorry all.
        Hide
        Keith Turner added a comment -

        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.

        Show
        Keith Turner added a comment - 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.
        Hide
        Billie Rinaldi added a comment -

        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

        Show
        Billie Rinaldi added a comment - 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
        Hide
        Billie Rinaldi added a comment -

        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.

        Show
        Billie Rinaldi added a comment - 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.
        Hide
        Jason Trost added a comment -

        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/

        Show
        Jason Trost added a comment - 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/
        Hide
        Jason Trost added a comment -

        I just committed test code and cleaned up the Storage classes. I think you can close out now.

        Show
        Jason Trost added a comment - I just committed test code and cleaned up the Storage classes. I think you can close out now.
        Hide
        Billie Rinaldi added a comment -

        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
        https://svn.apache.org/repos/asf/accumulo/contrib/pig/trunk/

        Show
        Billie Rinaldi added a comment - 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 https://svn.apache.org/repos/asf/accumulo/contrib/pig/trunk/
        Hide
        Jason Trost added a comment -

        This code is functional, but there are no test cases so I would say not yet. I will try to commit some soon.

        Show
        Jason Trost added a comment - This code is functional, but there are no test cases so I would say not yet. I will try to commit some soon.
        Hide
        Keith Turner added a comment -

        can this ticket be closed?

        Show
        Keith Turner added a comment - can this ticket be closed?
        Hide
        Billie Rinaldi added a comment -

        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.

        Show
        Billie Rinaldi added a comment - 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.
        Hide
        Jason Trost added a comment -

        functional accumulo/pig Storage implementation.

        Show
        Jason Trost added a comment - functional accumulo/pig Storage implementation.

          People

          • Assignee:
            Jason Trost
            Reporter:
            Keith Turner
          • Votes:
            2 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development