2011/10/21 Mark Thomas <ma...@apache.org>:
> On 20/10/2011 08:00, kfuj...@apache.org wrote:
>> Author: kfujino
>> Date: Thu Oct 20 07:00:57 2011
>> New Revision: 1186631
>>
>> URL: http://svn.apache.org/viewvc?rev=1186631&view=rev
>> Log:
>> Avoid an unnecessary session ID change notice.
>
> Given this...
>> Notice of changed session ID by JvmRouteBinderValve is unnecessary to 
>> BackupManager.
>
>> +        if (c != null && getManager(request) instanceof DeltaManager) {
>
> Might this be better to account for custom Manager implementations?
>
> if (c != null && !(getManager(request) instanceof BackupManager)) {
>
>
> Mark

Thanks for the review.
I committed on r1187149.


-- 
Keiichi.Fujino

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

Reply via email to