[SPARK-46325][CONNECT] Remove unnecessary override functions when constructing WrappedCloseableIterator in ResponseValidator#wrapIterator

[LuciferYang]

What changes were proposed in this pull request?

This pr removes the overridden hasNext and close functions in the construction of WrappedCloseableIterator in ResponseValidator#wrapIterator, as these functions are identical to those defined in WrappedCloseableIterator.

• WrappedCloseableIterator

spark/connector/connect/common/src/main/scala/org/apache/spark/sql/connect/client/CloseableIterator.scala

Lines 30 to 42 in 9ffdcc3

	private[sql] abstract class WrappedCloseableIterator[E] extends CloseableIterator[E] {
	def innerIterator: Iterator[E]
	override def next(): E = innerIterator.next()
	override def hasNext: Boolean = innerIterator.hasNext
	override def close(): Unit = innerIterator match {
	case it: CloseableIterator[E] => it.close()
	case _ => // nothing
	}
	}

• ResponseValidator#wrapIterator

spark/connector/connect/common/src/main/scala/org/apache/spark/sql/connect/client/ResponseValidator.scala

Lines 62 to 85 in 9ffdcc3

	def wrapIterator[T <: GeneratedMessageV3, V <: CloseableIterator[T]](
	inner: V): WrappedCloseableIterator[T] = {
	new WrappedCloseableIterator[T] {
	override def innerIterator: Iterator[T] = inner
	override def hasNext: Boolean = {
	innerIterator.hasNext
	}
	override def next(): T = {
	verifyResponse {
	innerIterator.next()
	}
	}
	override def close(): Unit = {
	innerIterator match {
	case it: CloseableIterator[T] => it.close()
	case _ => // nothing
	}
	}
	}
	}

Why are the changes needed?

Remove unnecessary override functions.

Does this PR introduce any user-facing change?

No

How was this patch tested?

Pass GitHub Actions

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

No

[dongjoon-hyun]

+1, LGTM.