Uploaded image for project: 'Apache ServiceComb'
  1. Apache ServiceComb
  2. SCB-2038

Rename AbstractRestInvocation and its class hierarchy to avoid confusion

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Trivial
    • Resolution: Won't Fix
    • None
    • java-chassis-2.1.1
    • None
    • None

    Description

      AbstractRestInvocation class is not an invocation itself, it contains an Invocation field which is the invocation. The responsibilities of AbstractRestInvocation are the preparation, creation, and coordination of the invocation. 

       

      The class name is quite confusing and so are the inherited classes of this class. Rename this class hierarchy to Coordinator class will be much clearer in terms of what they do.

       

      The proposed new class names:

       

      AbstractRestInvocation -> AbstractRestInvocationCoordinator

      RestProducerInvocation -> RestProducerInvocationCoordinator

      VertxRestInvocation -> VertxRestInvocationCoordinator

      EdgeInvocation -> EdgeInvocationCoordinator

      RestServletProducerInvocation -> RestServletProducerInvocationCoordinator

       

       

      Attachments

        Activity

          People

            liulikun Likun Liu
            liulikun Likun Liu
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: