Comment 26 for bug 1704396

Revision history for this message
Mike Rylander (mrylander) wrote : Re: Slowness for metecord and one-hit searches in 2.12

Jeff,

Well... that's unfortunate. Now I'm getting concerned about src/perl/lib/OpenSRF/Application.pm around lines 170-177. That's where we check to see if this drone has served as many requests as it's allowed.

The status() call in that block should be tossing a message at the caller with a STATUS_REDIRECTED statusCode, which should then invoke the code at src/perl/lib/OpenSRF/DomainObject/oilsMessage.pm lines 334-340. Part of that is to write a WARN-level message to the log that says "Disconnected because of redirect", which we're not seeing, and the request should be resent to the service.

I wonder if we could be dropping the socket and going away too fast, before the data makes it out the door, if we hit max requests. That shouldn't be, since the socket should have autoflush turned on (per https://perldoc.perl.org/IO/Socket.html ) in modern versions of IO::Socket.

**sad-emoji**

The above patch may still be useful.