[SPARK-45849] Remove uneccessary toSeq when encoding Set to catalyst

• Type: Improvement
• Status: Open
• Priority: Minor
• Resolution: Unresolved
• Affects Version/s: 4.0.0
• Component/s: SQL
• Labels: pull-request-available
• Target Version/s: 4.0.0
• Assignee: Unassigned
• Reporter: Emil Ejbyfeldt
• Created: 09/Nov/23 08:33
• Updated: 27/Feb/24 00:18

Description

Currently when encoding Sets to catalyst we first convert them into a Seq. There is no good reason to do this as the interface we are targeting for encoding is only `Iterable` which is implemented by Set. So by using Iterable instead of Seq in some places we should be able to avoid this extra copy when encoding Sets.

Issue Links

• links to: GitHub Pull Request #43734