Skip to content

Commit

Permalink
[SPARK-20616] RuleExecutor logDebug of batch results should show diff…
Browse files Browse the repository at this point in the history
… to start of batch

## What changes were proposed in this pull request?

Due to a likely typo, the logDebug msg printing the diff of query plans shows a diff to the initial plan, not diff to the start of batch.

## How was this patch tested?

Now the debug message prints the diff between start and end of batch.

Author: Juliusz Sompolski <julek@databricks.com>

Closes apache#17875 from juliuszsompolski/SPARK-20616.
  • Loading branch information
juliuszsompolski authored and rxin committed May 5, 2017
1 parent b31648c commit 5d75b14
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -122,7 +122,7 @@ abstract class RuleExecutor[TreeType <: TreeNode[_]] extends Logging {
logDebug(
s"""
|=== Result of Batch ${batch.name} ===
|${sideBySide(plan.treeString, curPlan.treeString).mkString("\n")}
|${sideBySide(batchStartPlan.treeString, curPlan.treeString).mkString("\n")}
""".stripMargin)
} else {
logTrace(s"Batch ${batch.name} has no effect.")
Expand Down

0 comments on commit 5d75b14

Please sign in to comment.