Skip to content

Commit

Permalink
Revert "[SPARK-31746][YARN][TESTS] Show the actual error message in L…
Browse files Browse the repository at this point in the history
…ocalityPlacementStrategySuite"

This reverts commit cbd8568.
  • Loading branch information
HyukjinKwon committed May 18, 2020
1 parent cbd8568 commit 88e00c3
Showing 1 changed file with 3 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -17,8 +17,7 @@

package org.apache.spark.deploy.yarn

import java.io.{PrintWriter, StringWriter}

import scala.collection.JavaConverters._
import scala.collection.mutable.{HashMap, HashSet, Set}

import org.apache.hadoop.yarn.api.records._
Expand Down Expand Up @@ -46,11 +45,7 @@ class LocalityPlacementStrategySuite extends SparkFunSuite {
thread.start()
thread.join()

if (error != null) {
val errors = new StringWriter()
error.printStackTrace(new PrintWriter(errors))
fail(s"StackOverflowError should not be thrown; however, got:\n\n$errors")
}
assert(error === null)
}

private def runTest(): Unit = {
Expand All @@ -61,6 +56,7 @@ class LocalityPlacementStrategySuite extends SparkFunSuite {
// goal is to create enough requests for localized containers (so there should be many
// tasks on several hosts that have no allocated containers).

val resource = Resource.newInstance(8 * 1024, 4)
val strategy = new LocalityPreferredContainerPlacementStrategy(new SparkConf(),
yarnConf, resource, new MockResolver())

Expand Down

0 comments on commit 88e00c3

Please sign in to comment.