Re: [PR] YARN-11361 : Remove jQuery dependency [hadoop]

2025-08-13 Thread via GitHub
Hean-Chhinling commented on PR #7694: URL: https://github.com/apache/hadoop/pull/7694#issuecomment-3183119304 Hello @riya2305, Since Removing JQuery is not needed anymore. Could we close this PR and https://github.com/apache/hadoop/pull/7724? -- This is an automated message from the

Re: [PR] YARN-11361 : Remove jQuery dependency [hadoop]

2025-06-03 Thread via GitHub
hadoop-yetus commented on PR #7694: URL: https://github.com/apache/hadoop/pull/7694#issuecomment-2934800142 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: | reexe

Re: [PR] YARN-11361 : Remove jQuery dependency [hadoop]

2025-06-02 Thread via GitHub
riya2305 commented on PR #7694: URL: https://github.com/apache/hadoop/pull/7694#issuecomment-2931616794 The error is not there when I am testing it. https://github.com/user-attachments/assets/d4881f16-bdca-4fe3-94fd-4739c9786228"; /> -- This is an automated message from the Apa

Re: [PR] YARN-11361 : Remove jQuery dependency [hadoop]

2025-06-02 Thread via GitHub
K0K0V0K commented on PR #7694: URL: https://github.com/apache/hadoop/pull/7694#issuecomment-2931759152 May be some caching? I can see there is a warning on the network tab. If you try to disable cache what will you see? Also maybe try to clear site data on the application tab. https://gi

Re: [PR] YARN-11361 : Remove jQuery dependency [hadoop]

2025-06-02 Thread via GitHub
K0K0V0K commented on PR #7694: URL: https://github.com/apache/hadoop/pull/7694#issuecomment-2931611837 I think the schema is missing from the node address https://github.com/user-attachments/assets/3ad77a0c-381e-4775-91bc-a2de6b316f2e"; /> -- This is an automated message from the A

Re: [PR] YARN-11361 : Remove jQuery dependency [hadoop]

2025-06-02 Thread via GitHub
K0K0V0K commented on PR #7694: URL: https://github.com/apache/hadoop/pull/7694#issuecomment-2931596898 Also when i try to sort i am facing with the following error https://github.com/user-attachments/assets/4ae5862b-8ecc-49ae-b0da-cbbc4959efc6"; /> -- This is an automated message f

Re: [PR] YARN-11361 : Remove jQuery dependency [hadoop]

2025-06-02 Thread via GitHub
K0K0V0K commented on PR #7694: URL: https://github.com/apache/hadoop/pull/7694#issuecomment-2931590609 Thanks for the PR! This will be very useful in our security improvement efforts. When i open the UI i see an error in the console. This is expected? https://github.com/user-att

Re: [PR] YARN-11361 : Remove jQuery dependency [hadoop]

2025-06-02 Thread via GitHub
K0K0V0K commented on PR #7694: URL: https://github.com/apache/hadoop/pull/7694#issuecomment-2930939226 The `hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/webapps/static/jt/jquery.jstree.js` is still needed? -- This is an automated message from the Apache Git Servi

Re: [PR] YARN-11361 : Remove jQuery dependency [hadoop]

2025-06-02 Thread via GitHub
hadoop-yetus commented on PR #7694: URL: https://github.com/apache/hadoop/pull/7694#issuecomment-2930324380 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: | reexe

Re: [PR] YARN-11361 : Remove jQuery dependency [hadoop]

2025-06-02 Thread via GitHub
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

Re: [PR] YARN-11361 : Remove jQuery dependency [hadoop]

2025-05-28 Thread via GitHub
Hean-Chhinling commented on code in PR #7694: URL: https://github.com/apache/hadoop/pull/7694#discussion_r206517 ## hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/webapps/static/yarn.dt.plugins.js: ## @@ -159,7 +159,6 @@ function renderHadoopElapsedTim

Re: [PR] YARN-11361 : Remove jQuery dependency [hadoop]

2025-05-27 Thread via GitHub
hadoop-yetus commented on PR #7694: URL: https://github.com/apache/hadoop/pull/7694#issuecomment-2912441106 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: | reexe

Re: [PR] YARN-11361: Remove jQuery dependency [hadoop]

2025-05-27 Thread via GitHub
riya2305 closed pull request #6965: YARN-11361: Remove jQuery dependency URL: https://github.com/apache/hadoop/pull/6965 -- 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 uns

Re: [PR] YARN-11361 : Remove jQuery dependency [hadoop]

2025-05-27 Thread via GitHub
hadoop-yetus commented on PR #7694: URL: https://github.com/apache/hadoop/pull/7694#issuecomment-2912092034 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: | reexe

Re: [PR] YARN-11361: Remove jQuery dependency [hadoop]

2025-05-19 Thread via GitHub
hadoop-yetus commented on PR #6965: URL: https://github.com/apache/hadoop/pull/6965#issuecomment-2889936984 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: | reexe

Re: [PR] YARN-11361: Remove jQuery dependency [hadoop]

2025-05-19 Thread via GitHub
hadoop-yetus commented on PR #6965: URL: https://github.com/apache/hadoop/pull/6965#issuecomment-2889916025 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: | reexe

Re: [PR] YARN-11361: Remove jQuery dependency [hadoop]

2024-08-16 Thread via GitHub
p-szucs commented on PR #6965: URL: https://github.com/apache/hadoop/pull/6965#issuecomment-2293469457 Hi @riya2305, As we discussed offline, turned out that the previous error was on my side. I tested the change again with the latest codeset, and this time I saw a console error on the S

Re: [PR] YARN-11361: Remove jQuery dependency [hadoop]

2024-08-13 Thread via GitHub
p-szucs commented on PR #6965: URL: https://github.com/apache/hadoop/pull/6965#issuecomment-2285617663 Hi @riya2305, Thank you for this change, I am refreshing my JS memories and looking into it. May I ask if we should check both UI versions regarding this change? I checked out th

Re: [PR] YARN-11361: Remove jQuery dependency [hadoop]

2024-08-06 Thread via GitHub
p-szucs commented on PR #6965: URL: https://github.com/apache/hadoop/pull/6965#issuecomment-2270793033 Hi @riya2305, as I can see that the CI is failing on spotbugs, missing ASF licences and there are some checkstyle issues as well. May I ask you to address them, please? -- This is an au

Re: [PR] YARN-11361: Remove jQuery dependency [hadoop]

2024-08-06 Thread via GitHub
hadoop-yetus commented on PR #6965: URL: https://github.com/apache/hadoop/pull/6965#issuecomment-2270635651 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: | reexe

Re: [PR] YARN-11361: Remove jQuery dependency [hadoop]

2024-07-29 Thread via GitHub
hadoop-yetus commented on PR #6965: URL: https://github.com/apache/hadoop/pull/6965#issuecomment-2255990829 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: | reexe