Rename quinary expression in Glow (#635)

Signed-off-by: Henry Davidge <henry@davidge.me>
Co-authored-by: Henry Davidge <henry@davidge.me>

Author: henrydavidge

core/src/main/scala/io/projectglow/sql/expressions/LogisticRegressionExpr.scala

@@ -22,7 +22,7 @@ import org.apache.spark.sql.SQLUtils
 import org.apache.spark.sql.catalyst.InternalRow
 import org.apache.spark.sql.catalyst.analysis.TypeCheckResult
 import org.apache.spark.sql.catalyst.expressions.codegen.{CodegenContext, ExprCode}
-import org.apache.spark.sql.catalyst.expressions.{Expression, ImplicitCastInputTypes, QuinaryExpression}
+import org.apache.spark.sql.catalyst.expressions.{Expression, ImplicitCastInputTypes, QuinaryOptionalExpression}
 import org.apache.spark.sql.catalyst.util.ArrayData
 import org.apache.spark.sql.types.{ArrayType, DataType, DoubleType, StringType}
 import org.apache.spark.unsafe.types.UTF8String
@@ -139,7 +139,7 @@ case class LogisticRegressionExpr(
     covariates: Expression,
     test: Expression,
     offsetOption: Option[Expression])
-    extends QuinaryExpression
+    extends QuinaryOptionalExpression
     with ImplicitCastInputTypes {
 
   def this(

…lyst/expressions/QuinaryExpression.scala …ressions/QuinaryOptionalExpression.scalacore/src/main/scala/org/apache/spark/sql/catalyst/expressions/QuinaryExpression.scala renamed to core/src/main/scala/org/apache/spark/sql/catalyst/expressions/QuinaryOptionalExpression.scala core/src/main/scala/org/apache/spark/sql/catalyst/expressions/QuinaryExpression.scala renamed to core/src/main/scala/org/apache/spark/sql/catalyst/expressions/QuinaryOptionalExpression.scala

@@ -24,7 +24,7 @@ import org.apache.spark.sql.catalyst.expressions.codegen._
  * An expression with four inputs + 5th optional input.
  * The output is by default evaluated to null if any input is evaluated to null.
  */
-abstract class QuinaryExpression extends Expression {
+abstract class QuinaryOptionalExpression extends Expression {
 
   override def foldable: Boolean = children.forall(_.foldable)
 

…expressions/QuinaryExpressionSuite.scala …ons/QuinaryOptionalExpressionSuite.scalacore/src/test/scala/org/apache/spark/sql/catalyst/expressions/QuinaryExpressionSuite.scala renamed to core/src/test/scala/org/apache/spark/sql/catalyst/expressions/QuinaryOptionalExpressionSuite.scala core/src/test/scala/org/apache/spark/sql/catalyst/expressions/QuinaryExpressionSuite.scala renamed to core/src/test/scala/org/apache/spark/sql/catalyst/expressions/QuinaryOptionalExpressionSuite.scala

@@ -27,7 +27,7 @@ case class testQuinaryExpr(
     child3: Expression,
     child4: Expression,
     child5: Option[Expression])
-    extends QuinaryExpression {
+    extends QuinaryOptionalExpression {
 
   override def dataType: DataType = IntegerType
 
@@ -55,7 +55,7 @@ case class testQuinaryExpr(
       child5 = Option(newChildren(5)))
 }
 
-class QuinaryExpressionSuite extends GlowBaseTest {
+class QuinaryOptionalExpressionSuite extends GlowBaseTest {
   test("nullSafeCodeGen for un-nullable expression with Some argument") {
     val ctx = new CodegenContext
     val testExpr = testQuinaryExpr(