Github user XuTingjun commented on the pull request:

    https://github.com/apache/spark/pull/6545#issuecomment-108708777
  
    I think the reason is not just because of the appCache. After my debug the 
code, I found there are two mainly reasons:
    1. First time send *"/history/appid"* request, the *handler "/history/*"* 
will deal with it. During this, the handler adds the handlers of sparkui into 
historyserver, contains the *handler "/history/appid"*. So when the second time 
send *"/history/appid"* request, the *handler "/history/appid"* will deal with 
it instead of *handler "/history/*"*. so the second time ui is the same with 
the first one;
    
    2. In the *handler "/history/*"*, the code use `appCache.get()` to cache 
app ui. I think we should use `appCache.refresh()` instead to make it can 
refresh.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

Reply via email to