Removed unused internal type getters

[vytautas-karpavicius]

What changed?
Removed all unused internal type getters.

Why?
Dead code. We should move away from using them at all. Those are legacy from previously used Thrift types.

How did you test it?
Successful compilation is enough to ensure that they are not needed.

Potential risks

Release notes

Documentation Changes

[coveralls]

Pull Request Test Coverage Report for Build 018113ed-24ec-4cf7-9307-3c5b372498d7

• 26 of 31 (83.87%) changed or added relevant lines in 1 file are covered.
• 72 unchanged lines in 8 files lost coverage.
• Overall coverage increased (+1.1%) to 57.971%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
common/types/shared.go	26	31	83.87%

Files with Coverage Reduction	New Missed Lines	%
common/types/shared.go	2	41.75%
service/history/task/transfer_active_task_executor.go	2	71.41%
common/persistence/nosql/nosqlplugin/cassandra/workflow.go	3	57.59%
common/types/mapper/thrift/shared.go	4	63.9%
common/persistence/nosql/nosqlplugin/cassandra/workflowUtils.go	12	77.59%
common/persistence/nosql/nosqlplugin/cassandra/tasks.go	14	72.8%
common/persistence/nosql/nosqlTaskStore.go	16	58.37%
common/persistence/nosql/nosqlplugin/cassandra/workflowParsingUtils.go	19	86.32%

Totals
Change from base Build 018113ec-bab3-4ae2-aec2-c3e6e85b267c:	1.1%
Covered Lines:	83913
Relevant Lines:	144750

---

💛 - Coveralls

[davidporter-id-au]

Red diffs are good diffs

Also appreciate the move away from Java-isms