You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spark.apache.org by rx...@apache.org on 2017/01/17 18:47:49 UTC

spark git commit: [MINOR][SQL] Remove duplicate call of reset() function in CurrentOrigin.withOrigin()

Repository: spark
Updated Branches:
  refs/heads/master 843ec8ec4 -> fee20df14


[MINOR][SQL] Remove duplicate call of reset() function in CurrentOrigin.withOrigin()

## What changes were proposed in this pull request?

Remove duplicate call of reset() function in CurrentOrigin.withOrigin().

## How was this patch tested?

Existing test cases.

Author: jiangxingbo <ji...@gmail.com>

Closes #16615 from jiangxb1987/dummy-code.


Project: http://git-wip-us.apache.org/repos/asf/spark/repo
Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/fee20df1
Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/fee20df1
Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/fee20df1

Branch: refs/heads/master
Commit: fee20df143e4dadf6d72fd25e7de2768ff327e0f
Parents: 843ec8e
Author: jiangxingbo <ji...@gmail.com>
Authored: Tue Jan 17 10:47:46 2017 -0800
Committer: Reynold Xin <rx...@databricks.com>
Committed: Tue Jan 17 10:47:46 2017 -0800

----------------------------------------------------------------------
 .../main/scala/org/apache/spark/sql/catalyst/trees/TreeNode.scala   | 1 -
 1 file changed, 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/spark/blob/fee20df1/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/trees/TreeNode.scala
----------------------------------------------------------------------
diff --git a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/trees/TreeNode.scala b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/trees/TreeNode.scala
index 8cc16d6..8fec9dd 100644
--- a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/trees/TreeNode.scala
+++ b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/trees/TreeNode.scala
@@ -68,7 +68,6 @@ object CurrentOrigin {
   def withOrigin[A](o: Origin)(f: => A): A = {
     set(o)
     val ret = try f finally { reset() }
-    reset()
     ret
   }
 }


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@spark.apache.org
For additional commands, e-mail: commits-help@spark.apache.org