[CALCITE-6325] Add LOG function (enabled in Mysql Style library)

Co-authored-by: Tanner Clary <[email protected]>
apache · Apr 1, 2024 · 3a5f076 · 3a5f076
1 parent 22a8767
commit 3a5f076
Show file tree

Hide file tree

Showing 5 changed files with 13 additions and 13 deletions.
diff --git a/core/src/main/java/org/apache/calcite/adapter/enumerable/RexImpTable.java b/core/src/main/java/org/apache/calcite/adapter/enumerable/RexImpTable.java
@@ -211,7 +211,7 @@
 import static org.apache.calcite.sql.fun.SqlLibraryOperators.LOG2;
 import static org.apache.calcite.sql.fun.SqlLibraryOperators.LOGICAL_AND;
 import static org.apache.calcite.sql.fun.SqlLibraryOperators.LOGICAL_OR;
-import static org.apache.calcite.sql.fun.SqlLibraryOperators.LOG_MYSQL;
+import static org.apache.calcite.sql.fun.SqlLibraryOperators.LOG_MYSQL_STYLE;
 import static org.apache.calcite.sql.fun.SqlLibraryOperators.LPAD;
 import static org.apache.calcite.sql.fun.SqlLibraryOperators.MAP;
 import static org.apache.calcite.sql.fun.SqlLibraryOperators.MAP_CONCAT;
@@ -651,7 +651,7 @@ Builder populate() {
 
       LogImplementor logMysqlImplementor =
           new LogImplementor(BuiltInMethod.LOG_MYSQL.method);
-      map.put(LOG_MYSQL, logMysqlImplementor);
+      map.put(LOG_MYSQL_STYLE, logMysqlImplementor);
       map.put(LOG2, logMysqlImplementor);
 
       defineReflective(RAND, BuiltInMethod.RAND.method,

diff --git a/core/src/main/java/org/apache/calcite/runtime/SqlFunctions.java b/core/src/main/java/org/apache/calcite/runtime/SqlFunctions.java
@@ -2790,26 +2790,26 @@ public static double log(BigDecimal d0, BigDecimal d1) {
 
   /** SQL {@code LOG(number, number2)} function applied to double values.
    * but return null when number2 is 0. */
-  public static @Nullable Double logMysql(double number, double number2) {
+  public static @Nullable Double logMysqlStyle(double number, double number2) {
     return (number <= 0) ? null : log(number, number2);
   }
 
   /** SQL {@code LOG(number, number2)} function applied to double and BigDecimal values.
    * but return null when number2 is 0. */
-  public static @Nullable Double logMysql(double number, BigDecimal number2) {
-    return logMysql(number, number2.doubleValue());
+  public static @Nullable Double logMysqlStyle(double number, BigDecimal number2) {
+    return logMysqlStyle(number, number2.doubleValue());
   }
 
   /** SQL {@code LOG(number, number2)} function applied to BigDecimal and double values.
    * but return null when number2 is 0. */
-  public static @Nullable Double logMysql(BigDecimal number, double number2) {
-    return logMysql(number.doubleValue(), number2);
+  public static @Nullable Double logMysqlStyle(BigDecimal number, double number2) {
+    return logMysqlStyle(number.doubleValue(), number2);
   }
 
   /** SQL {@code LOG(number, number2)} function applied to double values.
    * but return null when number2 is 0. */
-  public static @Nullable Double logMysql(BigDecimal number, BigDecimal number2) {
-    return logMysql(number.doubleValue(), number2.doubleValue());
+  public static @Nullable Double logMysqlStyle(BigDecimal number, BigDecimal number2) {
+    return logMysqlStyle(number.doubleValue(), number2.doubleValue());
   }
 
   // MOD

diff --git a/core/src/main/java/org/apache/calcite/sql/fun/SqlLibraryOperators.java b/core/src/main/java/org/apache/calcite/sql/fun/SqlLibraryOperators.java
@@ -2207,7 +2207,7 @@ private static RelDataType deriveTypeMapFromEntries(SqlOperatorBinding opBinding
 
   /** The "LOG(numeric1, numeric2)" function. Returns the base numeric1 logarithm of numeric. */
   @LibraryOperator(libraries = {MYSQL, SPARK})
-  public static final SqlFunction LOG_MYSQL =
+  public static final SqlFunction LOG_MYSQL_STYLE =
       SqlBasicFunction.create("LOG",
           ReturnTypes.DOUBLE_FORCE_NULLABLE,
           OperandTypes.NUMERIC_OPTIONAL_NUMERIC,

diff --git a/core/src/main/java/org/apache/calcite/util/BuiltInMethod.java b/core/src/main/java/org/apache/calcite/util/BuiltInMethod.java
@@ -512,7 +512,7 @@ public enum BuiltInMethod {
   SAFE_MULTIPLY(SqlFunctions.class, "safeMultiply", double.class, double.class),
   SAFE_SUBTRACT(SqlFunctions.class, "safeSubtract", double.class, double.class),
   LOG(SqlFunctions.class, "log", long.class, long.class),
-  LOG_MYSQL(SqlFunctions.class, "logMysql", long.class, long.class),
+  LOG_MYSQL(SqlFunctions.class, "logMysqlStyle", long.class, long.class),
   SEC(SqlFunctions.class, "sec", double.class),
   SECH(SqlFunctions.class, "sech", double.class),
   SIGN(SqlFunctions.class, "sign", long.class),

diff --git a/testkit/src/main/java/org/apache/calcite/test/SqlOperatorTest.java b/testkit/src/main/java/org/apache/calcite/test/SqlOperatorTest.java
@@ -6340,12 +6340,12 @@ void checkRegexpExtract(SqlOperatorFixture f0, FunctionAlias functionAlias) {
 
   /** Test case for
    * <a href="https://issues.apache.org/jira/browse/CALCITE-6325">[CALCITE-6325]
-   * Add LOG function (enabled in Mysql library)</a>. */
+   * Add LOG function (enabled in Mysql Style library)</a>. */
   @Test void testLogMysqlFunc() {
     final SqlOperatorFixture f0 = fixture();
     f0.checkFails("^log(100, 10)^",
         "No match found for function signature LOG\\(<NUMERIC>, <NUMERIC>\\)", false);
-    f0.setFor(SqlLibraryOperators.LOG_MYSQL);
+    f0.setFor(SqlLibraryOperators.LOG_MYSQL_STYLE);
     final Consumer<SqlOperatorFixture> consumer = f -> {
       f.checkScalarApprox("log(10, 10)", "DOUBLE",
           isWithin(1.0, 0.000001));