[SPARK-50524][SQL] Lower RowBasedKeyValueBatch.spill warning message to debug level

[dongjoon-hyun]

What changes were proposed in this pull request?

This PR aims to lower RowBasedKeyValueBatch.spill warning message to debug level.

   public final long spill(long size, MemoryConsumer trigger) throws IOException {
-    logger.warn("Calling spill() on RowBasedKeyValueBatch. Will not spill but return 0.");
+    logger.debug("Calling spill() on RowBasedKeyValueBatch. Will not spill but return 0.");
     return 0;
   }

Why are the changes needed?

Although Apache Spark has been showing a warning message since 2.1.0, there is nothing for a user to be able to do further. This is more like a dev-side debug message. So, we had better lower the level to DEBUG from WARN.

• [SPARK-16524][SQL] Add RowBatch and RowBasedHashMapGenerator #14349

Does this PR introduce any user-facing change?

No behavior change. This is a log message.

How was this patch tested?

Manual review.

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

No.

[dongjoon-hyun]

Could you review this PR, @viirya ?

[viirya]

Looks good to me.

…

On Mon, Dec 9, 2024 at 2:04 PM Dongjoon Hyun ***@***.***> wrote: Could you review this PR, @viirya <https://github.com/viirya> ? — Reply to this email directly, view it on GitHub <#49116 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAAQZ576KCNM2HUMBT5RVLT2EYHWFAVCNFSM6AAAAABTJVOV4CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMRZGYZDCNJWHE> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[dongjoon-hyun]

Thank you!

[HyukjinKwon]

Merged to master.

[dongjoon-hyun]

Thank you, @HyukjinKwon !