#378 Carry over hash value for all literal types in remote caching Pull request opened by nicholasjng TL;DR Previously, the hash on an input literal would be ignored if the literal was of either the

Collection

or

Map

type. This is fixed here by moving the hash-checking branch up in front of the special type logic, so that an empty literal with hash is constructed in either case if a hash is found. Adds a regression test to verify the hashes are carried over into the representer literals for both map and collection types. Type ☑︎ Bug Fix ☐ Feature ☐ Plugin Are all requirements met? ☑︎ Code completed ☐ Smoke tested ☑︎ Unit tests added ☑︎ Code documentation added ☑︎ Any pending items have an associated Issue Complete description See the discussion in flyteorg/flytekit#1720 for the problem statement. This PR is a direct follow-up. The remote cache fix was missing from that PR, and that is exactly what is added here. The code here is the same fix approach, just in Go this time. Tracking Issue N/A flyteorg/flyteplugins ✅ All checks have passed 7/7 successful checks