uncleGen edited a comment on issue #26201: [SPARK-29543][SS][UI] Init 
structured streaming ui
URL: https://github.com/apache/spark/pull/26201#issuecomment-557377114
 
 
   Thanks for your code review. @dongjoon-hyun @xuanyuanking @sarutak 
@HeartSaVioR 
   Here is a summarize of code change after `e30af98`:
   - remove `Last Progress` column in `StreamingQueryPage`.
   - `maxProcessRate` can be NaN. This problem will be fixed in #26610.
   - add a new UI tab, i.e "Structure Streaming", to show the list of streaming 
query. Remove the entrance in `SQL Tab`.
   - the legend area can overlap with the graph area. fixed.
   - itemize the properties of streaming query in its statistics page.
   - add a new "structured-streaming-page.js" to separate logic for `Structured 
Streaming`
   - code clean to reduce deduplicate and redundant work
   - remove redundant lock
   - refactor the `activeStreamingQueries` in `sharedState` to keep inactive 
streaming query.
   - use `<no name>` for those query without name instead of `null`
   - fix some code style
   
   
![image](https://user-images.githubusercontent.com/7402327/69396673-06a6dc00-0d1e-11ea-94a8-546d252e4fce.png)
   
   
![image](https://user-images.githubusercontent.com/7402327/69396988-3b676300-0d1f-11ea-9218-4be245078ef4.png)
   
   
![image](https://user-images.githubusercontent.com/7402327/69396684-10304400-0d1e-11ea-8594-0722e81f39be.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:
us...@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to