zhangshuyan0 merged PR #6429:
URL: https://github.com/apache/hadoop/pull/6429
--
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-yetus commented on PR #6429:
URL: https://github.com/apache/hadoop/pull/6429#issuecomment-1898411792
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexe
zhangshuyan0 commented on PR #6429:
URL: https://github.com/apache/hadoop/pull/6429#issuecomment-1897716060
@ThinkerLei Please check the failed tests.
--
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
hadoop-yetus commented on PR #6429:
URL: https://github.com/apache/hadoop/pull/6429#issuecomment-1894507993
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexe
hadoop-yetus commented on PR #6429:
URL: https://github.com/apache/hadoop/pull/6429#issuecomment-1892249873
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexe
zhangshuyan0 commented on PR #6429:
URL: https://github.com/apache/hadoop/pull/6429#issuecomment-1891779812
@ThinkerLei Please fix the checkstyle.
--
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
ThinkerLei commented on PR #6429:
URL: https://github.com/apache/hadoop/pull/6429#issuecomment-1891556272
The failed junit tests hadoop.hdfs.server.datanode.TestDirectoryScanner is
not related to my pull request. I will trigger CI again.
--
This is an automated message from the Apache G
hadoop-yetus commented on PR #6429:
URL: https://github.com/apache/hadoop/pull/6429#issuecomment-1891531391
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexe
hadoop-yetus commented on PR #6429:
URL: https://github.com/apache/hadoop/pull/6429#issuecomment-1889721759
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexe
ThinkerLei commented on PR #6429:
URL: https://github.com/apache/hadoop/pull/6429#issuecomment-1889087759
@zhangshuyan0 I have modified it, please review again in your free time.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHu
ThinkerLei commented on code in PR #6429:
URL: https://github.com/apache/hadoop/pull/6429#discussion_r1450236578
##
hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocolPB/PBHelperClient.java:
##
@@ -795,7 +798,12 @@ static public DatanodeInfo conv
zhangshuyan0 commented on code in PR #6429:
URL: https://github.com/apache/hadoop/pull/6429#discussion_r1450227729
##
hadoop-hdfs-project/hadoop-hdfs-rbf/src/main/java/org/apache/hadoop/hdfs/server/federation/router/RouterClientProtocol.java:
##
@@ -1105,6 +1105,8 @@ private Dat
hadoop-yetus commented on PR #6429:
URL: https://github.com/apache/hadoop/pull/6429#issuecomment-1887688267
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexe
hadoop-yetus commented on PR #6429:
URL: https://github.com/apache/hadoop/pull/6429#issuecomment-1886633693
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexe
ThinkerLei commented on PR #6429:
URL: https://github.com/apache/hadoop/pull/6429#issuecomment-1884330546
@hadoop-yetus trigger CI again
--
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 spec
slfan1989 commented on PR #6429:
URL: https://github.com/apache/hadoop/pull/6429#issuecomment-1884036195
@ThinkerLei Thanks for the contribution! We need to fix the checkstyle
issue.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
16 matches
Mail list logo