[GitHub] spark pull request #22908: [MONOR][SQL] Replace all TreeNode's node name in ...

2018-11-01 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/22908#discussion_r230060094 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/DataSourceV2Relation.scala --- @@ -56,7 +56,7 @@ case class

[GitHub] spark pull request #22908: [MONOR][SQL] Replace all TreeNode's node name in ...

2018-10-31 Thread wangyum
Github user wangyum commented on a diff in the pull request: https://github.com/apache/spark/pull/22908#discussion_r229761443 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/HashAggregateExec.scala --- @@ -932,12 +932,12 @@ case class HashAggregateExec(

[GitHub] spark pull request #22908: [MONOR][SQL] Replace all TreeNode's node name in ...

2018-10-31 Thread wangyum
GitHub user wangyum opened a pull request: https://github.com/apache/spark/pull/22908 [MONOR][SQL] Replace all TreeNode's node name in the simpleString as their nodeName ## What changes were proposed in this pull request? This pr replace all `TreeNode`'s node name in the