Skip to content
Closed
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
SPARK-20213: Fix SQLListener test.
Opening up metrics in the last commit caused this test to fail.
  • Loading branch information
rdblue committed May 2, 2017
commit a043e5c51a96f31d96691a1431d5f6a266b1734c
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,9 @@ class SQLListenerSuite extends SparkFunSuite with SharedSQLContext with JsonTest

test("basic") {
def checkAnswer(actual: Map[Long, String], expected: Map[Long, Long]): Unit = {
assert(actual.size == expected.size)
// TODO: Remove greater-than case when all metrics are correctly linked into the physical plan
// See SQLListener#getExecutionMetrics
assert(actual.size >= expected.size)
expected.foreach { e =>
// The values in actual can be SQL metrics meaning that they contain additional formatting
// when converted to string. Verify that they start with the expected value.
Expand Down