bilaharith commented on a change in pull request #2707:
URL: https://github.com/apache/hadoop/pull/2707#discussion_r613989593
##
File path:
hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/ITestCustomerProvidedKey.java
##
@@ -0,0 +1,937 @@
+/**
+ * License
bilaharith commented on a change in pull request #2707:
URL: https://github.com/apache/hadoop/pull/2707#discussion_r613989746
##
File path:
hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/ITestCustomerProvidedKey.java
##
@@ -0,0 +1,937 @@
+/**
+ * License
bilaharith commented on a change in pull request #2707:
URL: https://github.com/apache/hadoop/pull/2707#discussion_r613990480
##
File path:
hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/ITestCustomerProvidedKey.java
##
@@ -0,0 +1,937 @@
+/**
+ * License
iwasakims merged pull request #2886:
URL: https://github.com/apache/hadoop/pull/2886
--
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.
For queries about this service, pleas
iwasakims commented on pull request #2886:
URL: https://github.com/apache/hadoop/pull/2886#issuecomment-820386755
Thanks, @aajisaka. I merged this.
--
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 t
hadoop-yetus commented on pull request #2889:
URL: https://github.com/apache/hadoop/pull/2889#issuecomment-820392449
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok
hadoop-yetus commented on pull request #2913:
URL: https://github.com/apache/hadoop/pull/2913#issuecomment-820397465
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok
iwasakims commented on pull request #2914:
URL: https://github.com/apache/hadoop/pull/2914#issuecomment-820399933
> I found the latest list:
https://ci.bigtop.apache.org/job/Bigtop-trunk-repos/
Yeah. You can see the discussion on
[BIGTOP-3471](https://issues.apache.org/jira/browse/B
hadoop-yetus commented on pull request #2737:
URL: https://github.com/apache/hadoop/pull/2737#issuecomment-820433086
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok
hadoop-yetus commented on pull request #2520:
URL: https://github.com/apache/hadoop/pull/2520#issuecomment-820436221
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok
tasanuma commented on pull request #2896:
URL: https://github.com/apache/hadoop/pull/2896#issuecomment-820451390
Mostly looks good to me. Is it possible to extend to Router UI as @ayushtkn
commented? (maybe another jira?)
--
This is an automated message from the Apache Git Service.
To re
hadoop-yetus commented on pull request #2907:
URL: https://github.com/apache/hadoop/pull/2907#issuecomment-820500567
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok
tasanuma commented on a change in pull request #2907:
URL: https://github.com/apache/hadoop/pull/2907#discussion_r614222329
##
File path:
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/datanode/fsdataset/impl/FsDatasetCache.java
##
@@ -607,11 +608,
tasanuma commented on pull request #2907:
URL: https://github.com/apache/hadoop/pull/2907#issuecomment-820568403
> Nice one. For metrics and statistics, LongAdder provides much better
throughput.
Agreed with @virajjasani. Thanks for working on this, @tomscut.
--
This is an automat
ayushtkn commented on a change in pull request #2833:
URL: https://github.com/apache/hadoop/pull/2833#discussion_r614224919
##
File path:
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/datanode/DirectoryScanner.java
##
@@ -315,6 +316,40 @@ public D
virajjasani removed a comment on pull request #2895:
URL: https://github.com/apache/hadoop/pull/2895#issuecomment-820126450
Build#8 has timed-out after 20 hr and has these test failures recorded.
Pasting screenshots here as the build data will be archived very soon.
https://user-imag
hadoop-yetus commented on pull request #2520:
URL: https://github.com/apache/hadoop/pull/2520#issuecomment-820684101
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :o
hadoop-yetus commented on pull request #2913:
URL: https://github.com/apache/hadoop/pull/2913#issuecomment-820749035
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok
xiaoyuyao merged pull request #2784:
URL: https://github.com/apache/hadoop/pull/2784
--
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.
For queries about this service, pleas
goiri commented on a change in pull request #2896:
URL: https://github.com/apache/hadoop/pull/2896#discussion_r614449369
##
File path:
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/NetworkTopologyServlet.java
##
@@ -0,0 +1,115 @@
+/**
+ *
goiri merged pull request #2903:
URL: https://github.com/apache/hadoop/pull/2903
--
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.
For queries about this service, please co
hadoop-yetus commented on pull request #2895:
URL: https://github.com/apache/hadoop/pull/2895#issuecomment-820788402
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok
hadoop-yetus commented on pull request #2737:
URL: https://github.com/apache/hadoop/pull/2737#issuecomment-820829126
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok
tomscut commented on a change in pull request #2907:
URL: https://github.com/apache/hadoop/pull/2907#discussion_r614496017
##
File path:
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/datanode/fsdataset/impl/FsDatasetCache.java
##
@@ -607,11 +608,1
qizhu-lucas commented on pull request #2911:
URL: https://github.com/apache/hadoop/pull/2911#issuecomment-820857577
Thanks @fdalsotto for 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
qizhu-lucas commented on pull request #2833:
URL: https://github.com/apache/hadoop/pull/2833#issuecomment-820872414
Thanks very much for @ayushtkn patient review.
I have updated above in latest PR.
--
This is an automated message from the Apache Git Service.
To respond to the message,
zhuxiangyi opened a new pull request #2915:
URL: https://github.com/apache/hadoop/pull/2915
[HDFS-15974](https://issues.apache.org/jira/browse/HDFS-15974)
--
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
aajisaka merged pull request #2914:
URL: https://github.com/apache/hadoop/pull/2914
--
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.
For queries about this service, please
aajisaka commented on pull request #2914:
URL: https://github.com/apache/hadoop/pull/2914#issuecomment-820897074
Merged. Thank you @GauthamBanasandra, @iwasakims, and @goiri.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub an
hadoop-yetus commented on pull request #2915:
URL: https://github.com/apache/hadoop/pull/2915#issuecomment-820903604
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok
langlaile1221 opened a new pull request #2916:
URL: https://github.com/apache/hadoop/pull/2916
When the block report satisfies the block distribution strategy, the block
is removed from the lowredundancyBlocks. But removing the block from the
lowredundancyBlocks is a redundant operation.
surendralilhore merged pull request #2885:
URL: https://github.com/apache/hadoop/pull/2885
--
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.
For queries about this service,
surendralilhore commented on pull request #2842:
URL: https://github.com/apache/hadoop/pull/2842#issuecomment-820920271
Triggered build 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
jianghuazhu commented on pull request #2913:
URL: https://github.com/apache/hadoop/pull/2913#issuecomment-820937321
@virajjasani , thank you very much for your comment.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
qizhu-lucas commented on pull request #2912:
URL: https://github.com/apache/hadoop/pull/2912#issuecomment-820942379
Fixed checkstyle in latest PR.
@bibinchundatt @jbrennan333
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHu
hadoop-yetus commented on pull request #2860:
URL: https://github.com/apache/hadoop/pull/2860#issuecomment-820960349
(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at
https://ci-hadoop.apac
ayushtkn commented on pull request #2895:
URL: https://github.com/apache/hadoop/pull/2895#issuecomment-820963341
Test failures doesn't seems related, mostly due to timeouts, or are known
flaky
--
This is an automated message from the Apache Git Service.
To respond to the message, please
ayushtkn merged pull request #2895:
URL: https://github.com/apache/hadoop/pull/2895
--
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.
For queries about this service, please
surendralilhore commented on a change in pull request #2549:
URL: https://github.com/apache/hadoop/pull/2549#discussion_r614609260
##
File path:
hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azurebfs/AzureBlobFileSystem.java
##
@@ -393,6 +397,32 @@ public boolea
jianghuazhu opened a new pull request #2917:
URL: https://github.com/apache/hadoop/pull/2917
## NOTICE
Please create an issue in ASF JIRA before opening a pull request,
and you need to set the title of the pull request which starts with
the corresponding JIRA issue number. (e.g.
hadoop-yetus commented on pull request #2833:
URL: https://github.com/apache/hadoop/pull/2833#issuecomment-821010516
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok
surendralilhore commented on a change in pull request #2549:
URL: https://github.com/apache/hadoop/pull/2549#discussion_r614661413
##
File path:
hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azurebfs/services/ContentSummaryProcessor.java
##
@@ -0,0 +1,144 @@
+/*
surendralilhore commented on a change in pull request #2549:
URL: https://github.com/apache/hadoop/pull/2549#discussion_r614661413
##
File path:
hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azurebfs/services/ContentSummaryProcessor.java
##
@@ -0,0 +1,144 @@
+/*
surendralilhore commented on a change in pull request #2549:
URL: https://github.com/apache/hadoop/pull/2549#discussion_r614671618
##
File path:
hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azurebfs/services/ContentSummaryProcessor.java
##
@@ -0,0 +1,144 @@
+/*
surendralilhore commented on a change in pull request #2549:
URL: https://github.com/apache/hadoop/pull/2549#discussion_r614674625
##
File path:
hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azurebfs/AzureBlobFileSystem.java
##
@@ -393,6 +397,32 @@ public boolea
hadoop-yetus commented on pull request #2845:
URL: https://github.com/apache/hadoop/pull/2845#issuecomment-821037219
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok
hadoop-yetus commented on pull request #2907:
URL: https://github.com/apache/hadoop/pull/2907#issuecomment-821037653
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok
tomscut commented on a change in pull request #2907:
URL: https://github.com/apache/hadoop/pull/2907#discussion_r614688538
##
File path:
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/datanode/fsdataset/impl/FsDatasetCache.java
##
@@ -607,11 +608,1
whbing opened a new pull request #2918:
URL: https://github.com/apache/hadoop/pull/2918
…elimited format
## NOTICE
Please create an issue in ASF JIRA before opening a pull request,
and you need to set the title of the pull request which starts with
the corresponding JIRA
Hexiaoqiao commented on pull request #2889:
URL: https://github.com/apache/hadoop/pull/2889#issuecomment-821042143
@jojochuang Hi Wei-Chiu, any furthermore comment here? If not, I would like
to commit this PR shortly. Thanks.
--
This is an automated message from the Apache Git Service.
tasanuma commented on a change in pull request #2907:
URL: https://github.com/apache/hadoop/pull/2907#discussion_r614693047
##
File path:
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/datanode/fsdataset/impl/FsDatasetCache.java
##
@@ -607,11 +608,
tomscut commented on a change in pull request #2907:
URL: https://github.com/apache/hadoop/pull/2907#discussion_r614696047
##
File path:
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/datanode/fsdataset/impl/FsDatasetCache.java
##
@@ -607,11 +608,1
tomscut commented on pull request #2896:
URL: https://github.com/apache/hadoop/pull/2896#issuecomment-821056074
> Mostly looks good to me. Is it possible to extend to Router UI as
@ayushtkn commented? (maybe another jira?)
@tasanuma @ayushtkn I'll create another JIRA later to support
hadoop-yetus commented on pull request #2912:
URL: https://github.com/apache/hadoop/pull/2912#issuecomment-821056361
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok
jojochuang commented on pull request #2889:
URL: https://github.com/apache/hadoop/pull/2889#issuecomment-82112
Go ahead. Thanks!
--
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
amahussein opened a new pull request #2919:
URL: https://github.com/apache/hadoop/pull/2919
[HDFS-15979](https://issues.apache.org/jira/browse/HDFS-15979) Move within
EZ fails and cannot remove nested EZs
The test failures are not related to the changes. Those three tests have
relat
hadoop-yetus commented on pull request #2781:
URL: https://github.com/apache/hadoop/pull/2781#issuecomment-821158673
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :o
hadoop-yetus commented on pull request #2916:
URL: https://github.com/apache/hadoop/pull/2916#issuecomment-821161737
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok
hadoop-yetus commented on pull request #2860:
URL: https://github.com/apache/hadoop/pull/2860#issuecomment-821163395
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok
hadoop-yetus commented on pull request #2917:
URL: https://github.com/apache/hadoop/pull/2917#issuecomment-821165566
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok
steveloughran opened a new pull request #2920:
URL: https://github.com/apache/hadoop/pull/2920
Contributed by Steve Loughran.
--
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
steveloughran commented on pull request #2920:
URL: https://github.com/apache/hadoop/pull/2920#issuecomment-821181146
testing: azure cardiff. Test now downgrades to skipping those cases for
which oauth creds are required
--
This is an automated message from the Apache Git Service.
To res
daryn-sharp commented on a change in pull request #2878:
URL: https://github.com/apache/hadoop/pull/2878#discussion_r614865039
##
File path:
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/FSEditLogAsync.java
##
@@ -378,13 +381,18 @@ public
hadoop-yetus commented on pull request #2860:
URL: https://github.com/apache/hadoop/pull/2860#issuecomment-821218156
(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at
https://ci-hadoop.apac
hadoop-yetus commented on pull request #2918:
URL: https://github.com/apache/hadoop/pull/2918#issuecomment-821233568
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok
hadoop-yetus commented on pull request #2920:
URL: https://github.com/apache/hadoop/pull/2920#issuecomment-821247866
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :o
goiri commented on a change in pull request #2915:
URL: https://github.com/apache/hadoop/pull/2915#discussion_r614937252
##
File path:
hadoop-hdfs-project/hadoop-hdfs-rbf/src/main/java/org/apache/hadoop/hdfs/server/federation/metrics/NamenodeBeanMetrics.java
##
@@ -855,7 +855,
Hexiaoqiao merged pull request #2889:
URL: https://github.com/apache/hadoop/pull/2889
--
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.
For queries about this service, plea
Hexiaoqiao commented on pull request #2889:
URL: https://github.com/apache/hadoop/pull/2889#issuecomment-821284155
Committed to trunk. Thanks @zhangshuyan0 for your works! Thanks @jojochuang
for your reviews!
Will backport to other active branches for a while.
--
This is an automated
tomscut commented on a change in pull request #2896:
URL: https://github.com/apache/hadoop/pull/2896#discussion_r614993153
##
File path:
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/NetworkTopologyServlet.java
##
@@ -0,0 +1,115 @@
+/**
+
hadoop-yetus commented on pull request #2907:
URL: https://github.com/apache/hadoop/pull/2907#issuecomment-821314734
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok
uzshao opened a new pull request #2921:
URL: https://github.com/apache/hadoop/pull/2921
See https://issues.apache.org/jira/browse/HDFS-15980
HDFS-15754 introduces 4 new metrics in DataNodeMetrics. However the test
associated with the patch has some bugs. This issue is to fix th
hadoop-yetus commented on pull request #2919:
URL: https://github.com/apache/hadoop/pull/2919#issuecomment-821354712
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok
virajjasani opened a new pull request #2922:
URL: https://github.com/apache/hadoop/pull/2922
--
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.
For queries about this servic
hadoop-yetus commented on pull request #2860:
URL: https://github.com/apache/hadoop/pull/2860#issuecomment-821541081
(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at
https://ci-hadoop.apac
hadoop-yetus commented on pull request #2860:
URL: https://github.com/apache/hadoop/pull/2860#issuecomment-821556485
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok
hadoop-yetus commented on pull request #2918:
URL: https://github.com/apache/hadoop/pull/2918#issuecomment-821559317
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok
hadoop-yetus commented on pull request #2896:
URL: https://github.com/apache/hadoop/pull/2896#issuecomment-821608597
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok
hadoop-yetus commented on pull request #2922:
URL: https://github.com/apache/hadoop/pull/2922#issuecomment-821687427
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok
hadoop-yetus commented on pull request #2921:
URL: https://github.com/apache/hadoop/pull/2921#issuecomment-821743561
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok
hadoop-yetus commented on pull request #2860:
URL: https://github.com/apache/hadoop/pull/2860#issuecomment-821753013
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok
tasanuma merged pull request #2907:
URL: https://github.com/apache/hadoop/pull/2907
--
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.
For queries about this service, please
tasanuma commented on pull request #2907:
URL: https://github.com/apache/hadoop/pull/2907#issuecomment-821784149
Merged it. Thanks for your contribution, @tomscut. Thanks for your reviews,
@virajjasani.
--
This is an automated message from the Apache Git Service.
To respond to the messag
tasanuma commented on pull request #2907:
URL: https://github.com/apache/hadoop/pull/2907#issuecomment-821784876
@tomscut I think this is a good optimization. Would you mind creating
another PR for branch-3.3?
--
This is an automated message from the Apache Git Service.
To respond to the
ayushtkn commented on pull request #2860:
URL: https://github.com/apache/hadoop/pull/2860#issuecomment-821797015
TestDirectoryScanner ran this time and passed:
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2860/15/testReport/org.apache.hadoop.hdfs.server.datanode/TestDirecto
hadoop-yetus commented on pull request #2860:
URL: https://github.com/apache/hadoop/pull/2860#issuecomment-821797317
(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at
https://ci-hadoop.apac
virajjasani opened a new pull request #2923:
URL: https://github.com/apache/hadoop/pull/2923
--
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.
For queries about this servic
Hexiaoqiao commented on a change in pull request #2918:
URL: https://github.com/apache/hadoop/pull/2918#discussion_r615238591
##
File path:
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/tools/offlineImageViewer/PBImageTextWriter.java
##
@@ -649,14 +679,1
Hexiaoqiao commented on a change in pull request #2918:
URL: https://github.com/apache/hadoop/pull/2918#discussion_r615238591
##
File path:
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/tools/offlineImageViewer/PBImageTextWriter.java
##
@@ -649,14 +679,1
Hexiaoqiao commented on pull request #2737:
URL: https://github.com/apache/hadoop/pull/2737#issuecomment-821803540
@functioner would you mind to submit addendum blank commit to trigger CI
again. Just notice that there are many failed unit tests but it seems not
related to this PR(not very
aajisaka merged pull request #2922:
URL: https://github.com/apache/hadoop/pull/2922
--
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.
For queries about this service, please
linyiqun commented on a change in pull request #2737:
URL: https://github.com/apache/hadoop/pull/2737#discussion_r615246158
##
File path:
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/FSEditLogAsync.java
##
@@ -67,6 +77,12 @@
linyiqun commented on pull request #2737:
URL: https://github.com/apache/hadoop/pull/2737#issuecomment-821811306
> @functioner would you mind to submit addendum blank commit to trigger CI
again. Just notice that there are many failed unit tests but it seems not
related to this PR(not very
ayushtkn commented on pull request #2861:
URL: https://github.com/apache/hadoop/pull/2861#issuecomment-821818236
Circling back:
@steveloughran let me know, if that makes sense, if not I can add an extra
option for target as well. or if you have any other suggestions or concerns,
Let me
whbing commented on a change in pull request #2918:
URL: https://github.com/apache/hadoop/pull/2918#discussion_r615265357
##
File path:
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/tools/offlineImageViewer/PBImageTextWriter.java
##
@@ -649,14 +679,123 @
tomscut commented on pull request #2907:
URL: https://github.com/apache/hadoop/pull/2907#issuecomment-821837006
> @tomscut I think this is a good optimization. Would you mind creating
another PR for branch-3.3?
Thanks @tasanuma for the merge. I will create PR for branch-3.3 later.
whbing commented on a change in pull request #2918:
URL: https://github.com/apache/hadoop/pull/2918#discussion_r615265357
##
File path:
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/tools/offlineImageViewer/PBImageTextWriter.java
##
@@ -649,14 +679,123 @
hadoop-yetus commented on pull request #2860:
URL: https://github.com/apache/hadoop/pull/2860#issuecomment-821848462
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok
hadoop-yetus commented on pull request #2923:
URL: https://github.com/apache/hadoop/pull/2923#issuecomment-821861425
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok
hadoop-yetus commented on pull request #2923:
URL: https://github.com/apache/hadoop/pull/2923#issuecomment-821874183
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok
401 - 500 of 62117 matches
Mail list logo