Jackie-Jiang commented on code in PR #9702: URL: https://github.com/apache/pinot/pull/9702#discussion_r1014576988
########## pinot-common/src/main/java/org/apache/pinot/common/function/scalar/ObjectFunctions.java: ########## @@ -92,13 +94,40 @@ private static Object coalesceVar(Object... objects) { return null; } - @Nullable - private static Object coalesce(Object... objects) { - for (Object o : objects) { - if (o != null) { - return o; + @ScalarFunction + public static Object caseWhen(boolean c1, Object o1, Object oe) { Review Comment: Interesting.. But does that mean these functions won't be recognized by v1 engine because the function name is `case`? -- 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: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org