Manya0407 commented on code in PR #6656:
URL: https://github.com/apache/hive/pull/6656#discussion_r3987043835
##########
service/src/resources/hive-webapps/hiveserver2/llap.html:
##########
@@ -15,8 +15,8 @@
<link rel="stylesheet" type="text/css" href="/static/css/json.human.css">
<script src="/static/js/jquery.min.js"></script>
- <script src="/static/js/json.human.js"></script>
- <script src="/static/js/llap.js"></script>
+ <script src="/static/js/json.human.js?v=1"></script>
+ <script src="/static/js/llap.js?v=2"></script>
Review Comment:
Added ?v= during testing because browsers kept serving old cached
llap.js/json.human.js (still pointing at missing PNGs), so the UI looked broken
until a hard refresh. It's a standard cache-bust for upgrades — doesn't break
anything or change how caching works.
That said, the actual fix is in the JS changes, so it's optional. As Per
feedback I've removed the cache-busting params in the latest push
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
