Revert "Improve the merging of operator stats"

[shangm2]

This reverts commit 510024d.

Description

Motivation and Context

Impact

Test Plan

Contributor checklist

• Please make sure your submission complies with our contributing guide, in particular code style and commit standards.
• PR description addresses the issue accurately and concisely. If the change is non-trivial, a GitHub Issue is referenced.
• Documented new properties (with its default value), SQL syntax, functions, or other functionality.
• If release notes are required, they follow the release notes guidelines.
• Adequate tests were added if applicable.
• CI passed.

Release Notes

== NO RELEASE NOTE ==

[facebook-github-bot]

@shangm2 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

[amitkdutta]

@shangm2 Thanks for the revert. Do we need to revert any related PRs (especially for naive engine) that was dependent on this?

[shangm2]

@shangm2 Thanks for the revert. Do we need to revert any related PRs (especially for naive engine) that was dependent on this?

We should be good.

[jaystarshot]

Can you please add more details on why the revert was needed?

[amitkdutta]

Can you please add more details on why the revert was needed?

Upon presto query completion, writtenpartitions is logged to other data lineage services at Meta. We observed that this PR makes writtenpartitions always empty, which breaks the contract. @shangm2 and @pgupta2 can provide further details, and we should update the PR description with more details.