Fix CanonicalPlanGenerate to canonicalize without plan ID

[jaystarshot]

Description

Fixes first part of #21449. Would need to canonicalize repeated variables for part2

Motivation and Context

Impact

Test Plan

Contributor checklist

• Please make sure your submission complies with our development, formatting, commit message, and attribution guidelines.
• 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

Please follow release notes guidelines and fill in the release notes below.

== RELEASE NOTES ==

General Changes
* Fix plan canonicalization by canonicalizing plan node ids.

[pranjalssh]

Can you check if it doesn't affect fragment result code as well which uses this codepath

[jaystarshot]

It can because they creates a plan canonical generator there too.

https://github.com/prestodb/presto/blob/master/presto-main/src/main/java/com/facebook/presto/sql/planner/CanonicalPlanGenerator.java#L131.

If we are not sure about the impact there we could not allow this for default strategy.

[pranjalssh]

@jaystarshot that is safer to do without going into more details.

Ideally, we should refactor this class so default strategy goes in a different class, and we reuse some parts from that class here.