agavra commented on code in PR #9901:
URL: https://github.com/apache/pinot/pull/9901#discussion_r1038440166
##########
pinot-common/src/main/java/org/apache/pinot/common/function/scalar/StringFunctions.java:
##########
@@ -97,14 +97,26 @@ public static String substr(String input, int beginIndex) {
* @param endIndex index at which substring should be terminated
* @return substring from beginIndex to endIndex
*/
- @ScalarFunction
+ @ScalarFunction(names = {"substr", "substring"})
public static String substr(String input, int beginIndex, int endIndex) {
if (endIndex == -1) {
return substr(input, beginIndex);
}
return StringUtils.substring(input, beginIndex, endIndex);
}
+ /**
+ * Join two input string with seperator in between
+ * @param input1
+ * @param input2
+ * @param seperator
+ * @return The two input strings joined by the seperator
+ */
+ @ScalarFunction(names = "concat_ws")
+ public static String concatws(String seperator, String input1, String
input2) {
Review Comment:
I don't want to support `contactws`, that's not standard (for the other ones
it was backwards compatibility that I kept the old name)
--
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
