Uploaded image for project: 'OODT'
  1. OODT
  2. OODT-10

Make BatchMgrProxy an interface and node assignable

    Details

    • Type: Improvement
    • Status: Open
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: 0.1-incubating
    • Fix Version/s: 1.1
    • Component/s: resource manager
    • Labels:
      None
    • Environment:

      none

      Description

      Instead of Batchmgr being an interface:

      • currently the existing implementation of BatchMgr, 'XmlRpcBatchMgr', has nothing to do with XmlRpc besides that is uses XmlRpcBatchMgrProxy, which is really what does all the XmlRpc communications. XmlRpcBatchMgr currently does everything which all BatchMgr's need to perform (i.e. updating repo, monitor, etc), where as, i argue, XmlRpcBatchMgrProxy actually performs the tasks which one would want to be customizable.
      • my purposed change: Create an interface, 'BatchMgrProxy', and make XmlRpcBatchMgrProxy extend it, while also making it the default BatchMgrProxy, while adding to ResourceNode the ability to store a BatchMgrProxy, which is used to communicate with the batchstub at that ResourceNode uri. This would allow a developer to added additional Proxy-Monitoring to a BatchMgrProxy which may be needed on a per node bases (i.e. a batch stub running on a EC2 node needs additional Proxy functionality over the existing functionality needed to submit to a batch stub running locally). XmlRpcBatchMgr would become BatchMgr, and would get added support for handling node based Proxies.

        Activity

        Hide
        chrismattmann Chris A. Mattmann added a comment -
        • push out to 0.2
        Show
        chrismattmann Chris A. Mattmann added a comment - push out to 0.2
        Hide
        chrismattmann Chris A. Mattmann added a comment -
        • clean up title
        Show
        chrismattmann Chris A. Mattmann added a comment - clean up title
        Hide
        pramirez Paul Ramirez added a comment -

        Pushing off to 0.3 release.

        Show
        pramirez Paul Ramirez added a comment - Pushing off to 0.3 release.
        Hide
        chrismattmann Chris A. Mattmann added a comment -
        • push to 0.5
        Show
        chrismattmann Chris A. Mattmann added a comment - push to 0.5
        Hide
        chrismattmann Chris A. Mattmann added a comment -
        • push out to 0.5
        Show
        chrismattmann Chris A. Mattmann added a comment - push out to 0.5
        Hide
        chrismattmann Chris A. Mattmann added a comment -
        • push out to 0.6
        Show
        chrismattmann Chris A. Mattmann added a comment - push out to 0.6

          People

          • Assignee:
            Unassigned
            Reporter:
            bfoster Brian Foster
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:

              Development