Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/22570#discussion_r220933694
--- Diff:
core/src/main/scala/org/apache/spark/deploy/rest/StandaloneRestServer.scala ---
@@ -91,7 +91,7 @@ private[rest] class
StandaloneStatusRequestServlet(masterEndpoint: RpcEndpointRe
protected def handleStatus(submissionId: String):
SubmissionStatusResponse = {
val response =
masterEndpoint.askSync[DeployMessages.DriverStatusResponse](
DeployMessages.RequestDriverStatus(submissionId))
- val message = response.exception.map { s"Exception from the
cluster:\n" + formatException(_) }
+ val message = response.exception.map { "Exception from the cluster:\n"
+ formatException(_) }
--- End diff --
In cases like this, we should use interpolation in place of concatenation,
if anything. Likewise the previous instance, for example.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
