Jesper Pedersen wrote:
The only optimization in this area is not to have the valve enabled for deployment that don't need it.
Glancing at the code there appears to be room for optimization. You only need to *mark* the thread/request if a connection is actually opened by a servlet/jsp. Also I am not sure why it is trying suspend transactions. Looks like that is a debugging feature and not any kind of real necessity.