fix(tracing): handle noop mcl

[david-leifker]

Handle expected tracing output when no change to record template AND the trace id matches. The difference is that in this case the downstream storage change in ES is also an expected NO_OP.

Also prevent duplicate equality checks when determining the no op state.

Checklist

• The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
• Links to related issues (if applicable)
• Tests for the changes have been added/updated (if applicable)
• Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
• For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Files with missing lines	Coverage Δ
...m/linkedin/metadata/utils/SystemMetadataUtils.java	90.90% <100.00%> (+76.62%)	⬆️
...n/java/com/linkedin/metadata/entity/AspectDao.java	96.07% <100.00%> (+0.16%)	⬆️
...om/linkedin/metadata/entity/EntityServiceImpl.java	79.96% <100.00%> (+0.24%)	⬆️
.../com/linkedin/metadata/trace/TraceServiceImpl.java	85.47% <100.00%> (+1.15%)	⬆️

... and 5 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 16ef1ac...19df5c0. Read the comment docs.