[SPARK-55113][SQL] EnsureRequirements should copy tags

[peter-toth]

What changes were proposed in this pull request?

This is a minor fix of EnsureRequirements to make sure copied scan nodes inherit the tags from original ones in SPJ functions.

Why are the changes needed?

Fix possible bugs.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Existing tests.

Was this patch authored or co-authored using generative AI tooling?

No.

[github-actions]

JIRA Issue Information

=== Bug SPARK-55113 ===
Summary: EnsureRequirements should copy tags
Assignee: None
Status: Open
Affected: ["4.2.0"]

---

This comment was automatically generated by GitHub Actions

[peter-toth]

cc @dongjoon-hyun

[allisonwang-db]

Can we add a test if possible?

[peter-toth]

Can we add a test if possible?

Unfortunately, I don't have any concrete tests that exposes the bug with not copying the tags in populateCommonPartitionInfo() or in populateJoinKeyPositions().
But this issue came up at #53859 (comment) where a very similar populateNoGroupingPartitionInfo() is added and if it doesn't copy the tags a missing logical link tag causes trouble in some AQE tests.

[peter-toth]

Thanks for the review! Merged to master (4.2.0).

[dongjoon-hyun]

+1, LGTM. Thank you, @peter-toth and @HyukjinKwon .