Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 5.3
    • Component/s: tapestry-ioc
    • Labels:
      None

      Description

      For the implementation of tapestry-hibernate with multiple databases I need to be able to see whether certain services are realized or not.

      The attached patch improves ServiceActivityScoreboard in two ways.

      • ServiceActivity interface now includes getMarkers() (which calls serviceDef.getMarkers())
      • ServiceActivity now correctly handles perthread services.

        Activity

        Hide
        Hudson added a comment -

        Integrated in tapestry-5.2-freestyle #293 (See https://hudson.apache.org/hudson/job/tapestry-5.2-freestyle/293/)
        TAP5-1320: Applied Tom van Dijk's patch for adding getMarkers() and
        perthread service support to ServiceActivity without changes

        Show
        Hudson added a comment - Integrated in tapestry-5.2-freestyle #293 (See https://hudson.apache.org/hudson/job/tapestry-5.2-freestyle/293/ ) TAP5-1320 : Applied Tom van Dijk's patch for adding getMarkers() and perthread service support to ServiceActivity without changes
        Hide
        Kalle Korhonen added a comment -

        Applied, thanks to Tom for a high quality patch. Howard's comment might be a useful improvement to the page should be dealt as a separate issue, closing.

        Show
        Kalle Korhonen added a comment - Applied, thanks to Tom for a high quality patch. Howard's comment might be a useful improvement to the page should be dealt as a separate issue, closing.
        Hide
        Tom van Dijk added a comment -

        This one has tests

        Show
        Tom van Dijk added a comment - This one has tests
        Hide
        Howard M. Lewis Ship added a comment -

        At some point, I'd like to fix up this page to use a more ajax style: a left pane listing the services, and a right pane with all the details for the selected service.

        Show
        Howard M. Lewis Ship added a comment - At some point, I'd like to fix up this page to use a more ajax style: a left pane listing the services, and a right pane with all the details for the selected service.
        Hide
        Kalle Korhonen added a comment -

        Unit test missing, see comments on mailing list

        Show
        Kalle Korhonen added a comment - Unit test missing, see comments on mailing list
        Hide
        Tom van Dijk added a comment -

        Here is the patch file (using git format-patch)

        Show
        Tom van Dijk added a comment - Here is the patch file (using git format-patch)

          People

          • Assignee:
            Kalle Korhonen
            Reporter:
            Tom van Dijk
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development