Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.0-beta-1
    • Component/s: None
    • Labels:
      None

      Description

      It'd be handy to auto-restart the tomcat context when the project's dependency jar's timestamp changes. Also need to investigate the effect of this versus the context reloadable=true attribute.

        Activity

        Hide
        Mark Thomas added a comment -

        As the default value for tomcat is false see http://tomcat.apache.org/tomcat-6.0-doc/config/context.html
        I prefer to have same here.

        Show
        Mark Thomas added a comment - As the default value for tomcat is false see http://tomcat.apache.org/tomcat-6.0-doc/config/context.html I prefer to have same here.
        Hide
        Mark Thomas added a comment -

        Makes perfect sense to me. Thanks!

        Show
        Mark Thomas added a comment - Makes perfect sense to me. Thanks!
        Hide
        Mark Thomas added a comment -

        no need of patch . I have already pushed your

        Show
        Mark Thomas added a comment - no need of patch . I have already pushed your
        Hide
        Mark Thomas added a comment -

        so as not all users use a context file.
        I will add a mojo parameter :

             * @parameter expression="" default-value="false"
            private boolean contextReloadable = false;
        

        With this you will be able to use this new feature without adding a context file : mojo configuration or cli

        Show
        Mark Thomas added a comment - so as not all users use a context file. I will add a mojo parameter : * @parameter expression= "" default -value=" false " private boolean contextReloadable = false ; With this you will be able to use this new feature without adding a context file : mojo configuration or cli
        Hide
        Mark Thomas added a comment -

        fixed rev 13551.
        SNAPSHOT deployed.
        NOTE : I have added support for backgroundProcessorDelay (-Dmaven.tomcat.backgroundProcessorDelay )

        Show
        Mark Thomas added a comment - fixed rev 13551. SNAPSHOT deployed. NOTE : I have added support for backgroundProcessorDelay (-Dmaven.tomcat.backgroundProcessorDelay )
        Hide
        Mark Thomas added a comment -

        Here is simple solution to that problem. Context is reloaded every time some class or jar from dependecies is changed.

        Patch to the current trunk version of plugin.

        Show
        Mark Thomas added a comment - Here is simple solution to that problem. Context is reloaded every time some class or jar from dependecies is changed. Patch to the current trunk version of plugin.
        Hide
        Mark Thomas added a comment -

        I've updated the patch to be applicable to the current state of trunk. It would be great to get this introduced!

        Show
        Mark Thomas added a comment - I've updated the patch to be applicable to the current state of trunk. It would be great to get this introduced!
        Hide
        Mark Thomas added a comment -

        updated patch to include javadocs and maven code style.

        Show
        Mark Thomas added a comment - updated patch to include javadocs and maven code style.
        Hide
        Mark Thomas added a comment -

        new patch checks reloadable in any supplied context.xml if one should exist.

        Show
        Mark Thomas added a comment - new patch checks reloadable in any supplied context.xml if one should exist.

          People

          • Assignee:
            Olivier Lamy (*$^¨%`£)
            Reporter:
            Mark Hobson
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Due:
              Created:
              Updated:
              Resolved:

              Development