#1578 Remove duplicate reporting logic
Pull request opened by
wild-endeavor
TL;DR
This used to be a problem but is now less so. In early iterations of flytekit, we would run into issues with multiple definitions of flyte entities. They tended to have different definitions and so would trigger a 'duplicate entity with different definition' error in flyte admin. This occurs much less frequently now and this check is causing an issue with map tasks.
We will investigate post 1.5 release how to cache map task definitions so that multiple map_task() declarations on the same underlying task do not result in a new object being created.
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
flyteorg/flytekit
Codecov: 69.90% (-0.02%) compared to 9658b02
✅ 29 other checks have passed
29/30 successful checks