RH211-sys commented on PR #68029: URL: https://github.com/apache/doris/pull/68029#issuecomment-5696945401
Friendly ping @morrySnow @924060929 @englefly @starocean999 (Code Owners of `fe/fe-core/src/main/java/org/apache/doris/nereids`) and @yiguolei (recent committer on `be/src/exprs/function/math.cpp`). This PR adds the `gamma` scalar function. As a first-time contributor the CI workflows are still waiting for approval, so no checks have started yet; I also posted a request to [email protected]. Could someone approve the workflows and take a look? Local validation: full `sh build.sh --be --fe` on the dev branch, BE UT `MathFunctionTest.gamma_test` passes, the new regression suite `test_gamma` passes in compare mode, and the folded/unfolded values agree. Docs PR: apache/doris-website#4140 Thanks! -- 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]
