Copilot commented on code in PR #1989:
URL: https://github.com/apache/auron/pull/1989#discussion_r2778978398
##########
thirdparty/auron-uniffle/src/main/scala/org/apache/spark/sql/execution/auron/shuffle/uniffle/AuronUniffleShuffleReader.scala:
##########
@@ -164,7 +165,8 @@ class AuronUniffleShuffleReader[K, C](
}
}
if (!emptyPartitionIds.isEmpty) {
- logInfo(s"")
+ logDebug(s"Found ${emptyPartitionIds
+ .size()} empty shuffle partitions:
${emptyPartitionIds.asScala.mkString(",")}")
}
Review Comment:
The empty-partition summary log was changed from `logInfo` to `logDebug`. If
the intent is to make empty partitions observable in default Spark logs (as
described in the issue/PR rationale), `logDebug` may hide this message unless
debug logging is enabled. Consider keeping this as `logInfo` (or,
alternatively, downgrade the per-partition `logInfo` above to debug and keep a
single summary at info) to make the fix effective without excessive log noise.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
