[ 
https://issues.apache.org/jira/browse/HBASE-19325?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16266474#comment-16266474
 ] 

Chia-Ping Tsai commented on HBASE-19325:
----------------------------------------

{code}
13:52:18 -1 overall
13:52:18 
13:52:18  _____     _ _                _ 
13:52:18 |  ___|_ _(_) |_   _ _ __ ___| |
13:52:18 | |_ / _` | | | | | | '__/ _ \ |
13:52:18 |  _| (_| | | | |_| | | |  __/_|
13:52:18 |_|  \__,_|_|_|\__,_|_|  \___(_)
13:52:18                                 
13:52:18 
13:52:18 
13:52:18 | Vote |       Subsystem |  Runtime   | Comment
13:52:18 
============================================================================
13:52:18 |      |                 |            | Prechecks 
13:52:18 |  +1  |      hbaseanti  |   0m  0s   | Patch does not have any 
anti-patterns. 
13:52:18 |  +1  |        @author  |   0m  0s   | The patch does not contain any 
@author 
13:52:18 |      |                 |            | tags.
13:52:18 |  +1  |     test4tests  |   0m  0s   | The patch appears to include 1 
new or 
13:52:18 |      |                 |            | modified test files.
13:52:18 |      |                 |            | branch-1 Compile Tests 
13:52:18 |   0  |         mvndep  |   6m  1s   | Maven dependency ordering for 
branch 
13:52:18 |  +1  |     mvninstall  |   1m  2s   | branch-1 passed 
13:52:18 |  +1  |        compile  |   0m 25s   | branch-1 passed 
13:52:18 |  +1  |     checkstyle  |   0m 50s   | branch-1 passed 
13:52:18 |   0  |       findbugs  |   0m 57s   | hbase-server in branch-1 has 1 
extant 
13:52:18 |      |                 |            | Findbugs warnings.
13:52:18 |  +1  |        javadoc  |   0m 32s   | branch-1 passed 
13:52:18 |      |                 |            | Patch Compile Tests 
13:52:18 |   0  |         mvndep  |   0m  6s   | Maven dependency ordering for 
patch 
13:52:18 |  +1  |     mvninstall  |   0m 29s   | the patch passed 
13:52:18 |  +1  |        compile  |   0m 25s   | the patch passed 
13:52:18 |  +1  |          javac  |   0m 25s   | the patch passed 
13:52:19 |  +1  |     checkstyle  |   0m 37s   | the patch passed 
13:52:19 |  +1  |     whitespace  |   0m  0s   | The patch has no whitespace 
issues. 
13:52:19 |  +1  |    hadoopcheck  |   8m 38s   | The patch does not cause any 
errors 
13:52:19 |      |                 |            | with Hadoop 2.4.0 2.4.1 2.5.0 
2.5.1
13:52:19 |      |                 |            | 2.5.2 2.6.1 2.6.2 2.6.3 2.7.1.
13:52:19 |  +1  |       findbugs  |   1m 17s   | the patch passed 
13:52:19 |  +1  |        javadoc  |   0m 18s   | the patch passed 
13:52:19 |      |                 |            | Other Tests 
13:52:19 |  -1  |           unit  |  80m 29s   | hbase-server in the patch 
failed. 
13:52:19 |  +1  |           unit  |   3m 58s   | hbase-rsgroup in the patch 
passed. 
13:52:19 |  +1  |     asflicense  |   0m 19s   | The patch does not generate 
ASF License 
13:52:19 |      |                 |            | warnings.
13:52:19 |      |                 | 107m 21s   | 
13:52:19 
13:52:19 
13:52:19               Reason | Tests
13:52:19  Failed junit tests  |  
hadoop.hbase.client.TestHTableMultiplexerFlushCache 
13:52:19                      |  
hadoop.hbase.replication.TestReplicationSmallTests 
13:52:19                      |  hadoop.hbase.client.TestMetaWithReplicas 
13:52:19                      |  
hadoop.hbase.regionserver.TestEndToEndSplitTransaction 
13:52:19                      |  
hadoop.hbase.mapreduce.TestMultithreadedTableMapper 
13:52:19                      |  
hadoop.hbase.regionserver.TestCompactionInDeadRegionServer 
13:52:19 
13:52:19 
13:52:19 || Subsystem || Report/Notes ||
13:52:19 
============================================================================
13:52:19 | Optional Tests |  asflicense  javac  javadoc  unit  findbugs  
hadoopcheck  hbaseanti  checkstyle  compile  |
13:52:19 | uname | Linux 2060d48154ce 4.10.0-38-generic #42-Ubuntu SMP Tue Oct 
10 13:24:27 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux |
13:52:19 | Build tool | maven |
13:52:19 | Personality | /script/yetus/precommit/personality/hbase.sh |
13:52:19 | git revision | branch-1 / 20d811121f |
13:52:19 | maven | version: Apache Maven 3.3.9 |
13:52:19 | Default Java | 1.8.0_131 |
13:52:19 | findbugs | v3.1.0-RC1 |
13:52:19 | unit | /patchprocess/patch-unit-hbase-server.txt |
13:52:19 | modules | C: hbase-server hbase-rsgroup U: . |
13:52:19 | Powered by | Apache Yetus 0.6.0   http://yetus.apache.org |
{code}
TestCompactionInDeadRegionServer is traced by HBASE-19056.
TestMultithreadedTableMapper -> HBASE-19300
Others fail without the patch.

Will commit it tomorrow if no objections.

> Pass a list of server name to postClearDeadServers
> --------------------------------------------------
>
>                 Key: HBASE-19325
>                 URL: https://issues.apache.org/jira/browse/HBASE-19325
>             Project: HBase
>          Issue Type: Bug
>    Affects Versions: 2.0.0-beta-2
>            Reporter: Guangxu Cheng
>            Assignee: Guangxu Cheng
>             Fix For: 1.4.0, 2.0.0-beta-1
>
>         Attachments: HBASE-19325.branch-1.001.patch, 
> HBASE-19325.branch-1.001.patch, HBASE-19325.branch-2.001.patch
>
>
> Over on the tail of HBASE-18131. [~chia7712] said 
> {quote}
> (Revisiting the AccessController remind me of this issue) 
> Could we remove the duplicate code on the server side? Why not pass a list of 
> server name to postClearDeadServers and postListDeadServers?
> {quote}
> The duplicate code has been removed in HBASE-19131.Now Pass a list of server 
> name to postClearDeadServers



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to