Elias Naur wrote:
On 4/7/07, Rémy Maucherat <[EMAIL PROTECTED]> wrote:
On 4/6/07, Elias Naur <[EMAIL PROTECTED]> wrote:
> I'm aware that a correct implementation must empty the buffer on a
> READ event, but as I stated in the original post, a READ event is
> never received (not even when the client forces a disconnect), only
> the initial BEGIN event, so had I included a read on READ it wouldn't
> matter and would only serve to clutter the test case. If I'm supposed
> to read when receiving BEGIN, I'll admit my test is incorrect, but
> then the example at http://tomcat.apache.org/tomcat-6.0-doc/aio.html
> needs to be corrected too.
So you can also confirm APR and NIO do the same ? I don't see how
you're not going to get an error event in that particular case.
However, I can't test it personally, so maybe you can look into this a
bit deeper (otherwise, most likely Filip will).
Rémy
No. Compiling and enabling the APR connector instead of the NIO
connector made the problem go away. There were still no ERROR events
sent after the client disconnect (I guess the connection might timeout
sometime later), but the original problem of 100% CPU usage is gone.
ok, lets put an end to this now, shall we :)
The problem is not with any of the connectors, it is with your code.
On Windows Your Code does:
NIO: 100% CPU because you get issued READ after READ, and you are not
emptying the buffer
APR: 100% CPU because you get issued READ after READ, and you are not
emptying the buffer
On Linux Your Code does:
NIO: 100% CPU because you get issued READ after READ, and you are not
emptying the buffer
APR:BEGIN event, followed by an ERROR event.
As you might figure out by this point it's just how the different IO
apis work on different platforms.
This is still 100% fault of your own code, the Java NIO api doesn't
notify us of a connection close or hangup, it notifies us as if it was
an READ event
and you are not handling it.
This has nothing to do with APR or NIO, had your code was correct, you'd
never have 100% CPU, here is your code corrected.
public void event(CometEvent event) throws IOException, ServletException {
System.out.println("event.getEventType() = " +
event.getEventType());
if ( event.getEventType() == CometEvent.EventType.READ ) {
if (event.getHttpServletRequest().getInputStream().read(new
byte[1024]) == -1) {
event.close();
}
}
}
- elias
------------------------------------------------------------------------
No virus found in this incoming message.
Checked by AVG Free Edition.
Version: 7.5.446 / Virus Database: 268.18.26/748 - Release Date: 4/5/2007 3:33
PM
---------------------------------------------------------------------
To start a new topic, e-mail: users@tomcat.apache.org
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
