Update Pinto query validator failed log, minor refactor pinot visibility store to remove panics

[neil-xie]

What changed?
Update Pinto query validator failed log to include more details
Refactor pinot visibility store code to remove panics

Why?

How did you test it?
Unit test

Potential risks

Release notes

Documentation Changes

[coveralls]

Pull Request Test Coverage Report for Build 018dae76-e956-4174-8ebe-895cefe2ba98

Details

• -56 of 77 (27.27%) changed or added relevant lines in 2 files are covered.
• 38 unchanged lines in 9 files lost coverage.
• Overall coverage increased (+0.03%) to 62.74%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
common/pinot/pinotQueryValidator.go	4	9	44.44%
common/persistence/pinot/pinotVisibilityStore.go	17	68	25.0%

Files with Coverage Reduction	New Missed Lines	%
common/task/parallel_task_processor.go	2	93.06%
common/task/weighted_round_robin_task_scheduler.go	2	88.06%
service/history/shard/context.go	2	66.52%
service/matching/matcher.go	2	90.72%
service/matching/taskReader.go	2	84.88%
service/history/queue/timer_gate.go	3	95.83%
service/history/queue/timer_queue_processor_base.go	3	77.82%
common/persistence/pinot/pinotVisibilityStore.go	5	38.21%
service/history/execution/mutable_state_task_refresher.go	17	61.71%

Totals
Change from base Build 018dacf0-c704-41a4-9d3b-b82276de1999:	0.03%
Covered Lines:	92618
Relevant Lines:	147622

---

💛 - Coveralls