On 02/10/2014 15:41, Martin Grigorov wrote:
> Hi,
> 
> On Thu, Oct 2, 2014 at 4:36 PM, <schu...@apache.org> wrote:
> 
>> Author: schultz
>> Date: Thu Oct  2 14:36:27 2014
>> New Revision: 1628984
>>
>> URL: http://svn.apache.org/r1628984
>> Log:
>> Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=57049
>> Clarified that jvmRoute can be set using Engine.jvmRoute or a system
>> property.
>>
>> Modified:
>>     tomcat/trunk/webapps/docs/changelog.xml
>>     tomcat/trunk/webapps/docs/config/engine.xml
>>
>> Modified: tomcat/trunk/webapps/docs/changelog.xml
>> URL:
>> http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/changelog.xml?rev=1628984&r1=1628983&r2=1628984&view=diff
>>
>> ==============================================================================
>> --- tomcat/trunk/webapps/docs/changelog.xml (original)
>> +++ tomcat/trunk/webapps/docs/changelog.xml Thu Oct  2 14:36:27 2014
>> @@ -109,7 +109,12 @@
>>          Correct documentation for
>> <code>ServerCookie.ALLOW_NAME_ONLY</code>
>>          system property. (kkolinko)
>>        </fix>
>> -    </changelog>
>> +      <fix>
>> +        <bug>57049</bug>: Clarified that <code>jvmRoute</code> can be set
>> in
>> +        <code>&lt;Engine&gt;</code>'s <code>jvmRoute</code> or in a system
>> +        property. (schultz)
>> +      </fix>
>> +      </changelog>
>>    </subsection>
>>  </section>
>>  <section name="Tomcat 8.0.14 (markt)" rtext="2014-09-29">
>>
>> Modified: tomcat/trunk/webapps/docs/config/engine.xml
>> URL:
>> http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/config/engine.xml?rev=1628984&r1=1628983&r2=1628984&view=diff
>>
>> ==============================================================================
>> --- tomcat/trunk/webapps/docs/config/engine.xml (original)
>> +++ tomcat/trunk/webapps/docs/config/engine.xml Thu Oct  2 14:36:27 2014
>> @@ -93,6 +93,12 @@
>>          the generated session identifier, therefore allowing the front end
>>          proxy to always forward a particular session to the same Tomcat
>>          instance.</p>
>> +        <p>
>> +            Note that the <code>jvmRoute</code> can also be set using the
>> +            <code>jvmRoute</code> system property. The
>> <code>jvmRoute</code>
>> +            set in an <code>&lt;Engine&gt;</code> attribute will override
>> +            any <code>jvmRoute</code> system property.
>> +        </p>
>>
> 
> This sounds counter intuitive.
> Usually manually provided settings, like system properties, have higher
> priority than ones provided in configuration files.

Not in Tomcat they don't.

System properties are often introduced to provide configuration options
where there isn't an obvious / easy element on which to place a
configuration attribute.

At some point in the future, things are refactored making it possible to
have per element configuration. At this point the system property
becomes the default and the element level always overrides it.

The ideal solution would be to never use system properties in the first
place.

Mark


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to