[GitHub] [hadoop] hadoop-yetus commented on pull request #5363: YARN-11424. [Federation] Router Supports DeregisterSubCluster.

2023-04-01 Thread via GitHub
hadoop-yetus commented on PR #5363: URL: https://github.com/apache/hadoop/pull/5363#issuecomment-1493229965 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: | reexe

[GitHub] [hadoop] hadoop-yetus commented on pull request #5447: HDFS-15765. WebHdfs: add support for basic auth and custom API path

2023-04-01 Thread via GitHub
hadoop-yetus commented on PR #5447: URL: https://github.com/apache/hadoop/pull/5447#issuecomment-1493217160 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: | reexe

[GitHub] [hadoop] hadoop-yetus commented on pull request #5517: HDFS-16952. Support getLinkTarget API in WebHDFS

2023-04-01 Thread via GitHub
hadoop-yetus commented on PR #5517: URL: https://github.com/apache/hadoop/pull/5517#issuecomment-1493193692 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: | reexe

[GitHub] [hadoop] hadoop-yetus commented on pull request #5517: HDFS-16952. Support getLinkTarget API in WebHDFS

2023-04-01 Thread via GitHub
hadoop-yetus commented on PR #5517: URL: https://github.com/apache/hadoop/pull/5517#issuecomment-1493192091 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: | reex

[GitHub] [hadoop] hadoop-yetus commented on pull request #5517: HDFS-16952. Support getLinkTarget API in WebHDFS

2023-04-01 Thread via GitHub
hadoop-yetus commented on PR #5517: URL: https://github.com/apache/hadoop/pull/5517#issuecomment-1493143231 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: | reexe

[GitHub] [hadoop] trakos commented on pull request #5447: HDFS-15765. WebHdfs: add support for basic auth and custom API path

2023-04-01 Thread via GitHub
trakos commented on PR #5447: URL: https://github.com/apache/hadoop/pull/5447#issuecomment-1493142645 Thanks for the additional look @saxenapranav! I've applied the suggested changes. Hey @Hexiaoqiao, Pranev mentioned that you could help with further review. Please let me know if you

[GitHub] [hadoop] trakos commented on a diff in pull request #5447: HDFS-15765. WebHdfs: add support for basic auth and custom API path

2023-04-01 Thread via GitHub
trakos commented on code in PR #5447: URL: https://github.com/apache/hadoop/pull/5447#discussion_r1155186841 ## hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/WebHdfsFileSystem.java: ## @@ -307,6 +310,20 @@ public StorageStatistics provide() {

[GitHub] [hadoop] trakos commented on a diff in pull request #5447: HDFS-15765. WebHdfs: add support for basic auth and custom API path

2023-04-01 Thread via GitHub
trakos commented on code in PR #5447: URL: https://github.com/apache/hadoop/pull/5447#discussion_r1155186362 ## hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/BasicAuthConfigurator.java: ## @@ -33,6 +33,16 @@ public class BasicAuthConfigurator im

[GitHub] [hadoop] hadoop-yetus commented on pull request #5517: HDFS-16952. Support getLinkTarget API in WebHDFS

2023-04-01 Thread via GitHub
hadoop-yetus commented on PR #5517: URL: https://github.com/apache/hadoop/pull/5517#issuecomment-1493134974 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: | reexe

[jira] [Created] (HADOOP-18689) Bump jettison from 1.5.3 to 1.5.4 in /hadoop-project

2023-04-01 Thread Ayush Saxena (Jira)
Ayush Saxena created HADOOP-18689: - Summary: Bump jettison from 1.5.3 to 1.5.4 in /hadoop-project Key: HADOOP-18689 URL: https://issues.apache.org/jira/browse/HADOOP-18689 Project: Hadoop Common

[GitHub] [hadoop] hadoop-yetus commented on pull request #5515: YARN-11037. Add configurable logic to split resource request to the least loaded SC.

2023-04-01 Thread via GitHub
hadoop-yetus commented on PR #5515: URL: https://github.com/apache/hadoop/pull/5515#issuecomment-1492969025 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: | reexe

[GitHub] [hadoop] hadoop-yetus commented on pull request #4963: YARN-11326. [Federation] Add RM FederationStateStoreService Metrics.

2023-04-01 Thread via GitHub
hadoop-yetus commented on PR #4963: URL: https://github.com/apache/hadoop/pull/4963#issuecomment-1492945827 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: | reex

[jira] [Commented] (HADOOP-18680) Insufficient heap during full test runs in Docker container.

2023-04-01 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/HADOOP-18680?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17707533#comment-17707533 ] ASF GitHub Bot commented on HADOOP-18680: - slfan1989 commented on PR #5511: URL

[GitHub] [hadoop] slfan1989 commented on pull request #5511: HADOOP-18680: Insufficient heap during full test runs in Docker container.

2023-04-01 Thread via GitHub
slfan1989 commented on PR #5511: URL: https://github.com/apache/hadoop/pull/5511#issuecomment-1492918106 LGTM +1 -- 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 unsubscr

[GitHub] [hadoop] slfan1989 commented on pull request #5491: HDFS-16951. Add description of GETSERVERDEFAULTS to WebHDFS doc

2023-04-01 Thread via GitHub
slfan1989 commented on PR #5491: URL: https://github.com/apache/hadoop/pull/5491#issuecomment-1492915734 @zhtttylz Merged. Thanks for the contribution! -- 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 g

[GitHub] [hadoop] slfan1989 merged pull request #5491: HDFS-16951. Add description of GETSERVERDEFAULTS to WebHDFS doc

2023-04-01 Thread via GitHub
slfan1989 merged PR #5491: URL: https://github.com/apache/hadoop/pull/5491 -- 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...@had

[GitHub] [hadoop] YuanbenWang commented on pull request #5520: HDFS-16965. Add switch to decide whether to enable native codec.

2023-04-01 Thread via GitHub
YuanbenWang commented on PR #5520: URL: https://github.com/apache/hadoop/pull/5520#issuecomment-1492893093 @cnauroth I've updated the PR based on your suggestion, could you review this please? Thanks. -- This is an automated message from the Apache Git Service. To respond to the mes

[GitHub] [hadoop] hadoop-yetus commented on pull request #4963: YARN-11326. [Federation] Add RM FederationStateStoreService Metrics.

2023-04-01 Thread via GitHub
hadoop-yetus commented on PR #4963: URL: https://github.com/apache/hadoop/pull/4963#issuecomment-1492862088 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: | reexe