foundationdb
foundationdb copied to clipboard
Unify peer address key in event traces
I didn't find any matching issue (if that is somewhere else address please close this issue). I noticed that some event have the PeerAddress key and some others have PeerAddr. As far as I can tell these always reference the same value (e.g. the address of the peer). It would be nice to have the same key in all events to make it easier for an operator to search for events with a peer address. Personally I would prefer PeerAddress. In the transition phase the changed event should probably log both and then in the next release we can drop the deprecated key.
TraceEvent has a specialization for address and we could add a specialization like .peerAddr().