Optimize Min/Max using Delta metadata

## Description

Follow up of #1192, which optimizes COUNT. This PR adds support for MIN/MAX as well.

Fix #2092

Created additional unit tests to cover MIN/MAX.

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

Only performance improvement

Closes #1525

Signed-off-by: vkorukanti <[email protected]>
GitOrigin-RevId: 9b88f76bf99cc38bd4cf9d3397b7bb8ade822d0b

Author: felipepessoto

spark/src/main/scala/org/apache/spark/sql/delta/actions/actions.scala

@@ -528,7 +528,7 @@ object Protocol {
     }
     if (manifestGenerationEnabled) {
       // Only allow enabling this, if there are no DVs present.
-      if (!DeletionVectorUtils.isTableDVFree(spark, snapshot)) {
+      if (!DeletionVectorUtils.isTableDVFree(snapshot)) {
         throw new DeltaTablePropertyValidationFailedException(
           table = tableName,
           subClass = ExistingDeletionVectorsWithIncrementalManifestGeneration)

spark/src/main/scala/org/apache/spark/sql/delta/commands/DeletionVectorUtils.scala

@@ -33,7 +33,7 @@ trait DeletionVectorUtils {
    * Run a query on the delta log to determine if the given snapshot contains no deletion vectors.
    * Return `false` if it does contain deletion vectors.
    */
-  def isTableDVFree(spark: SparkSession, snapshot: Snapshot): Boolean = {
+  def isTableDVFree(snapshot: Snapshot): Boolean = {
     val dvsReadable = deletionVectorsReadable(snapshot)
 
     if (dvsReadable) {

spark/src/main/scala/org/apache/spark/sql/delta/hooks/GenerateSymlinkManifest.scala

@@ -245,7 +245,7 @@ trait GenerateSymlinkManifestImpl extends PostCommitHook with DeltaLogging with
   }
 
   protected def assertTableIsDVFree(spark: SparkSession, snapshot: Snapshot): Unit = {
-    if (!isTableDVFree(spark, snapshot)) {
+    if (!isTableDVFree(snapshot)) {
       throw DeltaErrors.generateNotSupportedWithDeletionVectors()
     }
   }