OK,
 I have a fix for pconn.cc and idleConnList.
Looks OK now. I am still getting assertion here or there but looks that there are unrelated to connections pool, I hope will be able to fix them.

My commit to trunk with revno:11534 was wrong. What is the best way, should I undo (Is it possible?) the commit or just commit the new fixes?





On 07/06/2011 04:25 PM, Amos Jeffries wrote:
On 06/07/11 04:04, Tsantilas Christos wrote:
On 07/05/2011 12:23 PM, Amos Jeffries wrote:


OK, for now I will try to just remove these tests ...


You mentioned other asserts. Still happening and what are they?

Yep.
I disabled the read_pending flag tests and now I am getting the
following assertion:
assertion failed: comm.cc:170: "fd_table[conn->fd].halfClosedReader !=
NULL"

The above assertion occurs with or without ICAP enabled, and looks that
it has to do with pconn.cc code....
Still I am not able to make current trunk working, for my testing
configurations. Even If I fixed some problems, I am getting again and
again new assertions....

Ahmm.. I think IdleConnList::pop() and IdleConnList::findUsable() are
both missing a call to clearHandlers(result) before calling removeAt().


Amos

Reply via email to