Use named port to select transport for outbound calls

[mantas-sidlauskas]

What changed?
AddressMapper is changed to named port setting.

Some additional changes in this PR:

• CreateShardOwnershipLostError is returning TChannel address for backwards compatibility
• LookupByAddress returns membership.HostInfo which is handy to find peer based on IP:port without knowing transport details
• Use zaptest as logger in unit tests

Why?
Named port (tchannel or grpc) will now define what transport is to be used for outbound calls.

How did you test it?
Unit tests, integration tests

Potential risks

Release notes

Documentation Changes

[coveralls]

Pull Request Test Coverage Report for Build 59d8203a-4cbd-4467-b391-6c762d541078

• 131 of 171 (76.61%) changed or added relevant lines in 14 files are covered.
• 36 unchanged lines in 12 files lost coverage.
• Overall coverage increased (+0.03%) to 56.824%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
client/history/peerResolver.go	6	7	85.71%
common/membership/hostinfo.go	25	26	96.15%
cmd/server/cadence/server.go	0	2	0.0%
host/membership_resolver.go	9	12	75.0%
common/peerprovider/ringpopprovider/provider.go	0	4	0.0%
service/history/handler.go	0	6	0.0%
common/membership/resolver.go	0	10	0.0%
host/onebox.go	55	68	80.88%

Files with Coverage Reduction	New Missed Lines	%
client/history/client.go	2	39.33%
client/history/metricClient.go	2	46.37%
common/membership/hashring.go	2	83.54%
common/peerprovider/ringpopprovider/config.go	2	81.58%
common/peerprovider/ringpopprovider/provider.go	2	0%
common/task/weightedRoundRobinTaskScheduler.go	2	88.6%
service/history/handler.go	2	48.72%
common/cache/lru.go	3	90.73%
common/log/tag/tags.go	3	51.22%
common/task/fifoTaskScheduler.go	3	84.54%

Totals
Change from base Build 43de9000-0634-408f-b6fb-9a22dea6c19b:	0.03%
Covered Lines:	83435
Relevant Lines:	146831

---

💛 - Coveralls

[vytautas-karpavicius]

Hmm, shouldn't his return an error? Loop is finished but host not found yet.