Github user pgandhi999 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21688#discussion_r227515191
--- Diff: core/src/main/resources/org/apache/spark/ui/static/stagepage.js
---
@@ -52,22 +52,20 @@ $.extend( $.fn.dataTable.ext.type.order, {
function stageEndPoint(appId) {
var urlRegex =
/https\:\/\/[^\/]+\/([^\/]+)\/([^\/]+)\/([^\/]+)?\/?([^\/]+)?\/?([^\/]+)?\/?([^\/]+)?/gm;
var urlArray = urlRegex.exec(document.baseURI);
- var ind = urlArray.indexOf("proxy");
+ var indexOfProxy = urlArray.indexOf("proxy");
var queryString = document.baseURI.split('?');
var words = document.baseURI.split('/');
var stageId = queryString[1].split("&").filter(word =>
word.includes("id="))[0].split("=")[1];
- if (ind > 0) {
+ if (indexOfProxy > 0) {
var appId = urlArray[2];
- var indexOfProxy = words.indexOf("proxy");
- var newBaseURI = words.slice(0, indexOfProxy + 2).join('/');
+ var newBaseURI = words.slice(0, words.indexOf("proxy") +
2).join('/');
return newBaseURI + "/api/v1/applications/" + appId + "/stages/" +
stageId;
}
- ind = urlArray.indexOf("history");
- if (ind > 0) {
+ var indexOfHistory = urlArray.indexOf("history");
+ if (indexOfHistory > 0) {
var appId = urlArray[2];
- var appAttemptId = urlArray[ind + 2];
- var indexOfHistory = words.indexOf("history");
- var newBaseURI = words.slice(0, indexOfHistory).join('/');
+ var appAttemptId = urlArray[indexOfHistory + 2];
--- End diff --
Done
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
