[SPARK-52008][FOLLOWUP] Fixing StateStoreCoordinator warn compilation messages from batch commit tracking

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

Changing the empty `context.reply()` in the StateStoreCoordinator to use `context.reply(())`

### Why are the changes needed?

This PR: #51706 introduced the following `warn` compilation messages,
```
[warn] /home/runner/work/spark/spark/sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/state/StateStoreCoordinator.scala:397:22: adaptation of an empty argument list by inserting () is deprecated: leaky (Object-receiving) target makes this especially dangerous
[warn]         signature: RpcCallContext.reply(response: Any): Unit
[warn]   given arguments: <none>
[warn]  after adaptation: RpcCallContext.reply((): Unit)
[warn] Applicable -Wconf / nowarn filters for this warning: msg=<part of the message>, cat=deprecation, site=org.apache.spark.sql.execution.streaming.state.StateStoreCoordinator.receiveAndReply, origin=org.apache.spark.rpc.RpcCallContext.reply, version=2.11.0
[warn]         context.reply()
[warn]                      ^
[warn] /home/runner/work/spark/spark/sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/state/StateStoreCoordinator.scala:407:24: adaptation of an empty argument list by inserting () is deprecated: leaky (Object-receiving) target makes this especially dangerous
[warn]         signature: RpcCallContext.reply(response: Any): Unit
[warn]   given arguments: <none>
[warn]  after adaptation: RpcCallContext.reply((): Unit)
[warn] Applicable -Wconf / nowarn filters for this warning: msg=<part of the message>, cat=deprecation, site=org.apache.spark.sql.execution.streaming.state.StateStoreCoordinator.receiveAndReply, origin=org.apache.spark.rpc.RpcCallContext.reply, version=2.11.0
[warn]           context.reply()
[warn]                        ^
[warn] /home/runner/work/spark/spark/sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/state/StateStoreCoordinator.scala:410:24: adaptation of an empty argument list by inserting () is deprecated: leaky (Object-receiving) target makes this especially dangerous
[warn]         signature: RpcCallContext.reply(response: Any): Unit
[warn]   given arguments: <none>
[warn]  after adaptation: RpcCallContext.reply((): Unit)
[warn] Applicable -Wconf / nowarn filters for this warning: msg=<part of the message>, cat=deprecation, site=org.apache.spark.sql.execution.streaming.state.StateStoreCoordinator.receiveAndReply, origin=org.apache.spark.rpc.RpcCallContext.reply, version=2.11.0
[warn]           context.reply()
[warn]                        ^
[warn] /home/runner/work/spark/spark/sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/state/StateStoreCoordinator.scala:420:26: adaptation of an empty argument list by inserting () is deprecated: leaky (Object-receiving) target makes this especially dangerous
[warn]         signature: RpcCallContext.reply(response: Any): Unit
[warn]   given arguments: <none>
[warn]  after adaptation: RpcCallContext.reply((): Unit)
[warn] Applicable -Wconf / nowarn filters for this warning: msg=<part of the message>, cat=deprecation, site=org.apache.spark.sql.execution.streaming.state.StateStoreCoordinator.receiveAndReply, origin=org.apache.spark.rpc.RpcCallContext.reply, version=2.11.0
[warn]             context.reply()
```
By changing these to `context.reply(())` these messages do not show up anymore.

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

No

### How was this patch tested?

By running `build/sbt clean "sql/Test/compile"` locally.

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

No

Closes #51975 from ericm-db/feb-df-consumption-followup.

Authored-by: Eric Marnadi <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>

Author: ericm-db

sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/state/StateStoreCoordinator.scala

@@ -394,7 +394,7 @@ private class StateStoreCoordinator(
         batchCommitTrackers.put(key, new BatchCommitTracker(runId, batchId, expectedStores))
         logInfo(s"Started tracking commits for batch $batchId with " +
           s"${expectedStores.values.map(_.values.sum).sum} expected stores")
-        context.reply()
+        context.reply(())
       }
 
     case ReportStateStoreCommit(storeId, version, storeName) =>
@@ -404,10 +404,10 @@ private class StateStoreCoordinator(
       batchCommitTrackers.get(key) match {
         case Some(tracker) =>
           tracker.recordCommit(storeId, storeName)
-          context.reply()
+          context.reply(())
         case None =>
           // In case no commit tracker for this batch was found
-          context.reply()
+          context.reply(())
       }
 
     case ValidateStateStoreCommitForBatch(runId, batchId) =>
@@ -417,7 +417,7 @@ private class StateStoreCoordinator(
           try {
             tracker.validateAllCommitted()
             batchCommitTrackers.remove(key) // Clean up after validation
-            context.reply()
+            context.reply(())
           } catch {
             case e: StateStoreCommitValidationFailed =>
               batchCommitTrackers.remove(key) // Clean up even on failure