address comments

Signed-off-by: Hongbin Ma (Mahone) <mahongbin@apache.org>

sql-plugin/src/main/scala/com/nvidia/spark/rapids/GpuAggregateExec.scala

@@ -281,16 +281,16 @@ object AggregateUtils extends Logging {
         if (needRepartitionAgain(bucket)) {
           if (recursiveDepth >= maxRecursiveDepth) {
             // Normally this should not happen, because we are repartitioning data that has
-            // already went through first round of aggregation, so there shouldn't be too many
+            // already gone through first round of aggregation, so there shouldn't be too many
             // duplicated rows (the duplication only happens in different batches) to prevent
             // repartitioning out (considering we're changing seed each time we repartition).
-            // However for some test cases with really small batch size, this can happen. So
+            // However, for some test cases with really small batch size, this can happen. So
             // we're just logging some warnings here.
-            log.warn("The bucket is still too large after " + recursiveDepth +
-              " times of repartition. Size for each batch in " +
-              "current bucket: " + bucket.map(_.sizeInBytes).mkString(", ") + " rows: " +
-              bucket.map(_.numRows()).mkString(", ") + " targetMergeBatchSize: "
-              + targetMergeBatchSize)
+            log.warn(s"The bucket is still too large after $recursiveDepth repartitions. " +
+              s"See https://github.com/NVIDIA/spark-rapids/issues/11834. " +
+              s"Sizes for each batch in current bucket: ${bucket.map(_.sizeInBytes).mkString(", ")}"
+              + s" rows: ${bucket.map(_.numRows()).mkString(", ")}" +
+              s" targetMergeBatchSize: $targetMergeBatchSize")
             ArrayBuffer(bucket)
           } else {
             val nextLayerBuckets =