Fix race condition in Describe handler

[Shaddoll]

What changed?
Deep copy maps in describe handler

Why?
To avoid concurrent read/write map, because we do some type conversion in Thrift/Grpc handler when the maps are not protected by any lock.

How did you test it?
unit tests

Potential risks

Release notes

Documentation Changes

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 73.26%. Comparing base (7b6846d) to head (c4a4ebc).
Report is 4 commits behind head on master.

Additional details and impacted files

Files with missing lines	Coverage Δ
common/persistence/data_manager_interfaces.go	96.03% <100.00%> (+0.55%)	⬆️
...y/engine/engineimpl/describe_workflow_execution.go	79.03% <100.00%> (ø)

... and 4 files with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7b6846d...c4a4ebc. Read the comment docs.