Uploaded image for project: 'Core Server'
  1. Core Server
  2. SERVER-40989

Scrub HostAndPort of kruft

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major - P3
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.2.0-rc3, 4.3.1
    • Component/s: None
    • Labels:
      None
    • Backwards Compatibility:
      Fully Compatible
    • Backport Requested:
      v4.2
    • Sprint:
      Service Arch 2019-05-20, Service Arch 2019-06-03, Service Arch 2019-06-17

      Description

      HostAndPort now has a SockAddr optional that it carries around for relatively little use. We should obviate that.

      As a stretch goal that probably won't happen, we also carry around a std::string host for every HaP. Standard use sees maybe 6 hosts ever used to construct HaP. We could probably figure out a place to stash these strings and actually have StringData instead.

        Attachments

          Activity

            People

            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: