xiangfu0 commented on PR #19091: URL: https://github.com/apache/pinot/pull/19091#issuecomment-5101847657
### Now disjoint from #18875 #18875 has been updated to drop these files, so the two PRs no longer overlap and can be reviewed and merged in either order: - **#19091** (this PR) — UUID scalar functions + multi-stage UDF wrappers. Based on `master`, 15 files. - **#18875** — UUID partitioning (`UuidPartitionFunction`, `PartitionerFactory` / `TableConfigPartitioner` wiring, `UUID_ARRAY` array-function entries), retitled **[UUID 7/8] UUID partitioning**. Doc comments here were also converted to `///` markdown Javadoc to match the surrounding UUID code (`UuidPartitionFunction`, `TableConfigPartitioner`, `DataSchema`). Re-verified on the current `master` base: `pinot-query-runtime -am` `test-compile` succeeds, `UuidConversionFunctionsTest` + `StringFunctionsTest` pass 191/191, and spotless / checkstyle / license are clean. -- 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]
