Improve/simplify archival config validation

[longquanzheng]

What changed?
Improve archival config validation

Why?
For #4361

How did you test it?
Unit tests

Potential risks
No

Release notes

Documentation Changes

[coveralls]

Pull Request Test Coverage Report for Build 5acfe373-4d40-4ad9-82f6-fff791464145

• 16 of 16 (100.0%) changed or added relevant lines in 1 file are covered.
• 38 unchanged lines in 11 files lost coverage.
• Overall coverage increased (+0.06%) to 56.534%

---

Files with Coverage Reduction	New Missed Lines	%
common/task/weightedRoundRobinTaskScheduler.go	1	89.64%
service/history/queue/timer_queue_processor_base.go	1	78.56%
client/history/client.go	2	41.25%
client/history/metricClient.go	2	47.05%
service/history/execution/mutable_state_builder.go	2	69.86%
service/history/handler.go	2	48.15%
service/history/queue/transfer_queue_processor.go	2	57.18%
service/frontend/workflowHandler.go	4	58.8%
common/task/fifoTaskScheduler.go	5	82.47%
common/persistence/nosql/nosqlExecutionStore.go	8	59.12%

Totals
Change from base Build 1560c792-cbd8-4996-9f1a-9f3912c0c6ae:	0.06%
Covered Lines:	79061
Relevant Lines:	139848

---

💛 - Coveralls