Fix more generated code for private stack types

[dain]

Fixes #29126

Release notes

(x) This is not user-visible or is docs only, and no release notes are required.

[gertjanal]

Verified, works 🥳

[wendigo]

(rebased and resolved conflicts after row compiler changes from @raunaqmorarka)

[dain]

@wendigo I read your message wrong, and I rebased an pushed over the branch. Sorry about that

[raunaqmorarka]

@coderabbitai review

[coderabbitai]

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 9f653535-130b-4fd2-a2bd-091fa647499c

📥 Commits

Reviewing files that changed from the base of the PR and between a10f96a and 2f78147.

⛔ Files ignored due to path filters (15)

• core/trino-main/src/main/java/io/trino/sql/gen/AndCodeGenerator.java is excluded by !**/gen/**
• core/trino-main/src/main/java/io/trino/sql/gen/ArrayConstructorCodeGenerator.java is excluded by !**/gen/**
• core/trino-main/src/main/java/io/trino/sql/gen/ArrayMapBytecodeExpression.java is excluded by !**/gen/**
• core/trino-main/src/main/java/io/trino/sql/gen/BetweenCodeGenerator.java is excluded by !**/gen/**
• core/trino-main/src/main/java/io/trino/sql/gen/CoalesceCodeGenerator.java is excluded by !**/gen/**
• core/trino-main/src/main/java/io/trino/sql/gen/DereferenceCodeGenerator.java is excluded by !**/gen/**
• core/trino-main/src/main/java/io/trino/sql/gen/InputReferenceCompiler.java is excluded by !**/gen/**
• core/trino-main/src/main/java/io/trino/sql/gen/IsNullCodeGenerator.java is excluded by !**/gen/**
• core/trino-main/src/main/java/io/trino/sql/gen/LambdaBytecodeGenerator.java is excluded by !**/gen/**
• core/trino-main/src/main/java/io/trino/sql/gen/NullIfCodeGenerator.java is excluded by !**/gen/**
• core/trino-main/src/main/java/io/trino/sql/gen/OrCodeGenerator.java is excluded by !**/gen/**
• core/trino-main/src/main/java/io/trino/sql/gen/RowConstructorCodeGenerator.java is excluded by !**/gen/**
• core/trino-main/src/main/java/io/trino/sql/gen/SqlTypeBytecodeExpression.java is excluded by !**/gen/**
• core/trino-main/src/main/java/io/trino/sql/gen/SwitchCodeGenerator.java is excluded by !**/gen/**
• core/trino-main/src/test/java/io/trino/sql/gen/TestPageFunctionCompiler.java is excluded by !**/gen/**

📒 Files selected for processing (4)

• core/trino-main/src/main/java/io/trino/operator/scalar/ArrayTransformFunction.java
• core/trino-main/src/main/java/io/trino/operator/scalar/MapFilterFunction.java
• core/trino-main/src/main/java/io/trino/operator/scalar/MapTransformKeysFunction.java
• core/trino-main/src/main/java/io/trino/operator/scalar/MapTransformValuesFunction.java

---

📝 Walkthrough

Walkthrough

The pull request modifies four scalar array and map transformation functions to resolve Java element types through CallSiteBinder.getAccessibleType(...) instead of directly using Primitives.wrap(...). This ensures that generated bytecode uses types accessible from the correct class loader context. The corresponding value casting operations are updated to use the binder-resolved types, aligning element casts and variable declarations with the accessible runtime types.

Assessment against linked issues

Objective	Addressed	Explanation
Fix ClassCastException in array/map transformation functions caused by class loader mismatch [#29126]	✅

Out-of-scope changes

No out-of-scope changes detected. All modifications are directly related to addressing the class loader mismatch issue in transformation functions by using binder.getAccessibleType() for type resolution in the generated bytecode.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}