[SPARK-39973][CORE] Suppress error logs when the number of timers is set to 0

[HyukjinKwon]

What changes were proposed in this pull request?

This PR proposes to:

• Suppress error logs when the number of timers is set to 0.
• Add documentation for describing its behaviour clearly:
  • 0: disable the feature
  • -1: no limit
  • Otherwise, explicit limit.
• Throw an exception if the configuration is set to the number lower than -1.
  This is an internal configuration, numbers lower than -1 don't make much sense, and this configuration sort of less known. So I think it's pretty safe to do this.

Why are the changes needed?

To avoid noisy error logs, and document the feature properly.

Does this PR introduce any user-facing change?

Yes. When spark.scheduler.listenerbus.metrics.maxListenerClassesTimed is set to 0, it does not show a warning such as:

LiveListenerBusMetrics: Not measuring processing time for listener class org.apache.spark.sql.util.ExecutionListenerBus because a maximum of 0 listener classes are already timed.

How was this patch tested?

Unittest is added.

[dongjoon-hyun]

+1, LGTM. It makes sense and looks safe.

[HyukjinKwon]

Merged to master.

[HyukjinKwon]

thank you guys!