[SPARK-46563][SQL] View simpleString not follow conf spark.sql.debug.maxToStringFields

[AngersZhuuuu]

What changes were proposed in this pull request?

View simpleString not follow conf spark.sql.debug.maxToStringFields

Why are the changes needed?

When columns is too much, the info is noisy. View simpleString not follow conf spark.sql.debug.maxToStringFields

Does this PR introduce any user-facing change?

Yea, view's plan will follow spark.sql.debug.maxToStringFields

How was this patch tested?

No

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

No

[MaxGekk]

@AngersZhuuuu Could you check other places, please. I have found a couple more places where we could restrict the number of outputted fields:

case class ExternalRDDScanExec[T](
...
  override def simpleString(maxFields: Int): String = {
    s"$nodeName${output.mkString("[", ",", "]")}"
  }

and

case class UnresolvedTableSpec(
...
  override def simpleString(maxFields: Int): String = {
    this.copy(properties = Utils.redact(properties).toMap).toString
  }

[MaxGekk]

Waiting for CI.

[AngersZhuuuu]

@AngersZhuuuu Could you check other places, please. I have found a couple more places where we could restrict the number of outputted fields:

case class ExternalRDDScanExec[T](
...
  override def simpleString(maxFields: Int): String = {
    s"$nodeName${output.mkString("[", ",", "]")}"
  }

and

case class UnresolvedTableSpec(
...
  override def simpleString(maxFields: Int): String = {
    this.copy(properties = Utils.redact(properties).toMap).toString
  }

Yea, let me check

[AngersZhuuuu]

Checked all and done

[AngersZhuuuu]

The remain failed GA seems not related?

[MaxGekk]

The remain failed GA seems not related?

Seems not.

+1, LGTM. Merging to master.
Thank you, @AngersZhuuuu and @yaooqinn @HyukjinKwon for review.