address comments

sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala

@@ -2005,8 +2005,9 @@ class Analyzer(override val catalogManager: CatalogManager) extends RuleExecutor
                 //             analyze the plan. Using a different integer literal may lead to
                 //             a repeat GROUP BY ordinal resolution which is wrong. GROUP BY
                 //             constant is meaningless so whatever value does not matter here.
-                // TODO: GROUP BY ordinal should pull out grouping expressions to a Project, then
-                //       the resolved ordinal expression is always `AttributeReference`.
+                // TODO: (SPARK-45932) GROUP BY ordinal should pull out grouping expressions to
+                //       a Project, then the resolved ordinal expression is always
+                //       `AttributeReference`.
                 case Literal(_: Int, IntegerType) =>
                   Literal(index)
                 case _ => ordinalExpr

sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/analysis/SubstituteUnresolvedOrdinalsSuite.scala

@@ -69,7 +69,7 @@ class SubstituteUnresolvedOrdinalsSuite extends AnalysisTest {
     }
   }
 
-  test("group by ordinal repeated analysis") {
+  test("SPARK-45920: group by ordinal repeated analysis") {
     val plan = testRelation.groupBy(Literal(1))(Literal(100).as("a")).analyze
     comparePlans(
       plan,