riya2305 commented on code in PR #7694:
URL: https://github.com/apache/hadoop/pull/7694#discussion_r2120311169
##########
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/webapps/static/yarn.dt.plugins.js:
##########
@@ -159,7 +159,6 @@ function renderHadoopElapsedTime(data, type, full) {
return toReturn;
}
// 'sort', 'type' and undefined all just use the number
- return data;
Review Comment:
Thanks for catching that! You're absolutely right - I accidentally removed
the return data statement during my changes. Without it, the function returns
undefined instead of the processed data, which would break any code that
depends on the return value. I'll add it back.
--
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: common-issues-unsubscr...@hadoop.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org
