Accumulo
  1. Accumulo
  2. ACCUMULO-442

Please add comments to src/trace/thift.sh

    Details

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

      Description

      I'm poking about the source code and found this script. It is probably used to generate Java files from a Thrift template? Can someone please add a comment or two to the file to indicate when it should be run. And what is is doing? Does Thrift normally install and call itself 'thrift0.6'?

        Activity

        David Medinets created issue -
        David Medinets made changes -
        Field Original Value New Value
        Labels docs
        jv made changes -
        Assignee John Vines [ jvines ]
        Hide
        jv added a comment -

        David - if that meets your standards, go ahead and close it. If not, let me know what needs clarification

        Show
        jv added a comment - David - if that meets your standards, go ahead and close it. If not, let me know what needs clarification
        Hide
        David Medinets added a comment -

        The comments look great. They lead me to my next question. The script does not delete old generated Java files. I don't know much about Thift. Could it be possible for the templates to have changed so that some generated Java files are no longer needed? How would they be managed?

        Show
        David Medinets added a comment - The comments look great. They lead me to my next question. The script does not delete old generated Java files. I don't know much about Thift. Could it be possible for the templates to have changed so that some generated Java files are no longer needed? How would they be managed?
        Hide
        Eric Newton added a comment -

        You're right, there are sometimes issues. I just had problems building when the package name changed for the cloudtrace stuff. Fortunately, the thrift calls change so infrequently, that we just deal with it manually. But, if you wanted to do something better, that would be great.

        I would like to look at Avro, too. Maybe something to experiment for 1.5. We'll be moving up to the latest version of thrift, at least, and that will require some changes to the tracing stuff.

        Show
        Eric Newton added a comment - You're right, there are sometimes issues. I just had problems building when the package name changed for the cloudtrace stuff. Fortunately, the thrift calls change so infrequently, that we just deal with it manually. But, if you wanted to do something better, that would be great. I would like to look at Avro, too. Maybe something to experiment for 1.5. We'll be moving up to the latest version of thrift, at least, and that will require some changes to the tracing stuff.
        Hide
        David Medinets added a comment -

        Is there any reason why this script should not delete the contents of the src/trace/src/main/java/cloudtrace/thrift directory for each run of the script? Is anything in that directory other than thrift-generated files?

        Show
        David Medinets added a comment - Is there any reason why this script should not delete the contents of the src/trace/src/main/java/cloudtrace/thrift directory for each run of the script? Is anything in that directory other than thrift-generated files?
        Hide
        Eric Newton added a comment -

        I don't think there's a reason. The script in src/core only copies files over if they are different: you definitely want to preserve that behavior or eclipse will constantly recompile the files generated by thrift.

        Show
        Eric Newton added a comment - I don't think there's a reason. The script in src/core only copies files over if they are different: you definitely want to preserve that behavior or eclipse will constantly recompile the files generated by thrift.
        David Medinets made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Assignee John Vines [ jvines ] David Medinets [ medined ]
        Fix Version/s 1.5.0-SNAPSHOT [ 12319889 ]
        Resolution Fixed [ 1 ]
        David Medinets made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Gavin made changes -
        Workflow no-reopen-closed, patch-avail [ 12656575 ] patch-available, re-open possible [ 12671772 ]
        Christopher Tubbs made changes -
        Fix Version/s 1.5.0 [ 12318645 ]
        Fix Version/s 1.5.0-SNAPSHOT [ 12319889 ]
        Anonymous made changes -
        Status Closed [ 6 ] Resolved [ 5 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        4d 16h 5m 1 David Medinets 09/Mar/12 19:23
        Resolved Resolved Closed Closed
        2h 40m 1 David Medinets 09/Mar/12 22:03
        Closed Closed Resolved Resolved
        402d 23h 21m 1 16/Apr/13 22:25

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development