There's an #ifdef WITH_DETAILED_VCONNECTION_LOGGING which is always "disabled" in the code. This feature depends on DetailedLog.
, and in fact, it's the only piece in the code that uses this DetailedLog class.
I'd suggest two possible improvements
1) Make WITH_DETAILED_VCONNECTION_LOGGING ./configure settable (this is assuming we need / want this detailed log feature for vconnections). If we do this, then compiling DetailedLog.cc should be conditional on this option being enabled.
2) Remove this feature, and the supporting code, entirely.