shahidki31 opened a new pull request #23994: [SPARK-27075] Remove duplicate 
execution tag parameters from the url, when accessing the execution table in 
the SQL page
URL: https://github.com/apache/spark/pull/23994
 
 
   ## What changes were proposed in this pull request?
   
   When we sort any columns of the execution table in the SQL page of the 
WEBUI, throws IllegalArgumentException. The root cause is that,  in the url, we 
are duplicating the execution tag parameters in the 'parameterPath'. Actually 
we should filter out the executionTag related entries while getting the 
'parameterOtherTable'
   
https://github.com/apache/spark/blob/e9e8bb33ef9ad785473ded168bc85867dad4ee70/sql/core/src/main/scala/org/apache/spark/sql/execution/ui/AllExecutionsPage.scala#L161-L163
   
https://github.com/apache/spark/blob/e9e8bb33ef9ad785473ded168bc85867dad4ee70/sql/core/src/main/scala/org/apache/spark/sql/execution/ui/AllExecutionsPage.scala#L241
   
https://github.com/apache/spark/blob/e9e8bb33ef9ad785473ded168bc85867dad4ee70/sql/core/src/main/scala/org/apache/spark/sql/execution/ui/AllExecutionsPage.scala#L263-L266
   
   ## How was this patch tested?
   Manually tested
   Test steps:
   Sort any column in the sql page execution table
   Before fix:
   ![screenshot from 2019-03-07 
01-38-17](https://user-images.githubusercontent.com/23054875/53913261-f0b69580-4080-11e9-88ea-f238b47a21d5.png)
   
   After fix:
   ![screenshot from 2019-03-07 
02-01-40](https://user-images.githubusercontent.com/23054875/53913285-01670b80-4081-11e9-81b6-78cdbf5a0817.png)
   
   
   

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