hadoop-yetus commented on PR #5123:
URL: https://github.com/apache/hadoop/pull/5123#issuecomment-1324647250
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reex
hadoop-yetus commented on PR #5131:
URL: https://github.com/apache/hadoop/pull/5131#issuecomment-1324581269
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexe
[
https://issues.apache.org/jira/browse/HADOOP-18535?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17637577#comment-17637577
]
ASF GitHub Bot commented on HADOOP-18535:
-
hadoop-yetus commented on PR #5159:
hadoop-yetus commented on PR #5158:
URL: https://github.com/apache/hadoop/pull/5158#issuecomment-1324577751
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexe
hadoop-yetus commented on PR #5159:
URL: https://github.com/apache/hadoop/pull/5159#issuecomment-1324577713
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexe
hadoop-yetus commented on PR #5123:
URL: https://github.com/apache/hadoop/pull/5123#issuecomment-1324498190
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reex
slfan1989 commented on PR #5153:
URL: https://github.com/apache/hadoop/pull/5153#issuecomment-1324462049
> LGTM
Thank you very much for your help in reviewing the code!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub an
slfan1989 commented on PR #5152:
URL: https://github.com/apache/hadoop/pull/5152#issuecomment-1324461098
@ayushtkn @aajisaka Can you help to review this PR again? Thank you very
much!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
brahmareddybattula commented on PR #5160:
URL: https://github.com/apache/hadoop/pull/5160#issuecomment-1324454278
@xinglin curious to know, did you observe any connection leak b/w KMS and NN
here when prelaunch is failed..? how does this will help closing the
connection.?
--
This is an a
slfan1989 commented on code in PR #5131:
URL: https://github.com/apache/hadoop/pull/5131#discussion_r1029962161
##
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common/src/test/java/org/apache/hadoop/yarn/server/federation/store/impl/FederationStateStoreBaseTest.
ZanderXu commented on PR #5063:
URL: https://github.com/apache/hadoop/pull/5063#issuecomment-1324449754
Merged. Thanks @haiyang1987 for your report and thanks @goiri for your
review.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to G
ZanderXu merged PR #5063:
URL: https://github.com/apache/hadoop/pull/5063
--
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...@hado
ZanderXu commented on code in PR #5147:
URL: https://github.com/apache/hadoop/pull/5147#discussion_r1029955572
##
hadoop-hdfs-project/hadoop-hdfs-rbf/src/main/java/org/apache/hadoop/hdfs/server/federation/store/driver/impl/StateStoreZooKeeperImpl.java:
##
@@ -84,6 +102,20 @@ pub
slfan1989 commented on PR #5147:
URL: https://github.com/apache/hadoop/pull/5147#issuecomment-1324415952
@howzi Thank you very much for your contribution, but if there are only a
few thousand moutables, this problem should not occur.
--
This is an automated message from the Apache Git Se
ZanderXu commented on PR #4872:
URL: https://github.com/apache/hadoop/pull/4872#issuecomment-1324397973
@xkrogen Sir, thanks for your explanation.
Yeah, ObserverNN also knows the states of DataNodes as well. But the
`scheduledSize` is not shared, and almost all datanode chosen are han
tomscut commented on PR #5125:
URL: https://github.com/apache/hadoop/pull/5125#issuecomment-1324397671
Hi @ZanderXu , could you please fix the SpotBugs warn.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL abov
goiri commented on code in PR #5131:
URL: https://github.com/apache/hadoop/pull/5131#discussion_r1029923406
##
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common/src/test/java/org/apache/hadoop/yarn/server/federation/store/impl/FederationStateStoreBaseTest.java
slfan1989 commented on PR #5130:
URL: https://github.com/apache/hadoop/pull/5130#issuecomment-1324369394
@goiri Thank you very much for helping to review the code!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
UR
[
https://issues.apache.org/jira/browse/HADOOP-18535?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17637501#comment-17637501
]
ASF GitHub Bot commented on HADOOP-18535:
-
hadoop-yetus commented on PR #5159:
hadoop-yetus commented on PR #5159:
URL: https://github.com/apache/hadoop/pull/5159#issuecomment-1324359389
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexe
goiri commented on code in PR #5155:
URL: https://github.com/apache/hadoop/pull/5155#discussion_r1029857848
##
hadoop-hdfs-project/hadoop-hdfs-rbf/src/main/java/org/apache/hadoop/hdfs/tools/federation/RouterAdmin.java:
##
@@ -97,6 +106,7 @@
public class RouterAdmin extends Conf
goiri merged PR #5130:
URL: https://github.com/apache/hadoop/pull/5130
--
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.
hadoop-yetus commented on PR #5145:
URL: https://github.com/apache/hadoop/pull/5145#issuecomment-1324316459
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexe
[
https://issues.apache.org/jira/browse/HADOOP-18535?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Hector Sandoval Chaverri updated HADOOP-18535:
--
Description:
Hadoop RBF supports custom implementations of secret man
[
https://issues.apache.org/jira/browse/HADOOP-18535?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Hector Sandoval Chaverri updated HADOOP-18535:
--
Description:
Hadoop RBF supports custom implementations of secret man
hadoop-yetus commented on PR #5160:
URL: https://github.com/apache/hadoop/pull/5160#issuecomment-1324284486
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexe
hadoop-yetus commented on PR #5142:
URL: https://github.com/apache/hadoop/pull/5142#issuecomment-1324273744
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexe
simbadzina commented on code in PR #5145:
URL: https://github.com/apache/hadoop/pull/5145#discussion_r1029773799
##
hadoop-hdfs-project/hadoop-hdfs-rbf/src/main/java/org/apache/hadoop/hdfs/server/federation/store/driver/impl/StateStoreFileBaseImpl.java:
##
@@ -349,10 +350,12 @@
xinglin commented on PR #5160:
URL: https://github.com/apache/hadoop/pull/5160#issuecomment-1324179644
@xkrogen Can you give a review? đŸ˜„
--
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 specif
xinglin opened a new pull request, #5160:
URL: https://github.com/apache/hadoop/pull/5160
### Description of PR
Swallow IllegalStateException in KeyProviderCache instead of throwing it.
For more details, please check HDFS-16852.
### How was this patch tested?
`mvn te
hadoop-yetus commented on PR #5125:
URL: https://github.com/apache/hadoop/pull/5125#issuecomment-1324166051
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexe
xkrogen commented on PR #4872:
URL: https://github.com/apache/hadoop/pull/4872#issuecomment-1324135159
But the ObserverNN should be aware of the states of DataNodes as well,
right? Staleness, maintenance state, etc. It might be slightly stale, but I
don't immediately see why that would caus
[
https://issues.apache.org/jira/browse/HADOOP-18399?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17637425#comment-17637425
]
ASF GitHub Bot commented on HADOOP-18399:
-
virajjasani commented on PR #5054:
U
virajjasani commented on PR #5054:
URL: https://github.com/apache/hadoop/pull/5054#issuecomment-1324125156
No worries, thank you!
--
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 comm
[
https://issues.apache.org/jira/browse/HADOOP-18146?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17637418#comment-17637418
]
ASF GitHub Bot commented on HADOOP-18146:
-
steveloughran commented on code in P
steveloughran commented on code in PR #4039:
URL: https://github.com/apache/hadoop/pull/4039#discussion_r1029705420
##
hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azurebfs/contracts/services/AppendRequestParameters.java:
##
@@ -72,4 +75,12 @@ public boolean isAp
[
https://issues.apache.org/jira/browse/HADOOP-18535?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17637416#comment-17637416
]
ASF GitHub Bot commented on HADOOP-18535:
-
hchaverri opened a new pull request,
[
https://issues.apache.org/jira/browse/HADOOP-18535?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
ASF GitHub Bot updated HADOOP-18535:
Labels: pull-request-available (was: )
> Implement token storage solution based on MySQL
hchaverri opened a new pull request, #5159:
URL: https://github.com/apache/hadoop/pull/5159
### Description of PR
These changes follow the same logic used in the existing ZK based secret
manager implementation.
1. Minor changes to DelegationTokenInformation class so it's easier
hadoop-yetus commented on PR #5119:
URL: https://github.com/apache/hadoop/pull/5119#issuecomment-1324102873
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexe
omalley commented on code in PR #5145:
URL: https://github.com/apache/hadoop/pull/5145#discussion_r1029681091
##
hadoop-hdfs-project/hadoop-hdfs-rbf/src/main/java/org/apache/hadoop/hdfs/server/federation/store/driver/impl/StateStoreFileBaseImpl.java:
##
@@ -349,10 +350,12 @@ pub
hadoop-yetus commented on PR #5158:
URL: https://github.com/apache/hadoop/pull/5158#issuecomment-1324012152
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexe
hadoop-yetus commented on PR #5123:
URL: https://github.com/apache/hadoop/pull/5123#issuecomment-1324006809
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reex
simbadzina commented on code in PR #5142:
URL: https://github.com/apache/hadoop/pull/5142#discussion_r1029625039
##
hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/NameNodeProxiesClient.java:
##
@@ -349,6 +349,13 @@ public static ClientProtocol
creat
simbadzina commented on code in PR #5145:
URL: https://github.com/apache/hadoop/pull/5145#discussion_r1029624340
##
hadoop-hdfs-project/hadoop-hdfs-rbf/src/test/java/org/apache/hadoop/hdfs/server/federation/store/driver/TestStateStoreDriverBase.java:
##
@@ -234,6 +234,25 @@ publ
[
https://issues.apache.org/jira/browse/HADOOP-18498?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17637363#comment-17637363
]
ASF GitHub Bot commented on HADOOP-18498:
-
steveloughran commented on code in P
steveloughran commented on code in PR #5136:
URL: https://github.com/apache/hadoop/pull/5136#discussion_r1029603628
##
hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/extensions/MockWithPrefixSASTokenProvider.java:
##
@@ -0,0 +1,46 @@
+/**
+ * Licensed to t
slfan1989 commented on PR #5131:
URL: https://github.com/apache/hadoop/pull/5131#issuecomment-1323873187
@goiri Can you help to review this PR again? Thank you very much!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
slfan1989 commented on code in PR #5158:
URL: https://github.com/apache/hadoop/pull/5158#discussion_r1029487315
##
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/ClientRMService.java:
slfan1989 commented on code in PR #5158:
URL: https://github.com/apache/hadoop/pull/5158#discussion_r1029486994
##
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/ClientRMService.java:
slfan1989 commented on PR #5158:
URL: https://github.com/apache/hadoop/pull/5158#issuecomment-1323854909
@curie71 Thanks for your contribution, LGTM.
Can you help to check again whether there is still auditlog missing in the
method of ClientRMService?
--
This is an automated messag
slfan1989 commented on code in PR #5158:
URL: https://github.com/apache/hadoop/pull/5158#discussion_r1029477113
##
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-applications/hadoop-yarn-applications-catalog/hadoop-yarn-applications-catalog-webapp/yarn-error.log:
##
@@ -0,0 +1,52 @
slfan1989 commented on code in PR #5158:
URL: https://github.com/apache/hadoop/pull/5158#discussion_r1029475376
##
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/ClientRMService.java:
curie71 opened a new pull request, #5158:
URL: https://github.com/apache/hadoop/pull/5158
ClientRMService forget to record some audit logs after accessCheck and just
throw an YarnException("User does not have privilege to do something……").
Here is an example in method "getContainers":
hadoop-yetus commented on PR #4602:
URL: https://github.com/apache/hadoop/pull/4602#issuecomment-1323728242
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexe
ZanderXu commented on PR #5098:
URL: https://github.com/apache/hadoop/pull/5098#issuecomment-1323707490
@goiri Sir, can help me review this PR again? Thanks so much.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
ZanderXu commented on PR #4872:
URL: https://github.com/apache/hadoop/pull/4872#issuecomment-1323677572
> Sorry for being late to the party here. The changes to
`getAdditionalBlock` look fine to me -- we already check for `WRITE` operation
later in the same method, this just moves the check
ZanderXu commented on PR #5125:
URL: https://github.com/apache/hadoop/pull/5125#issuecomment-1323653147
Sorry for the late update, @xinglin @aajisaka please help me review it
again. Thanks so much.
--
This is an automated message from the Apache Git Service.
To respond to the message, ple
szilard-nemeth closed pull request #4949: YARN-8262. get_executable in
container-executor should provide meaningful error codes
URL: https://github.com/apache/hadoop/pull/4949
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and u
szilard-nemeth commented on PR #4949:
URL: https://github.com/apache/hadoop/pull/4949#issuecomment-1323617448
Thanks @susheel-gupta for working on this.
Latest patch LGTM, committed to trunk.
Thanks @K0K0V0K for the review.
--
This is an automated message from the Apache Git Serv
hadoop-yetus commented on PR #5157:
URL: https://github.com/apache/hadoop/pull/5157#issuecomment-1323550976
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexe
[
https://issues.apache.org/jira/browse/HADOOP-18523?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Steve Loughran resolved HADOOP-18523.
-
Resolution: Won't Fix
> Allow to retrieve an object from MinIO (S3 API) with a very res
[
https://issues.apache.org/jira/browse/HADOOP-18501?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17637183#comment-17637183
]
ASF GitHub Bot commented on HADOOP-18501:
-
hadoop-yetus commented on PR #5109:
hadoop-yetus commented on PR #5109:
URL: https://github.com/apache/hadoop/pull/5109#issuecomment-1323505745
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reex
hadoop-yetus commented on PR #5142:
URL: https://github.com/apache/hadoop/pull/5142#issuecomment-1323455565
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexe
hadoop-yetus commented on PR #5142:
URL: https://github.com/apache/hadoop/pull/5142#issuecomment-1323448865
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexe
hadoop-yetus commented on PR #5129:
URL: https://github.com/apache/hadoop/pull/5129#issuecomment-1323447710
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexe
[
https://issues.apache.org/jira/browse/HADOOP-18526?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17637132#comment-17637132
]
ASF GitHub Bot commented on HADOOP-18526:
-
steveloughran commented on PR #5144:
steveloughran commented on PR #5144:
URL: https://github.com/apache/hadoop/pull/5144#issuecomment-1323403552
testing ideas
* scale test to create many fs instances in parallel and verify all is
good...need to make sure metrics is turned on first
* add probe method for state of s3a stat
skysiders opened a new pull request, #5157:
URL: https://github.com/apache/hadoop/pull/5157
### Description of PR
When creating "history/done_intermediate", we seem to assume that the parent
directory of the folder exists and has the correct permissions. But when I run
it, it doesn't wor
hadoop-yetus commented on PR #5147:
URL: https://github.com/apache/hadoop/pull/5147#issuecomment-1323369890
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reex
[
https://issues.apache.org/jira/browse/HADOOP-18501?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17637113#comment-17637113
]
ASF GitHub Bot commented on HADOOP-18501:
-
pranavsaxena-microsoft closed pull r
pranavsaxena-microsoft closed pull request #5109: HADOOP-18501: ABFS: Partial
read should add to throttling data: DRAFT
URL: https://github.com/apache/hadoop/pull/5109
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
U
[
https://issues.apache.org/jira/browse/HADOOP-18501?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17637112#comment-17637112
]
ASF GitHub Bot commented on HADOOP-18501:
-
pranavsaxena-microsoft opened a new
[
https://issues.apache.org/jira/browse/HADOOP-18501?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17637111#comment-17637111
]
ASF GitHub Bot commented on HADOOP-18501:
-
pranavsaxena-microsoft closed pull r
pranavsaxena-microsoft closed pull request #5109: HADOOP-18501: ABFS: Partial
read should add to throttling data
URL: https://github.com/apache/hadoop/pull/5109
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL abov
hadoop-yetus commented on PR #5147:
URL: https://github.com/apache/hadoop/pull/5147#issuecomment-1323366400
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reex
hadoop-yetus commented on PR #5147:
URL: https://github.com/apache/hadoop/pull/5147#issuecomment-1323365838
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reex
[
https://issues.apache.org/jira/browse/HADOOP-18536?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
ASF GitHub Bot updated HADOOP-18536:
Labels: pull-request-available (was: )
> RPC Client Improvement
> --
[
https://issues.apache.org/jira/browse/HADOOP-18536?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17637106#comment-17637106
]
ASF GitHub Bot commented on HADOOP-18536:
-
hadoop-yetus commented on PR #5156:
hadoop-yetus commented on PR #5156:
URL: https://github.com/apache/hadoop/pull/5156#issuecomment-1323347401
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexe
81 matches
Mail list logo