Traffic Server
  1. Traffic Server
  2. TS-392

Support (or remove) WITH_DETAILED_VCONNECTION_LOGGING

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.1.2
    • Component/s: Network
    • Labels:
      None

      Description

      There's an #ifdef WITH_DETAILED_VCONNECTION_LOGGING which is always "disabled" in the code. This feature depends on DetailedLog.

      {cc,h}

      , 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.

        Activity

        Leif Hedstrom created issue -
        Leif Hedstrom made changes -
        Field Original Value New Value
        Fix Version/s 2.1.2 [ 12315091 ]
        Leif Hedstrom made changes -
        Assignee Leif Hedstrom [ zwoop ]
        Leif Hedstrom made changes -
        Component/s Network [ 12313122 ]
        Hide
        Leif Hedstrom added a comment -

        Closing, there is now a new option to configure, --with-detailed-log

        Show
        Leif Hedstrom added a comment - Closing, there is now a new option to configure, --with-detailed-log
        Leif Hedstrom made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Leif Hedstrom made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Gavin made changes -
        Workflow jira [ 12513638 ] TS Workflow [ 12522381 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        22d 1h 11m 1 Leif Hedstrom 09/Jul/10 18:16
        Resolved Resolved Closed Closed
        46d 9h 37m 1 Leif Hedstrom 25/Aug/10 03:53

          People

          • Assignee:
            Leif Hedstrom
            Reporter:
            Leif Hedstrom
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development