OODT
  1. OODT
  2. OODT-10

Make BatchMgrProxy an interface and node assignable

    Details

    • Type: Improvement Improvement
    • Status: Open
    • Priority: Minor Minor
    • Resolution: Unresolved
    • Affects Version/s: 0.1-incubating
    • Fix Version/s: 0.9
    • 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
        Chris A. Mattmann added a comment -
        • push out to 0.2
        Show
        Chris A. Mattmann added a comment - push out to 0.2
        Hide
        Chris A. Mattmann added a comment -
        • clean up title
        Show
        Chris A. Mattmann added a comment - clean up title
        Hide
        Paul Ramirez added a comment -

        Pushing off to 0.3 release.

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

          People

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

            Dates

            • Created:
              Updated:

              Development