GitHub user JkSelf opened a pull request:

    https://github.com/apache/spark/pull/23204

    Revert "[SPARK-21052][SQL] Add hash map metrics to join"

    Because of the performance degradation discussion in 
[SPARK-26155](https://issues.apache.org/jira/browse/SPARK-26155), currently we 
revert [SPARK-21052](https://issues.apache.org/jira/browse/SPARK-21052)

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/JkSelf/spark revert-21052

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/spark/pull/23204.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #23204
    
----
commit 7d5008d11e37086f4a8206276791b9424bcf60b7
Author: jiake <ke.a.jia@...>
Date:   2018-12-03T08:18:44Z

    Revert "[SPARK-21052][SQL] Add hash map metrics to join"
    
    This reverts commit 18066f2e61f430b691ed8a777c9b4e5786bf9dbc.
    
    Conflicts:
        
sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/HashAggregateExec.scala
        
sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/TungstenAggregationIterator.scala
        
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/BroadcastHashJoinExec.scala
        
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/HashJoin.scala
        
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/HashedRelation.scala
        
sql/core/src/test/scala/org/apache/spark/sql/execution/metric/SQLMetricsSuite.scala

----


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to