ajithme commented on a change in pull request #23958: [SPARK-27045] SQL tab in 
UI shows actual SQL instead of callsite
URL: https://github.com/apache/spark/pull/23958#discussion_r264513581
 
 

 ##########
 File path: 
sql/core/src/main/scala/org/apache/spark/sql/execution/ui/AllExecutionsPage.scala
 ##########
 @@ -388,14 +388,15 @@ private[ui] class ExecutionPagedTable(
         +details
       </span> ++
       <div class="stage-details collapsed">
-        <pre>{execution.details}</pre>
+        <pre>{execution.description}<br></br>{execution.details}</pre>
 
 Review comment:
   We cannot know. There are two truncations here 
   1. Backend - Done @ ``SQLExecution#withNewExecutionId`` this is controlled 
by config and if exceeds threshold string will be truncated before posting 
``SparkListenerSQLExecutionStart``.
   2. UI - This gets input via ``SparkListenerSQLExecutionStart`` from status 
store (truncated or may not be truncated) but this may need additional 
truncation based on user window/browser size. this is done by css - onload or 
even window resize will trigger this truncation. This is not configurable. This 
is adjusted as per window size on runtime by css
   

----------------------------------------------------------------
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