xiangfu0 opened a new pull request, #19934: URL: https://github.com/apache/pinot/pull/19934
NTILE is supported, but the old unknown-function test passed because it supplied no operands and matched a generic constructor-failure wrapper. Rename it to testShouldThrowOnNtileWithoutOperand and require the IllegalArgumentException root cause and the exact arity message. Existing successful NTILE cases remain. Validation was run on the combined test-audit checkout at base `9de9e0115545`; this PR contains only the independent item above, with identical changed files and unchanged production dependencies. - WindowAggregateOperatorTest: all 120 cases passed, including the renamed negative case and successful NTILE execution cases. - Affected-module `spotless:apply`, `license:format`, `checkstyle:check`, and `license:check` passed. - Production/test compilation passed with `maven.compiler.showDeprecation=true` and `maven.compiler.showWarnings=true`; no compiler warnings on added lines. -- 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]
