This is an automated email from the ASF dual-hosted git repository.
yashmayya pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new a0df7fa8bc3 Fix typo: occurence -> occurrence in
RegexpReplaceVarFunctions and BenchmarkRegexpReplace (#18803)
a0df7fa8bc3 is described below
commit a0df7fa8bc3dcdb642017c1031bdc49efe83ae8a
Author: Akanksha kedia <[email protected]>
AuthorDate: Fri Jun 19 01:51:45 2026 +0530
Fix typo: occurence -> occurrence in RegexpReplaceVarFunctions and
BenchmarkRegexpReplace (#18803)
---
.../scalar/regexp/RegexpReplaceVarFunctions.java | 24 +++++++++++-----------
.../apache/pinot/perf/BenchmarkRegexpReplace.java | 10 ++++-----
2 files changed, 17 insertions(+), 17 deletions(-)
diff --git
a/pinot-common/src/main/java/org/apache/pinot/common/function/scalar/regexp/RegexpReplaceVarFunctions.java
b/pinot-common/src/main/java/org/apache/pinot/common/function/scalar/regexp/RegexpReplaceVarFunctions.java
index ca32921988e..9da23527c1c 100644
---
a/pinot-common/src/main/java/org/apache/pinot/common/function/scalar/regexp/RegexpReplaceVarFunctions.java
+++
b/pinot-common/src/main/java/org/apache/pinot/common/function/scalar/regexp/RegexpReplaceVarFunctions.java
@@ -32,13 +32,13 @@ public class RegexpReplaceVarFunctions {
/**
* Replace a regular expression pattern. If matchStr is not found, inputStr
will be returned. By default, all
- * occurences of match pattern in the input string will be replaced. Default
matching pattern is case sensitive.
+ * occurrences of match pattern in the input string will be replaced.
Default matching pattern is case sensitive.
*
* @param inputStr Input string to apply the regexpReplace
* @param matchStr Regexp or string to match against inputStr
* @param replaceStr Regexp or string to replace if matchStr is found
* @param matchStartPos Index of inputStr from where matching should start.
Default is 0.
- * @param occurence Controls which occurence of the matched pattern must
be replaced. Counting starts at 0.
+ * @param occurrence Controls which occurrence of the matched pattern
must be replaced. Counting starts at 0.
* Default
* is -1
* @param flag Single character flag that controls how the regex
finds matches in inputStr. If an
@@ -51,20 +51,20 @@ public class RegexpReplaceVarFunctions {
*/
@ScalarFunction
public String regexpReplaceVar(String inputStr, String matchStr, String
replaceStr, int matchStartPos,
- int occurence, String flag) {
+ int occurrence, String flag) {
int patternFlag = "i".equals(flag) ? Pattern.CASE_INSENSITIVE : 0;
Pattern p = Pattern.compile(matchStr, patternFlag);
Matcher matcher = p.matcher(inputStr).region(matchStartPos,
inputStr.length());
- if (occurence >= 0) {
+ if (occurrence >= 0) {
_buffer.setLength(0);
_buffer.append(inputStr);
- while (occurence >= 0 && matcher.find()) {
- if (occurence == 0) {
+ while (occurrence >= 0 && matcher.find()) {
+ if (occurrence == 0) {
_buffer.replace(matcher.start(), matcher.end(), replaceStr);
break;
}
- occurence--;
+ occurrence--;
}
} else {
_buffer.setLength(0);
@@ -79,7 +79,7 @@ public class RegexpReplaceVarFunctions {
/**
* See #regexpReplace(String, String, String, int, int, String). Matches
against entire inputStr and replaces all
- * occurences. Match is performed in case-sensitive mode.
+ * occurrences. Match is performed in case-sensitive mode.
*
* @param inputStr Input string to apply the regexpReplace
* @param matchStr Regexp or string to match against inputStr
@@ -93,7 +93,7 @@ public class RegexpReplaceVarFunctions {
/**
* See #regexpReplace(String, String, String, int, int, String). Matches
against entire inputStr and replaces all
- * occurences. Match is performed in case-sensitive mode.
+ * occurrences. Match is performed in case-sensitive mode.
*
* @param inputStr Input string to apply the regexpReplace
* @param matchStr Regexp or string to match against inputStr
@@ -113,13 +113,13 @@ public class RegexpReplaceVarFunctions {
* @param matchStr Regexp or string to match against inputStr
* @param replaceStr Regexp or string to replace if matchStr is found
* @param matchStartPos Index of inputStr from where matching should start.
Default is 0.
- * @param occurence Controls which occurence of the matched pattern must
be replaced. Counting starts
+ * @param occurrence Controls which occurrence of the matched pattern
must be replaced. Counting starts
* at 0. Default is -1
* @return replaced input string
*/
@ScalarFunction
public String regexpReplaceVar(String inputStr, String matchStr, String
replaceStr, int matchStartPos,
- int occurence) {
- return regexpReplaceVar(inputStr, matchStr, replaceStr, matchStartPos,
occurence, "");
+ int occurrence) {
+ return regexpReplaceVar(inputStr, matchStr, replaceStr, matchStartPos,
occurrence, "");
}
}
diff --git
a/pinot-perf/src/main/java/org/apache/pinot/perf/BenchmarkRegexpReplace.java
b/pinot-perf/src/main/java/org/apache/pinot/perf/BenchmarkRegexpReplace.java
index 3ef2bd00b44..c24218cc6ef 100644
--- a/pinot-perf/src/main/java/org/apache/pinot/perf/BenchmarkRegexpReplace.java
+++ b/pinot-perf/src/main/java/org/apache/pinot/perf/BenchmarkRegexpReplace.java
@@ -93,7 +93,7 @@ public class BenchmarkRegexpReplace {
//old regexp_replace implementation
public static String regexpReplaceOld(String inputStr, String matchStr,
String replaceStr, int matchStartPos,
- int occurence, String flag) {
+ int occurrence, String flag) {
Integer patternFlag;
switch (flag) {
@@ -115,14 +115,14 @@ public class BenchmarkRegexpReplace {
Matcher matcher = p.matcher(inputStr).region(matchStartPos,
inputStr.length());
StringBuffer sb;
- if (occurence >= 0) {
+ if (occurrence >= 0) {
sb = new StringBuffer(inputStr);
- while (occurence >= 0 && matcher.find()) {
- if (occurence == 0) {
+ while (occurrence >= 0 && matcher.find()) {
+ if (occurrence == 0) {
sb.replace(matcher.start(), matcher.end(), replaceStr);
break;
}
- occurence--;
+ occurrence--;
}
} else {
sb = new StringBuffer();
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
