ggjh-159 opened a new pull request, #12458: URL: https://github.com/apache/gluten/pull/12458
related issue: #12457 ## What changes are proposed in this pull request? Configures the velox4j `FLINK` preset before `Velox4j.initialize()` in both the production TaskManager and the gluten-flink UT harness. Background: velox4j is adding a `FLINK` preset (sibling to the default `SPARK`) so Flink-flavored SQL function registrations stay isolated from Spark's. The default remains `SPARK`, so any caller that wants the Flink behavior must `Velox4j.configure(Preset.KEY, Preset.FLINK)` before `initialize()`. gluten-flink is that caller. ### Changes - `TaskManagerRunner.java` — `Velox4j.configure(Preset.KEY, Preset.FLINK)` immediately before `Velox4j.initialize()` in the Gluten-specific block. - `Velox4jEnvironment.java` — same call in the UT harness constructor. Depends on the upstream velox4j change(bigo-sg/velox4j#54) that introduces the `Preset` enum and the `io.github.zhztheplayer.velox4j.config.Preset` class. ## How was this patch tested? Existing gluten-flink UTs (`Velox4jEnvironment` is the UT init wrapper). End-to-end behavior is verified on the velox4j side by `RegexpExtractTest` running under the FLINK preset. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
