[SPARK-53795][CONNECT] Remove unused parameters in LiteralValueProtoConverter

### What changes were proposed in this pull request?

This PR removes unused parameters from the LiteralValueProtoConverter:
1. **Removed `containsNull` parameter** from the `arrayBuilder` method signature and its usage in pattern matching for `ArrayType`
2. **Removed `valueContainsNull` parameter** from the `mapBuilder` method signature and its usage in pattern matching for `MapType`

### Why are the changes needed?

The `containsNull` and `valueContainsNull` parameters were not being used within the `arrayBuilder` and `mapBuilder` methods.

### Does this PR introduce _any_ user-facing change?

No

### How was this patch tested?

Existing tests

### Was this patch authored or co-authored using generative AI tooling?

Generated-by: Cursor 1.7.28

Closes #52512 from heyihong/SPARK-53795.

Authored-by: Yihong He <[email protected]>
Signed-off-by: Ruifeng Zheng <[email protected]>

Author: heyihong

sql/connect/common/src/main/scala/org/apache/spark/sql/connect/common/LiteralValueProtoConverter.scala

@@ -128,7 +128,7 @@ object LiteralValueProtoConverter {
       options: ToLiteralProtoOptions): proto.Expression.Literal.Builder = {
     val builder = proto.Expression.Literal.newBuilder()
 
-    def arrayBuilder(scalaValue: Any, elementType: DataType, containsNull: Boolean) = {
+    def arrayBuilder(scalaValue: Any, elementType: DataType) = {
       val ab = builder.getArrayBuilder
       scalaValue match {
         case a: Array[_] =>
@@ -148,11 +148,7 @@ object LiteralValueProtoConverter {
       ab
     }
 
-    def mapBuilder(
-        scalaValue: Any,
-        keyType: DataType,
-        valueType: DataType,
-        valueContainsNull: Boolean) = {
+    def mapBuilder(scalaValue: Any, keyType: DataType, valueType: DataType) = {
       val mb = builder.getMapBuilder
       scalaValue match {
         case map: scala.collection.Map[_, _] =>
@@ -214,10 +210,10 @@ object LiteralValueProtoConverter {
         toLiteralProtoBuilderInternal(v.unsafeArray, dataType, options)
       case (v: Array[Byte], ArrayType(_, _)) =>
         toLiteralProtoBuilderInternal(v, options)
-      case (v, ArrayType(elementType, containsNull)) =>
-        builder.setArray(arrayBuilder(v, elementType, containsNull))
-      case (v, MapType(keyType, valueType, valueContainsNull)) =>
-        builder.setMap(mapBuilder(v, keyType, valueType, valueContainsNull))
+      case (v, ArrayType(elementType, _)) =>
+        builder.setArray(arrayBuilder(v, elementType))
+      case (v, MapType(keyType, valueType, _)) =>
+        builder.setMap(mapBuilder(v, keyType, valueType))
       case (v, structType: StructType) =>
         builder.setStruct(structBuilder(v, structType))
       case (v: LocalTime, timeType: TimeType) =>