spark icon indicating copy to clipboard operation
spark copied to clipboard

[SPARK-49408][SQL] Use IndexedSeq in ProjectingInternalRow

Open wzx140 opened this issue 1 year ago • 1 comments

What changes were proposed in this pull request?

In ProjectingInternalRow, accessing colOrdinals causes poor performace. Replace colOrdinals with the IndexedSeq type.

Why are the changes needed?

Replace colOrdinals with the IndexedSeq type.

Does this PR introduce any user-facing change?

No

How was this patch tested?

No need to add UT

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

No

wzx140 avatar Aug 27 '24 11:08 wzx140

@aokolnychyi Could you please review it? Really thanks!

wzx140 avatar Aug 27 '24 13:08 wzx140

Merged to master/3.5/3.4.

Thank you all.

yaooqinn avatar Sep 05 '24 07:09 yaooqinn