uncleGen commented on a change in pull request #24609: [SPARK-27715][SQL] SQL 
query details in UI dose not show in correct format.
URL: https://github.com/apache/spark/pull/24609#discussion_r284575528
 
 

 ##########
 File path: 
sql/core/src/main/scala/org/apache/spark/sql/execution/ui/AllExecutionsPage.scala
 ##########
 @@ -382,13 +382,14 @@ private[ui] class ExecutionPagedTable(
   }
 
   private def descriptionCell(execution: SQLExecutionUIData): Seq[Node] = {
+    val jobDescription = UIUtils.makeDescription(execution.description, 
basePath, plainText = false)
     val details = if (execution.details != null && execution.details.nonEmpty) 
{
 
 Review comment:
   @srowen Change to a much safer function. In terms of security, only anchor 
tags with root relative links are supported. So any attempts to embed links 
outside Spark UI, or other tags like "script" will cause in the whole 
description to be treated as plain text.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to