Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/20692#discussion_r171373420
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/HiveExplainSuite.scala
---
@@ -182,9 +182,9 @@ class HiveExplainSuite extends QueryTest with
SQLTestUtils with TestHiveSingleto
|GlobalLimit 1
|+- LocalLimit 1
| +- AnalysisBarrier
- | +- Aggregate [a#0], [a#0, count(1) AS count#0L]
- | +- Project [_1#0 AS a#0, _2#0 AS b#0]
- | +- LocalRelation [_1#0, _2#0]
+ | +- Aggregate [a#0: int], [a#0: int, count(1) AS
count#0L]
+ | +- Project [_1#0: int AS a#0, _2#0: int AS b#0]
--- End diff --
Except the leaf nodes, I think we do not need to add the types for the
other nodes. cc @cloud-fan
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
