Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/22645#discussion_r223187979
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/ui/AllExecutionsPage.scala
---
@@ -17,16 +17,17 @@
package org.apache.spark.sql.execution.ui
+import java.net.URLEncoder
import javax.servlet.http.HttpServletRequest
+import scala.collection.JavaConverters._
import scala.collection.mutable
-import scala.xml.{Node, NodeSeq}
-
-import org.apache.commons.lang3.StringEscapeUtils
+import scala.xml._
import org.apache.spark.JobExecutionStatus
import org.apache.spark.internal.Logging
-import org.apache.spark.ui.{UIUtils, WebUIPage}
+import org.apache.spark.ui._
--- End diff --
Nit: can we avoid wildcard imports? we usually avoid them unless there are
a very large number of classes to import
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
