Re: Bug in DateTool in tomcat connector util?

2002-09-19 Thread Bill Barker


- Original Message -
From: "Hugh J. L." <[EMAIL PROTECTED]>
To: "Tomcat Developers List" <[EMAIL PROTECTED]>
Sent: Thursday, September 19, 2002 12:03 AM
Subject: Re: Bug in DateTool in tomcat connector util?


> Just viewed DateTool in CVS. Yes, "format" also has
> thread problem and need fix. But, has "parse" been
> fixed? I didn't see any change about it in v1.7.

This was a pretty big change. It was reported as bugzilla #8562.  Also see
o.a.t.core.Request[R1.115] and
o.a.t.modules.generators.StaticInterceptor[R1.23].

On the "parse" side, it uses the Thread-Local DateFormatters from the
Request to parse the date.  Time permitting, I plan to eventually convert
the "format" side to use them as well.

>
> --- Bill Barker <[EMAIL PROTECTED]> wrote:
> > I just checked in a fix for the "format" side
> > (setting the "Last-Modified"
> > header).  This one is much harder to hit, but that
> > doesn't mean that you
> > can't.  The "parse" side (getting the
> > "If-Modified-Since" header) has been
> > fixed in the nightly for quite some time now.
> >
> > - Original Message -
> > From: "Hugh J. L." <[EMAIL PROTECTED]>
> > To: "Tomcat Developers List"
> > <[EMAIL PROTECTED]>
> > Sent: Tuesday, September 17, 2002 7:12 PM
> > Subject: Bug in DateTool in tomcat connector util?
> >
> >
> > > Hi,
> > >
> > > I found a bug (?) when i was testing tomcat3.3
> > using
> > > MS web stress tool. I set up large amount of
> > > concurrent requests for static files, each with
> > header
> > > If-Modified-Since which is newer than actual
> > > last-modified-time of those requested files. I
> > should
> > > have got 304 response for ALL requests in this
> > case,
> > > however, i got many 304, a few 200, and even
> > > NumberFormatException occasionally. This didn't
> > happen
> > > if i tested using only one client thread. I looked
> > > into java.text.SimpleDateFormat and suspected it
> > was
> > > possibly a synchronization problem. Then I added
> > > synchronization control to DateTool.parse(String,
> > > DateFormat[]) as below and the problem was solved:
> > >
> > > synchronized(format[i]) {
> > > date = format[i].parse(dateString);
> > > }
>
>
> __
> Do you Yahoo!?
> New DSL Internet Access from SBC & Yahoo!
> http://sbc.yahoo.com
>
> --
> To unsubscribe, e-mail:
<mailto:[EMAIL PROTECTED]>
> For additional commands, e-mail:
<mailto:[EMAIL PROTECTED]>
>


--
To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>




Re: Bug in DateTool in tomcat connector util?

2002-09-18 Thread Hugh J. L.

Just viewed DateTool in CVS. Yes, "format" also has
thread problem and need fix. But, has "parse" been
fixed? I didn't see any change about it in v1.7.

--- Bill Barker <[EMAIL PROTECTED]> wrote:
> I just checked in a fix for the "format" side
> (setting the "Last-Modified"
> header).  This one is much harder to hit, but that
> doesn't mean that you
> can't.  The "parse" side (getting the
> "If-Modified-Since" header) has been
> fixed in the nightly for quite some time now.
> 
> - Original Message -
> From: "Hugh J. L." <[EMAIL PROTECTED]>
> To: "Tomcat Developers List"
> <[EMAIL PROTECTED]>
> Sent: Tuesday, September 17, 2002 7:12 PM
> Subject: Bug in DateTool in tomcat connector util?
> 
> 
> > Hi,
> >
> > I found a bug (?) when i was testing tomcat3.3
> using
> > MS web stress tool. I set up large amount of
> > concurrent requests for static files, each with
> header
> > If-Modified-Since which is newer than actual
> > last-modified-time of those requested files. I
> should
> > have got 304 response for ALL requests in this
> case,
> > however, i got many 304, a few 200, and even
> > NumberFormatException occasionally. This didn't
> happen
> > if i tested using only one client thread. I looked
> > into java.text.SimpleDateFormat and suspected it
> was
> > possibly a synchronization problem. Then I added
> > synchronization control to DateTool.parse(String,
> > DateFormat[]) as below and the problem was solved:
> >
> > synchronized(format[i]) {
> > date = format[i].parse(dateString);
> > }


__
Do you Yahoo!?
New DSL Internet Access from SBC & Yahoo!
http://sbc.yahoo.com

--
To unsubscribe, e-mail:   
For additional commands, e-mail: 




Re: Bug in DateTool in tomcat connector util?

2002-09-17 Thread Bill Barker

I just checked in a fix for the "format" side (setting the "Last-Modified"
header).  This one is much harder to hit, but that doesn't mean that you
can't.  The "parse" side (getting the "If-Modified-Since" header) has been
fixed in the nightly for quite some time now.

- Original Message -
From: "Hugh J. L." <[EMAIL PROTECTED]>
To: "Tomcat Developers List" <[EMAIL PROTECTED]>
Sent: Tuesday, September 17, 2002 7:12 PM
Subject: Bug in DateTool in tomcat connector util?


> Hi,
>
> I found a bug (?) when i was testing tomcat3.3 using
> MS web stress tool. I set up large amount of
> concurrent requests for static files, each with header
> If-Modified-Since which is newer than actual
> last-modified-time of those requested files. I should
> have got 304 response for ALL requests in this case,
> however, i got many 304, a few 200, and even
> NumberFormatException occasionally. This didn't happen
> if i tested using only one client thread. I looked
> into java.text.SimpleDateFormat and suspected it was
> possibly a synchronization problem. Then I added
> synchronization control to DateTool.parse(String,
> DateFormat[]) as below and the problem was solved:
>
> synchronized(format[i]) {
> date = format[i].parse(dateString);
> }
>
>
> __
> Do you Yahoo!?
> Yahoo! News - Today's headlines
> http://news.yahoo.com
>
> --
> To unsubscribe, e-mail:

> For additional commands, e-mail:

>


--
To unsubscribe, e-mail:   
For additional commands, e-mail: