Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.4.4, 1.5.0, 1.6.0
    • Component/s: None
    • Labels:
      None

      Description

      I looked at the proxy/README today and some things were unclear to me. When I compile Accumulo from the top level, isn't the proxy submodule included? Why would it need to be built separately? In order to build the proxy module from within the proxy directory, I needed to run 'mvn install' at the top level because of a dependency. The proxy.properties file which needs to be edited is not the one in the build directory, it's the one in the installation directory. Also, the 'accumulo proxy' command should be run from the installation directory.

      I also got a chance to look at the proxy/python/README file. Perhaps it's just that I know very little about python but "PYTHONPATH=path/to/generated/api:path/to/thrift/libs python TestClient.py" does not look right. Should there be a ';' before the python command? Will python people know what the 'path/to/generated/api' and 'path/to/thrift/libs' are?

      My last note - does thrift really need to be installed?

        Activity

        Hide
        elserj Josh Elser added a comment -

        My last note - does thrift really need to be installed?

        IIRC, yes. You would need the language specific library for thrift installed to be able to use said thrift client code in that language (you need the python thrift library installed to use the python with the proxy)

        Show
        elserj Josh Elser added a comment - My last note - does thrift really need to be installed? IIRC, yes. You would need the language specific library for thrift installed to be able to use said thrift client code in that language (you need the python thrift library installed to use the python with the proxy)
        Hide
        medined David Medinets added a comment -

        SVN#1478618. I updated the proxy/README file but feel it should be reviewed before this ticket is resolved. Can someone please take ownership of this ticket?

        Show
        medined David Medinets added a comment - SVN#1478618. I updated the proxy/README file but feel it should be reviewed before this ticket is resolved. Can someone please take ownership of this ticket?
        Hide
        elserj Josh Elser added a comment -

        Can someone please take ownership of this ticket?

        got yer back.

        Show
        elserj Josh Elser added a comment - Can someone please take ownership of this ticket? got yer back.
        Hide
        elserj Josh Elser added a comment -

        Pulled the changes back into 1.5, reworded a few things, and merged back into trunk. Thanks for the help, David Medinets

        Show
        elserj Josh Elser added a comment - Pulled the changes back into 1.5, reworded a few things, and merged back into trunk. Thanks for the help, David Medinets
        Hide
        medined David Medinets added a comment -

        one tiny change ... see resolution note.

        Show
        medined David Medinets added a comment - one tiny change ... see resolution note.
        Hide
        medined David Medinets added a comment -

        SVN#1478631. Add $ACCUMULO_HOME in front of proxy/proxy.properties so the current directory is irrelevant.

        Show
        medined David Medinets added a comment - SVN#1478631. Add $ACCUMULO_HOME in front of proxy/proxy.properties so the current directory is irrelevant.
        Hide
        elserj Josh Elser added a comment -

        David Medinets is there a reason you didn't make your change in 1.5? I thought the point of this was to make (more) correct documentation for the 1.5 release.

        Show
        elserj Josh Elser added a comment - David Medinets is there a reason you didn't make your change in 1.5? I thought the point of this was to make (more) correct documentation for the 1.5 release.
        Hide
        medined David Medinets added a comment -

        The only excuse is that it was late at night. Sorry.

        Show
        medined David Medinets added a comment - The only excuse is that it was late at night. Sorry.
        Hide
        elserj Josh Elser added a comment -

        No problemo. I just pulled back the changes to 1.4 (forgot that the proxy is there now too!) and then made sure 1.5 is up-to-date too.

        Show
        elserj Josh Elser added a comment - No problemo. I just pulled back the changes to 1.4 (forgot that the proxy is there now too!) and then made sure 1.5 is up-to-date too.
        Hide
        hudson Hudson added a comment -

        Integrated in Accumulo-1.5-Hadoop-2.0 #100 (See https://builds.apache.org/job/Accumulo-1.5-Hadoop-2.0/100/)
        ACCUMULO-1375 Pull the changes from 1.4 (which came from 1.6) to 1.5 (Revision 1479124)
        ACCUMULO-1375 Some more cleanup on the proxy/README (Revision 1478625)

        Result = SUCCESS
        elserj :
        Files :

        • /accumulo/branches/1.5/proxy/README

        elserj :
        Files :

        • /accumulo/branches/1.5/proxy/README
        Show
        hudson Hudson added a comment - Integrated in Accumulo-1.5-Hadoop-2.0 #100 (See https://builds.apache.org/job/Accumulo-1.5-Hadoop-2.0/100/ ) ACCUMULO-1375 Pull the changes from 1.4 (which came from 1.6) to 1.5 (Revision 1479124) ACCUMULO-1375 Some more cleanup on the proxy/README (Revision 1478625) Result = SUCCESS elserj : Files : /accumulo/branches/1.5/proxy/README elserj : Files : /accumulo/branches/1.5/proxy/README
        Hide
        hudson Hudson added a comment -

        Integrated in Accumulo-Trunk-Hadoop-2.0 #209 (See https://builds.apache.org/job/Accumulo-Trunk-Hadoop-2.0/209/)
        ACCUMULO-1375: Update README files in proxy module. (Revision 1478631)
        ACCUMULO-1375 Merge proxy/README changes from 1.5 up to trunk (Revision 1478627)
        ACCUMULO-1375: Update README files in proxy module. (Revision 1478618)

        Result = SUCCESS
        medined :
        Files :

        • /accumulo/trunk/proxy/README

        elserj :
        Files :

        • /accumulo/trunk
        • /accumulo/trunk/proxy/README

        medined :
        Files :

        • /accumulo/trunk/proxy/README
        Show
        hudson Hudson added a comment - Integrated in Accumulo-Trunk-Hadoop-2.0 #209 (See https://builds.apache.org/job/Accumulo-Trunk-Hadoop-2.0/209/ ) ACCUMULO-1375 : Update README files in proxy module. (Revision 1478631) ACCUMULO-1375 Merge proxy/README changes from 1.5 up to trunk (Revision 1478627) ACCUMULO-1375 : Update README files in proxy module. (Revision 1478618) Result = SUCCESS medined : Files : /accumulo/trunk/proxy/README elserj : Files : /accumulo/trunk /accumulo/trunk/proxy/README medined : Files : /accumulo/trunk/proxy/README
        Hide
        hudson Hudson added a comment -

        Integrated in Accumulo-Trunk #851 (See https://builds.apache.org/job/Accumulo-Trunk/851/)
        ACCUMULO-1375: Update README files in proxy module. (Revision 1478631)
        ACCUMULO-1375 Merge proxy/README changes from 1.5 up to trunk (Revision 1478627)
        ACCUMULO-1375: Update README files in proxy module. (Revision 1478618)

        Result = SUCCESS
        medined :
        Files :

        • /accumulo/trunk/proxy/README

        elserj :
        Files :

        • /accumulo/trunk
        • /accumulo/trunk/proxy/README

        medined :
        Files :

        • /accumulo/trunk/proxy/README
        Show
        hudson Hudson added a comment - Integrated in Accumulo-Trunk #851 (See https://builds.apache.org/job/Accumulo-Trunk/851/ ) ACCUMULO-1375 : Update README files in proxy module. (Revision 1478631) ACCUMULO-1375 Merge proxy/README changes from 1.5 up to trunk (Revision 1478627) ACCUMULO-1375 : Update README files in proxy module. (Revision 1478618) Result = SUCCESS medined : Files : /accumulo/trunk/proxy/README elserj : Files : /accumulo/trunk /accumulo/trunk/proxy/README medined : Files : /accumulo/trunk/proxy/README
        Hide
        hudson Hudson added a comment -

        Integrated in Accumulo-1.5 #98 (See https://builds.apache.org/job/Accumulo-1.5/98/)
        ACCUMULO-1375 Pull the changes from 1.4 (which came from 1.6) to 1.5 (Revision 1479124)
        ACCUMULO-1375 Some more cleanup on the proxy/README (Revision 1478625)

        Result = SUCCESS
        elserj :
        Files :

        • /accumulo/branches/1.5/proxy/README

        elserj :
        Files :

        • /accumulo/branches/1.5/proxy/README
        Show
        hudson Hudson added a comment - Integrated in Accumulo-1.5 #98 (See https://builds.apache.org/job/Accumulo-1.5/98/ ) ACCUMULO-1375 Pull the changes from 1.4 (which came from 1.6) to 1.5 (Revision 1479124) ACCUMULO-1375 Some more cleanup on the proxy/README (Revision 1478625) Result = SUCCESS elserj : Files : /accumulo/branches/1.5/proxy/README elserj : Files : /accumulo/branches/1.5/proxy/README
        Hide
        hudson Hudson added a comment -

        Integrated in Accumulo-1.4.x #300 (See https://builds.apache.org/job/Accumulo-1.4.x/300/)
        ACCUMULO-1375 Pull back the final proxy/README from trunk to 1.4 (Revision 1479113)

        Result = SUCCESS
        elserj :
        Files :

        • /accumulo/branches/1.4/src/proxy/README
        Show
        hudson Hudson added a comment - Integrated in Accumulo-1.4.x #300 (See https://builds.apache.org/job/Accumulo-1.4.x/300/ ) ACCUMULO-1375 Pull back the final proxy/README from trunk to 1.4 (Revision 1479113) Result = SUCCESS elserj : Files : /accumulo/branches/1.4/src/proxy/README

          People

          • Assignee:
            medined David Medinets
            Reporter:
            medined David Medinets
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - Not Specified
              Not Specified
              Remaining:
              Remaining Estimate - 0h
              0h
              Logged:
              Time Spent - 0.5h
              0.5h

                Development