Use a common HashTable for global APM and log subscribers

XMLWordPrintableJSON

    • Type: Improvement
    • Resolution: Unresolved
    • Priority: Unknown
    • None
    • Affects Version/s: None
    • Component/s: None
    • None
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      The initial implementation of LogSubscriber in PHPC-2180 maintains a separate global HashTable for registered loggers. This dates back to the original prototype, which used separate add/removeLogger functions, but was retained after modifying the existing add/removeSubscriber methods to handle loggers.

      Logging is technically more performant this way since it does not need to filter objects in a HashTable (as is done in phongo_apm_get_subscribers_to_notify); however, that may not be worth the code duplication and additional maintenance. Logging is also not a critical code path.

            Assignee:
            Unassigned
            Reporter:
            Jeremy Mikola
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated: