carloea2 opened a new pull request, #4843:
URL: https://github.com/apache/texera/pull/4843
### What changes were proposed in this PR?
This is PR 2 of the UiParameter split stack, based on PR 1.
This PR adds Scala-side infrastructure for injecting UI parameter values
into Python UDF code, but does not activate it in execution yet.
It includes:
- `PythonUdfUiParameterInjector`.
- Validation for missing attributes, conflicting duplicate names, reserved
hook collisions, and unsupported binary parameter types.
- Scala tests for injection behavior.
- String encoding support for parameter attribute names.
Execution remains unchanged in this PR: Python UDF descriptors still call
`OpExecWithCode(code, "python")`.
```mermaid
sequenceDiagram
autonumber
participant Desc as Scala UDF Descriptor
participant Params as uiParameters Metadata
participant Injector as PythonUdfUiParameterInjector
participant Tests as Injector Specs
participant Exec as Existing Execution Path
Desc->>Params: Holds inert uiParameters
Tests->>Injector: Validate injection behavior
Injector->>Injector: Validate names, types, reserved hook
Injector->>Injector: Generate hook code when called
Desc->>Exec: Still executes original code
Note over Desc,Exec: Injector exists but is not wired into runtime yet
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
