text:
HBASE-20368 Fix RIT stuck when a rsgroup has no online servers but AM…

…'s pendingAssginQueue is cleared

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
83
Docker mode activated.

_ Prechecks _

+1
hbaseanti
0
Patch does not have any anti-patterns.

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 1 new or modified test files.

_ master Compile Tests _

0
mvndep
30
Maven dependency ordering for branch

+1
mvninstall
275
master passed

+1
compile
76
master passed

+1
checkstyle
78
master passed

+1
shadedjars
265
branch has no errors when building our shaded downstream artifacts.

+1
findbugs
260
master passed

+1
javadoc
54
master passed

_ Patch Compile Tests _

0
mvndep
13
Maven dependency ordering for patch

+1
mvninstall
245
the patch passed

+1
compile
79
the patch passed

+1
javac
79
the patch passed

+1
checkstyle
82
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedjars
268
patch has no errors when building our shaded downstream artifacts.

+1
hadoopcheck
755
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.

+1
findbugs
262
the patch passed

+1
javadoc
48
the patch passed

_ Other Tests _

-1
unit
15163
hbase-server in the patch failed.

+1
unit
435
hbase-rsgroup in the patch passed.

+1
asflicense
57
The patch does not generate ASF License warnings.

18873

Reason
Tests

Failed junit tests
hadoop.hbase.master.TestSplitWALManager

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-354/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/354

Optional Tests
dupname  asflicense  javac  javadoc  unit  findbugs  shadedjars  hadoopcheck  hbaseanti  checkstyle  compile

uname
Linux e82ec45ca400 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 GNU/Linux

Build tool
maven

Personality
/testptch/patchprocess/precommit/personality/provided.sh

git revision
master / bdf9d56f2d

maven
version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)

Default Java
1.8.0_181

findbugs
v3.1.11

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-354/1/artifact/out/patch-unit-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-354/1/testReport/

Max. process+thread count
5137 (vs. ulimit of 10000)

modules
C: hbase-server hbase-rsgroup U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-354/1/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

Without the changes, the newly added test throws: ConstraintException: Target RSGroup my_group is same as source Name:my_group,  Servers:[192.168.0.69:60479, 192.168.0.69:60482],  Tables:[Group_testKillAllRSInGroupAndThenAddNew] RSGroup. Is that what we were basing our test to fail upon?

This exception was throwed by which line code? I thougt the test should fail on the last checkRegionsOnline(tableName, true);

@jatsakthi
Thanks for reviewing.

what's the difference between already existing testKillRS in the same class vs the newly added testKillAllRSInGroupAndThenAddNew?

My patch was completed at Apr,2018, while testKillRS was added at Oct,2018. I'm sorry when I didn't care testKillRS when adding testKillAllRSInGroupAndThenAddNew. But I have looked through the two UTs, the difference is that testKillRS disables and enables table to make sure that table regions will be reassigned. But in testKillAllRSInGroupAndThenAddNew, I think table regions should be reassigned and opened correctly while no manual intervention after a none-online-servers rsgroup has online servers again, which may happen in real production environment.

Without the changes, the newly added test throws: ConstraintException: Target RSGroup my_group is same as source Name:my_group, Servers:[192.168.0.69:60479, 192.168.0.69:60482], Tables:[Group_testKillAllRSInGroupAndThenAddNew] RSGroup. Is that what we were basing our test to fail upon?

Also because of codes changing after the patch, there exists new problems.
Firstly, I will answer why throwing this ConstraintException: Target RSGroup ... is the same as ...
Following the UT progress:

add ut_group, which has only one server
move ut_table to ut_group (all regions of table will be on the rsgroup server)
ut_group server crashed
ServerCrashProcedure start, will reassign group table regions, start TransitRegionStateProcedure
assignCandidate is null, and table regions will be assgin to 'lastHost' ,which are in 'default' group
AM process assignment plans by LoadBalancer, which is RSGroupBasedLoadBalancer now, and in which, retainAssignment() will correct misplaced regions,see#205, and ut_table regions are all misplaced(the reason is in step 5, 'lastHost' belongs to 'default' group ) but candidates is empty, so they will be assigned to BOGUS server(localhost,1,1).
AM acceptPlans, will open ut_table regions in the BOGUS server(localhost,1,1).
ut_table regions will definitely failed open，and region location will be set to null to reassign the regions, but until there exist online servers in the ut_group, ut_table regions will be in failed_open state and the location may be null.
move a new server to the ut_group, when check the regions on the new server, we called getRegions(), see#115@RSGroupAdminServer, and at#128, when get location address, will throw NPE.
hbase client receipt Exception and will retry to call moveServer(), and got ConstraintException: Target RSGroup ... is same as source Name...

detailed error logs are:

2019-07-04 13:57:53,911 ERROR [RpcServer.default.FPBQ.Fifo.handler=29,queue=2,port=41041] ipc.RpcServer(432): Unexpected throwable object
java.lang.NullPointerException
at org.apache.hadoop.hbase.rsgroup.RSGroupAdminServer.getRegions(RSGroupAdminServer.java:129)
at org.apache.hadoop.hbase.rsgroup.RSGroupAdminServer.moveServerRegionsFromGroup(RSGroupAdminServer.java:215)
at org.apache.hadoop.hbase.rsgroup.RSGroupAdminServer.moveServers(RSGroupAdminServer.java:326)
at org.apache.hadoop.hbase.rsgroup.RSGroupAdminEndpoint$RSGroupAdminServiceImpl.moveServers(RSGroupAdminEndpoint.java:218)
at org.apache.hadoop.hbase.protobuf.generated.RSGroupAdminProtos$RSGroupAdminService.callMethod(RSGroupAdminProtos.java:13870)
at org.apache.hadoop.hbase.master.MasterRpcServices.execMasterService(MasterRpcServices.java:889)
at org.apache.hadoop.hbase.shaded.protobuf.generated.MasterProtos$MasterService$2.callBlockingMethod(MasterProtos.java)
at org.apache.hadoop.hbase.ipc.RpcServer.call(RpcServer.java:374)
at org.apache.hadoop.hbase.ipc.CallRunner.run(CallRunner.java:132)
at org.apache.hadoop.hbase.ipc.RpcExecutor$Handler.run(RpcExecutor.java:338)
at org.apache.hadoop.hbase.ipc.RpcExecutor$Handler.run(RpcExecutor.java:318)
2019-07-04 13:57:53,912 DEBUG [RpcServer.default.FPBQ.Fifo.handler=29,queue=2,port=41041] ipc.CallRunner(144): callId: 33 service: MasterService methodName: ExecMasterService size: 104 connection: 127.0.0.1:38874 deadline: 1562219933867, exception=java.io.IOException
2019-07-04 13:57:54,025 INFO  [RpcServer.default.FPBQ.Fifo.handler=29,queue=2,port=41041] rsgroup.RSGroupAdminEndpoint$RSGroupAdminServiceImpl(211): Client=haxiaolin//127.0.0.1 move servers [localhost:41689] to rsgroup my_group
2019-07-04 13:57:54,025 DEBUG [RpcServer.default.FPBQ.Fifo.handler=29,queue=2,port=41041] ipc.CallRunner(144): callId: 34 service: MasterService methodName: ExecMasterService size: 104 connection: 127.0.0.1:38874 deadline: 1562219934024, exception=org.apache.hadoop.hbase.constraint.ConstraintException: Target RSGroup my_group is same as source Name:my_group,  Servers:[localhost:36041, localhost:41689],  Tables:[Group_testKillAllRSInGroupAndThenAddNew] RSGroup.
org.apache.hadoop.hbase.constraint.ConstraintException: org.apache.hadoop.hbase.constraint.ConstraintException: Target RSGroup my_group is same as source Name:my_group,  Servers:[localhost:36041, localhost:41689],  Tables:[Group_testKillAllRSInGroupAndThenAddNew] RSGroup.
at org.apache.hadoop.hbase.rsgroup.RSGroupAdminServer.moveServers(RSGroupAdminServer.java:299)
at org.apache.hadoop.hbase.rsgroup.RSGroupAdminEndpoint$RSGroupAdminServiceImpl.moveServers(RSGroupAdminEndpoint.java:218)
at org.apache.hadoop.hbase.protobuf.generated.RSGroupAdminProtos$RSGroupAdminService.callMethod(RSGroupAdminProtos.java:13870)

I think my patch works well because I added BOGUS check in AM, and will retry to reassign these regions instead of accepting these plans and opening them on the BOGUS server.
But patch changes of RSGroupBasedLoadBalancer is useless in this case. May it should be deleted because queueAssgin()#TransitRegionStateProcedure was changed by Duo after my patch, and regions will always be misplaced when its last location is out of group and is reassigned. The process of misplaced regions in RSGroupBasedLoadBalancer I think is OK.

@jatsakthi I hava fixed the NPE produced by the UT I uploaded yesterday, and when only run the UT except the fixed server codes, it will pass, because when 'lastHost' is not in rsgroup, regions will be processed as misplaced, and TRSP will retry to confirm open and reassign it. But if we make 'lastHost' in our group, like the UT I updated right now, there will be rit stuck. I have replied in HBASE-20728 you mentioned before. If you have any concerns or problems, we can discuss more.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
31
Docker mode activated.

_ Prechecks _

+1
hbaseanti
0
Patch does not have any anti-patterns.

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 1 new or modified test files.

_ master Compile Tests _

0
mvndep
29
Maven dependency ordering for branch

+1
mvninstall
269
master passed

+1
compile
81
master passed

+1
checkstyle
83
master passed

+1
shadedjars
267
branch has no errors when building our shaded downstream artifacts.

+1
findbugs
239
master passed

+1
javadoc
49
master passed

_ Patch Compile Tests _

0
mvndep
15
Maven dependency ordering for patch

+1
mvninstall
232
the patch passed

+1
compile
75
the patch passed

+1
javac
75
the patch passed

+1
checkstyle
78
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedjars
263
patch has no errors when building our shaded downstream artifacts.

+1
hadoopcheck
733
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.

+1
findbugs
262
the patch passed

+1
javadoc
53
the patch passed

_ Other Tests _

+1
unit
8126
hbase-server in the patch passed.

+1
unit
211
hbase-rsgroup in the patch passed.

+1
asflicense
56
The patch does not generate ASF License warnings.

11486

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-354/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/354

Optional Tests
dupname  asflicense  javac  javadoc  unit  findbugs  shadedjars  hadoopcheck  hbaseanti  checkstyle  compile

uname
Linux 24b7a7978722 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 GNU/Linux

Build tool
maven

Personality
/testptch/patchprocess/precommit/personality/provided.sh

git revision
master / 9116534f5d

maven
version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)

Default Java
1.8.0_181

findbugs
v3.1.11

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-354/2/testReport/

Max. process+thread count
4980 (vs. ulimit of 10000)

modules
C: hbase-server hbase-rsgroup U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-354/2/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

@jatsakthi  Running the UT I updated just a moment ago without any change of RSGroupBasedLoadBalancer will fail at TestRSGroupsKillRS#207, which is caused by the wrong start timestamp of regionserver restarted, as mentioned in HBASE-20728. You can try it. Thank you.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
24
Docker mode activated.

_ Prechecks _

+1
hbaseanti
0
Patch does not have any anti-patterns.

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 2 new or modified test files.

_ master Compile Tests _

0
mvndep
26
Maven dependency ordering for branch

+1
mvninstall
241
master passed

+1
compile
76
master passed

+1
checkstyle
80
master passed

+1
shadedjars
260
branch has no errors when building our shaded downstream artifacts.

+1
findbugs
248
master passed

+1
javadoc
49
master passed

_ Patch Compile Tests _

0
mvndep
14
Maven dependency ordering for patch

+1
mvninstall
226
the patch passed

+1
compile
74
the patch passed

+1
javac
74
the patch passed

-1
checkstyle
10
hbase-rsgroup: The patch generated 3 new + 2 unchanged - 0 fixed = 5 total (was 2)

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedjars
262
patch has no errors when building our shaded downstream artifacts.

+1
hadoopcheck
708
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.

+1
findbugs
260
the patch passed

+1
javadoc
52
the patch passed

_ Other Tests _

+1
unit
8450
hbase-server in the patch passed.

+1
unit
239
hbase-rsgroup in the patch passed.

+1
asflicense
42
The patch does not generate ASF License warnings.

11720

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-354/3/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/354

Optional Tests
dupname  asflicense  javac  javadoc  unit  findbugs  shadedjars  hadoopcheck  hbaseanti  checkstyle  compile

uname
Linux 6480f46ad6fa 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 GNU/Linux

Build tool
maven

Personality
/testptch/patchprocess/precommit/personality/provided.sh

git revision
master / 605f8a15bb

maven
version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)

Default Java
1.8.0_181

findbugs
v3.1.11

checkstyle
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-354/3/artifact/out/diff-checkstyle-hbase-rsgroup.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-354/3/testReport/

Max. process+thread count
4687 (vs. ulimit of 10000)

modules
C: hbase-server hbase-rsgroup U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-354/3/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
23
Docker mode activated.

_ Prechecks _

+1
hbaseanti
0
Patch does not have any anti-patterns.

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 2 new or modified test files.

_ master Compile Tests _

0
mvndep
13
Maven dependency ordering for branch

+1
mvninstall
233
master passed

+1
compile
72
master passed

+1
checkstyle
81
master passed

+1
shadedjars
264
branch has no errors when building our shaded downstream artifacts.

+1
findbugs
245
master passed

+1
javadoc
51
master passed

_ Patch Compile Tests _

0
mvndep
15
Maven dependency ordering for patch

+1
mvninstall
232
the patch passed

+1
compile
73
the patch passed

+1
javac
73
the patch passed

+1
checkstyle
78
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedjars
264
patch has no errors when building our shaded downstream artifacts.

+1
hadoopcheck
821
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.

+1
findbugs
263
the patch passed

+1
javadoc
52
the patch passed

_ Other Tests _

+1
unit
8133
hbase-server in the patch passed.

+1
unit
222
hbase-rsgroup in the patch passed.

+1
asflicense
54
The patch does not generate ASF License warnings.

11510

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-354/4/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/354

Optional Tests
dupname  asflicense  javac  javadoc  unit  findbugs  shadedjars  hadoopcheck  hbaseanti  checkstyle  compile

uname
Linux 1267e668b363 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 GNU/Linux

Build tool
maven

Personality
/testptch/patchprocess/precommit/personality/provided.sh

git revision
master / 605f8a15bb

maven
version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)

Default Java
1.8.0_181

findbugs
v3.1.11

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-354/4/testReport/

Max. process+thread count
4675 (vs. ulimit of 10000)

modules
C: hbase-server hbase-rsgroup U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-354/4/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

@jatsakthi I have updated testKillAllRSInGroup() to cover both null lastHost and group-server lastHost. Please review again.

However I still doubt about the requirement of the testKillRS() which just stops a rs from the group and moves one rs from the default group to the custom group. (However, both the restart & move case has been covered in testKillAllRSInGroup()). Are we trying to keep that test alive for the case where the lastHost is null?

In testKillRS(), lastHost should not be null, because when creating test table, it adds group into to namespace config, as a result, all the table regions will be on group servers at the beginning, which is different from my moving table to group to make table regions on group servers.
And I think this test case used disable/enable table to enforce reassign, which is different from my auto-reassign. This can be kept for testing force-reassign?

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
169
Docker mode activated.

_ Prechecks _

+1
hbaseanti
0
Patch does not have any anti-patterns.

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 2 new or modified test files.

_ master Compile Tests _

0
mvndep
25
Maven dependency ordering for branch

+1
mvninstall
234
master passed

+1
compile
73
master passed

+1
checkstyle
74
master passed

+1
shadedjars
250
branch has no errors when building our shaded downstream artifacts.

+1
findbugs
238
master passed

+1
javadoc
48
master passed

_ Patch Compile Tests _

0
mvndep
14
Maven dependency ordering for patch

+1
mvninstall
219
the patch passed

+1
compile
74
the patch passed

+1
javac
74
the patch passed

+1
checkstyle
79
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedjars
262
patch has no errors when building our shaded downstream artifacts.

+1
hadoopcheck
692
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.

+1
findbugs
251
the patch passed

+1
javadoc
48
the patch passed

_ Other Tests _

-1
unit
13982
hbase-server in the patch failed.

+1
unit
507
hbase-rsgroup in the patch passed.

+1
asflicense
44
The patch does not generate ASF License warnings.

17580

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-354/5/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/354

Optional Tests
dupname  asflicense  javac  javadoc  unit  findbugs  shadedjars  hadoopcheck  hbaseanti  checkstyle  compile

uname
Linux c4d444d1664e 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 GNU/Linux

Build tool
maven

Personality
/testptch/patchprocess/precommit/personality/provided.sh

git revision
master / 605f8a15bb

maven
version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)

Default Java
1.8.0_181

findbugs
v3.1.11

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-354/5/artifact/out/patch-unit-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-354/5/testReport/

Max. process+thread count
5148 (vs. ulimit of 10000)

modules
C: hbase-server hbase-rsgroup U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-354/5/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

The failed UT is not relevant to this patch, its an ACL issue.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
33
Docker mode activated.

_ Prechecks _

+1
hbaseanti
0
Patch does not have any anti-patterns.

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 2 new or modified test files.

_ master Compile Tests _

0
mvndep
24
Maven dependency ordering for branch

+1
mvninstall
245
master passed

+1
compile
75
master passed

+1
checkstyle
78
master passed

+1
shadedjars
255
branch has no errors when building our shaded downstream artifacts.

+1
findbugs
243
master passed

+1
javadoc
49
master passed

_ Patch Compile Tests _

0
mvndep
13
Maven dependency ordering for patch

+1
mvninstall
229
the patch passed

+1
compile
74
the patch passed

+1
javac
74
the patch passed

+1
checkstyle
74
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedjars
255
patch has no errors when building our shaded downstream artifacts.

+1
hadoopcheck
701
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.

+1
findbugs
266
the patch passed

+1
javadoc
48
the patch passed

_ Other Tests _

-1
unit
1587
hbase-server in the patch failed.

-1
unit
829
hbase-rsgroup in the patch failed.

+1
asflicense
25
The patch does not generate ASF License warnings.

5406

Reason
Tests

Failed junit tests
hadoop.hbase.quotas.TestMasterQuotasObserverWithMocks

hadoop.hbase.regionserver.TestWALMonotonicallyIncreasingSeqId

hadoop.hbase.master.snapshot.TestSnapshotHFileCleaner

hadoop.hbase.master.snapshot.TestSnapshotManager

hadoop.hbase.quotas.TestTablesWithQuotas

hadoop.hbase.rsgroup.TestRSGroupsBalance

hadoop.hbase.rsgroup.TestRSGroupMajorCompactionTTL

hadoop.hbase.rsgroup.TestRSGroupsAdmin1

hadoop.hbase.rsgroup.TestRSGroupsBasics

hadoop.hbase.rsgroup.TestRSGroupsKillRS

hadoop.hbase.rsgroup.TestEnableRSGroups

Subsystem
Report/Notes

Docker
Client=18.09.7 Server=18.09.7 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-354/6/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/354

Optional Tests
dupname  asflicense  javac  javadoc  unit  findbugs  shadedjars  hadoopcheck  hbaseanti  checkstyle  compile

uname
Linux dc5ea2f92db0 4.4.0-139-generic #165-Ubuntu SMP Wed Oct 24 10:58:50 UTC 2018 x86_64 GNU/Linux

Build tool
maven

Personality
/testptch/patchprocess/precommit/personality/provided.sh

git revision
master / 438bf3284e

maven
version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)

Default Java
1.8.0_181

findbugs
v3.1.11

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-354/6/artifact/out/patch-unit-hbase-server.txt

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-354/6/artifact/out/patch-unit-hbase-rsgroup.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-354/6/testReport/

Max. process+thread count
3183 (vs. ulimit of 10000)

modules
C: hbase-server hbase-rsgroup U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-354/6/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
55
Docker mode activated.

_ Prechecks _

+1
hbaseanti
0
Patch does not have any anti-patterns.

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 2 new or modified test files.

_ master Compile Tests _

0
mvndep
30
Maven dependency ordering for branch

+1
mvninstall
270
master passed

+1
compile
83
master passed

+1
checkstyle
82
master passed

+1
shadedjars
280
branch has no errors when building our shaded downstream artifacts.

+1
findbugs
253
master passed

+1
javadoc
52
master passed

_ Patch Compile Tests _

0
mvndep
14
Maven dependency ordering for patch

+1
mvninstall
249
the patch passed

+1
compile
75
the patch passed

+1
javac
75
the patch passed

+1
checkstyle
77
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedjars
271
patch has no errors when building our shaded downstream artifacts.

+1
hadoopcheck
792
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.

+1
findbugs
266
the patch passed

+1
javadoc
51
the patch passed

_ Other Tests _

-1
unit
17710
hbase-server in the patch failed.

-1
unit
995
hbase-rsgroup in the patch failed.

+1
asflicense
49
The patch does not generate ASF License warnings.

21980

Reason
Tests

Failed junit tests
hadoop.hbase.client.TestCloneSnapshotFromClientNormal

hadoop.hbase.replication.TestSyncReplicationMoreLogsInLocalGiveUpSplitting

hadoop.hbase.client.TestSnapshotDFSTemporaryDirectory

hadoop.hbase.client.TestSnapshotTemporaryDirectory

hadoop.hbase.replication.TestSerialSyncReplication

hadoop.hbase.master.procedure.TestSCPWithReplicas

hadoop.hbase.rsgroup.TestRSGroupsKillRS

Subsystem
Report/Notes

Docker
Client=18.09.7 Server=18.09.7 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-354/7/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/354

Optional Tests
dupname  asflicense  javac  javadoc  unit  findbugs  shadedjars  hadoopcheck  hbaseanti  checkstyle  compile

uname
Linux e09fe51df46c 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 GNU/Linux

Build tool
maven

Personality
/testptch/patchprocess/precommit/personality/provided.sh

git revision
master / 74731c2a46

maven
version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)

Default Java
1.8.0_181

findbugs
v3.1.11

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-354/7/artifact/out/patch-unit-hbase-server.txt

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-354/7/artifact/out/patch-unit-hbase-rsgroup.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-354/7/testReport/

Max. process+thread count
4926 (vs. ulimit of 10000)

modules
C: hbase-server hbase-rsgroup U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-354/7/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
56
Docker mode activated.

_ Prechecks _

+1
hbaseanti
0
Patch does not have any anti-patterns.

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 2 new or modified test files.

_ master Compile Tests _

0
mvndep
15
Maven dependency ordering for branch

+1
mvninstall
313
master passed

+1
compile
99
master passed

+1
checkstyle
100
master passed

+1
shadedjars
342
branch has no errors when building our shaded downstream artifacts.

-1
findbugs
64
hbase-server in master failed.

+1
javadoc
63
master passed

_ Patch Compile Tests _

0
mvndep
16
Maven dependency ordering for patch

+1
mvninstall
328
the patch passed

+1
compile
103
the patch passed

+1
javac
103
the patch passed

+1
checkstyle
104
the patch passed

-1
whitespace
0
The patch has 2 line(s) that end in whitespace. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply

+1
shadedjars
351
patch has no errors when building our shaded downstream artifacts.

+1
hadoopcheck
990
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.

+1
findbugs
364
the patch passed

+1
javadoc
43
hbase-server generated 0 new + 0 unchanged - 3 fixed = 0 total (was 3)

+1
javadoc
23
hbase-rsgroup in the patch passed.

_ Other Tests _

-1
unit
14920
hbase-server in the patch failed.

+1
unit
528
hbase-rsgroup in the patch passed.

+1
asflicense
60
The patch does not generate ASF License warnings.

19359

Reason
Tests

Failed junit tests
hadoop.hbase.client.TestAsyncTableAdminApi3

Subsystem
Report/Notes

Docker
Client=18.09.7 Server=18.09.7 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-354/8/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/354

Optional Tests
dupname  asflicense  javac  javadoc  unit  findbugs  shadedjars  hadoopcheck  hbaseanti  checkstyle  compile

uname
Linux 971807cec9c3 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 GNU/Linux

Build tool
maven

Personality
/testptch/patchprocess/precommit/personality/provided.sh

git revision
master / cc38de1a39

maven
version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)

Default Java
1.8.0_181

findbugs
v3.1.11

findbugs
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-354/8/artifact/out/branch-findbugs-hbase-server.txt

whitespace
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-354/8/artifact/out/whitespace-eol.txt

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-354/8/artifact/out/patch-unit-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-354/8/testReport/

Max. process+thread count
4467 (vs. ulimit of 10000)

modules
C: hbase-server hbase-rsgroup U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-354/8/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
34
Docker mode activated.

_ Prechecks _

+1
hbaseanti
0
Patch does not have any anti-patterns.

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 2 new or modified test files.

_ master Compile Tests _

0
mvndep
25
Maven dependency ordering for branch

+1
mvninstall
249
master passed

+1
compile
78
master passed

+1
checkstyle
78
master passed

+1
shadedjars
260
branch has no errors when building our shaded downstream artifacts.

+1
findbugs
233
master passed

+1
javadoc
52
master passed

_ Patch Compile Tests _

0
mvndep
17
Maven dependency ordering for patch

+1
mvninstall
232
the patch passed

+1
compile
75
the patch passed

+1
javac
75
the patch passed

+1
checkstyle
79
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedjars
264
patch has no errors when building our shaded downstream artifacts.

+1
hadoopcheck
715
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.

+1
findbugs
244
the patch passed

+1
javadoc
50
the patch passed

_ Other Tests _

+1
unit
7824
hbase-server in the patch passed.

+1
unit
303
hbase-rsgroup in the patch passed.

+1
asflicense
55
The patch does not generate ASF License warnings.

11185

Subsystem
Report/Notes

Docker
Client=18.09.7 Server=18.09.7 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-354/9/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/354

Optional Tests
dupname  asflicense  javac  javadoc  unit  findbugs  shadedjars  hadoopcheck  hbaseanti  checkstyle  compile

uname
Linux 9ea8e0bac900 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 GNU/Linux

Build tool
maven

Personality
/testptch/patchprocess/precommit/personality/provided.sh

git revision
master / 72e58a8fed

maven
version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)

Default Java
1.8.0_181

findbugs
v3.1.11

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-354/9/testReport/

Max. process+thread count
4581 (vs. ulimit of 10000)

modules
C: hbase-server hbase-rsgroup U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-354/9/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

source: gharchive/pull-request

authors: Apache-HBase infraio sunhelly

repo: apache/hbase

license_type: permissive

license_source: github-api


text:
HBASE-26122: Implement an optional maximum size for Gets, after which a partial result is returned

This is a version of https://github.com/apache/hbase/pull/3532 which applies against master branch. When I created the previous PR I didn't realize the convention of doing master first. That PR did not cleanly apply to master due to changes in hbase-protocol vs hbase-protocol-shaded and minor differences in HRegion.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m  7s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 14s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
4m 14s
master passed

+1 :green_heart:
compile
2m 18s
master passed

+1 :green_heart:
shadedjars
9m 11s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 14s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 13s
Maven dependency ordering for patch

-1 :x:
mvninstall
2m  6s
root in the patch failed.

-1 :x:
compile
0m 49s
hbase-server in the patch failed.

-0 :warning:
javac
0m 49s
hbase-server in the patch failed.

-1 :x:
shadedjars
6m 52s
patch has 14 errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 10s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
0m 49s
hbase-protocol-shaded in the patch passed.

+1 :green_heart:
unit
1m 29s
hbase-client in the patch passed.

-1 :x:
unit
0m 47s
hbase-server in the patch failed.

35m 18s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3565

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 00d4ba7301e8 4.15.0-142-generic #146-Ubuntu SMP Tue Apr 13 01:11:19 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / da950b9be2

Default Java
AdoptOpenJDK-1.8.0_282-b08

mvninstall
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/1/artifact/yetus-jdk8-hadoop3-check/output/patch-mvninstall-root.txt

compile
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/1/artifact/yetus-jdk8-hadoop3-check/output/patch-compile-hbase-server.txt

javac
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/1/artifact/yetus-jdk8-hadoop3-check/output/patch-compile-hbase-server.txt

shadedjars
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/1/artifact/yetus-jdk8-hadoop3-check/output/patch-shadedjars.txt

unit
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/1/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-hbase-server.txt

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/1/testReport/

Max. process+thread count
225 (vs. ulimit of 30000)

modules
C: hbase-protocol-shaded hbase-client hbase-server U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m  0s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 14s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
5m  3s
master passed

+1 :green_heart:
compile
2m 53s
master passed

+1 :green_heart:
shadedjars
9m 17s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 20s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 14s
Maven dependency ordering for patch

-1 :x:
mvninstall
2m 32s
root in the patch failed.

-1 :x:
compile
0m 57s
hbase-server in the patch failed.

-0 :warning:
javac
0m 57s
hbase-server in the patch failed.

-1 :x:
shadedjars
7m  3s
patch has 14 errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 19s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
1m  4s
hbase-protocol-shaded in the patch passed.

+1 :green_heart:
unit
1m 40s
hbase-client in the patch passed.

-1 :x:
unit
0m 58s
hbase-server in the patch failed.

38m 47s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3565

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux b217827c2c58 4.15.0-147-generic #151-Ubuntu SMP Fri Jun 18 19:21:19 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / da950b9be2

Default Java
AdoptOpenJDK-11.0.10+9

mvninstall
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/1/artifact/yetus-jdk11-hadoop3-check/output/patch-mvninstall-root.txt

compile
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/1/artifact/yetus-jdk11-hadoop3-check/output/patch-compile-hbase-server.txt

javac
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/1/artifact/yetus-jdk11-hadoop3-check/output/patch-compile-hbase-server.txt

shadedjars
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/1/artifact/yetus-jdk11-hadoop3-check/output/patch-shadedjars.txt

unit
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/1/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-hbase-server.txt

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/1/testReport/

Max. process+thread count
212 (vs. ulimit of 30000)

modules
C: hbase-protocol-shaded hbase-client hbase-server U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 16s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+0 :ok:
prototool
0m  0s
prototool was not available.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+0 :ok:
mvndep
0m 14s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
4m 17s
master passed

+1 :green_heart:
compile
5m 44s
master passed

+1 :green_heart:
checkstyle
1m 56s
master passed

+1 :green_heart:
spotbugs
7m 32s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 12s
Maven dependency ordering for patch

-1 :x:
mvninstall
2m  3s
root in the patch failed.

-1 :x:
compile
1m 57s
hbase-server in the patch failed.

-0 :warning:
cc
1m 57s
hbase-server in the patch failed.

-0 :warning:
javac
1m 57s
hbase-server in the patch failed.

-0 :warning:
checkstyle
1m 11s
hbase-server: The patch generated 6 new + 230 unchanged - 0 fixed = 236 total (was 230)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

-1 :x:
hadoopcheck
2m 17s
The patch causes 14 errors with Hadoop v3.1.2.

-1 :x:
hadoopcheck
4m 41s
The patch causes 14 errors with Hadoop v3.2.1.

-1 :x:
hadoopcheck
7m  3s
The patch causes 14 errors with Hadoop v3.3.0.

-1 :x:
hbaseprotoc
0m 35s
hbase-server in the patch failed.

-1 :x:
spotbugs
0m 31s
hbase-server in the patch failed.

_ Other Tests _

+1 :green_heart:
asflicense
0m 29s
The patch does not generate ASF License warnings.

47m 12s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3565

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti checkstyle compile cc hbaseprotoc prototool

uname
Linux 9893e06c4b5a 4.15.0-143-generic #147-Ubuntu SMP Wed Apr 14 16:10:11 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / da950b9be2

Default Java
AdoptOpenJDK-1.8.0_282-b08

mvninstall
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/1/artifact/yetus-general-check/output/patch-mvninstall-root.txt

compile
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/1/artifact/yetus-general-check/output/patch-compile-hbase-server.txt

cc
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/1/artifact/yetus-general-check/output/patch-compile-hbase-server.txt

javac
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/1/artifact/yetus-general-check/output/patch-compile-hbase-server.txt

checkstyle
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/1/artifact/yetus-general-check/output/diff-checkstyle-hbase-server.txt

hadoopcheck
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/1/artifact/yetus-general-check/output/patch-javac-3.1.2.txt

hadoopcheck
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/1/artifact/yetus-general-check/output/patch-javac-3.2.1.txt

hadoopcheck
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/1/artifact/yetus-general-check/output/patch-javac-3.3.0.txt

hbaseprotoc
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/1/artifact/yetus-general-check/output/patch-hbaseprotoc-hbase-server.txt

spotbugs
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/1/artifact/yetus-general-check/output/patch-spotbugs-hbase-server.txt

Max. process+thread count
86 (vs. ulimit of 30000)

modules
C: hbase-protocol-shaded hbase-client hbase-server U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/1/console

versions
git=2.17.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
2m 11s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+0 :ok:
prototool
0m  0s
prototool was not available.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+0 :ok:
mvndep
0m 15s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
4m 38s
master passed

+1 :green_heart:
compile
6m  6s
master passed

+1 :green_heart:
checkstyle
1m 54s
master passed

+1 :green_heart:
spotbugs
7m 55s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 13s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
4m 25s
the patch passed

+1 :green_heart:
compile
5m 59s
the patch passed

+1 :green_heart:
cc
5m 59s
the patch passed

-0 :warning:
javac
0m 57s
hbase-client generated 1 new + 121 unchanged - 1 fixed = 122 total (was 122)

-0 :warning:
javac
3m 28s
hbase-server generated 2 new + 191 unchanged - 2 fixed = 193 total (was 193)

-0 :warning:
checkstyle
1m 14s
hbase-server: The patch generated 3 new + 230 unchanged - 0 fixed = 233 total (was 230)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
23m 51s
Patch does not cause any errors with Hadoop 3.1.2 3.2.1 3.3.0.

+1 :green_heart:
hbaseprotoc
2m 19s
the patch passed

+1 :green_heart:
spotbugs
9m 33s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 38s
The patch does not generate ASF License warnings.

82m 11s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/2/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3565

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti checkstyle compile cc hbaseprotoc prototool

uname
Linux 54ca1b8867c1 4.15.0-143-generic #147-Ubuntu SMP Wed Apr 14 16:10:11 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / c9383f2c89

Default Java
AdoptOpenJDK-1.8.0_282-b08

javac
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/2/artifact/yetus-general-check/output/diff-compile-javac-hbase-client.txt

javac
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/2/artifact/yetus-general-check/output/diff-compile-javac-hbase-server.txt

checkstyle
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/2/artifact/yetus-general-check/output/diff-checkstyle-hbase-server.txt

Max. process+thread count
86 (vs. ulimit of 30000)

modules
C: hbase-protocol-shaded hbase-client hbase-server U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/2/console

versions
git=2.17.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m  7s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 12s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
4m 45s
master passed

+1 :green_heart:
compile
2m 56s
master passed

+1 :green_heart:
shadedjars
9m  5s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 19s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 13s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
4m 48s
the patch passed

+1 :green_heart:
compile
2m 53s
the patch passed

+1 :green_heart:
javac
2m 53s
the patch passed

+1 :green_heart:
shadedjars
9m  7s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 21s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
1m  3s
hbase-protocol-shaded in the patch passed.

+1 :green_heart:
unit
1m 37s
hbase-client in the patch passed.

-1 :x:
unit
204m 42s
hbase-server in the patch failed.

247m 31s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/2/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3565

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 182e5cdbc46c 4.15.0-147-generic #151-Ubuntu SMP Fri Jun 18 19:21:19 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / c9383f2c89

Default Java
AdoptOpenJDK-11.0.10+9

unit
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/2/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-hbase-server.txt

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/2/testReport/

Max. process+thread count
3613 (vs. ulimit of 30000)

modules
C: hbase-protocol-shaded hbase-client hbase-server U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/2/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
2m  7s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 15s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
4m 35s
master passed

+1 :green_heart:
compile
2m 30s
master passed

+1 :green_heart:
shadedjars
9m 20s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 19s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 15s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
4m 25s
the patch passed

+1 :green_heart:
compile
2m 29s
the patch passed

+1 :green_heart:
javac
2m 29s
the patch passed

+1 :green_heart:
shadedjars
9m 25s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 14s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
0m 51s
hbase-protocol-shaded in the patch passed.

+1 :green_heart:
unit
1m 30s
hbase-client in the patch passed.

-1 :x:
unit
219m 52s
hbase-server in the patch failed.

262m 31s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/2/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3565

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 1bc8326361e6 4.15.0-143-generic #147-Ubuntu SMP Wed Apr 14 16:10:11 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / c9383f2c89

Default Java
AdoptOpenJDK-1.8.0_282-b08

unit
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/2/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-hbase-server.txt

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/2/testReport/

Max. process+thread count
3015 (vs. ulimit of 30000)

modules
C: hbase-protocol-shaded hbase-client hbase-server U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/2/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 23s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+0 :ok:
prototool
0m  0s
prototool was not available.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+0 :ok:
mvndep
0m 25s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
4m  7s
master passed

+1 :green_heart:
compile
5m 45s
master passed

+1 :green_heart:
checkstyle
1m 54s
master passed

+1 :green_heart:
spotbugs
7m 19s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 12s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
4m  0s
the patch passed

+1 :green_heart:
compile
5m 45s
the patch passed

+1 :green_heart:
cc
5m 45s
the patch passed

+1 :green_heart:
javac
5m 45s
the patch passed

-0 :warning:
checkstyle
1m 14s
hbase-server: The patch generated 3 new + 230 unchanged - 0 fixed = 233 total (was 230)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
20m 21s
Patch does not cause any errors with Hadoop 3.1.2 3.2.1 3.3.0.

+1 :green_heart:
hbaseprotoc
2m  4s
the patch passed

+1 :green_heart:
spotbugs
8m  0s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 33s
The patch does not generate ASF License warnings.

72m 47s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/3/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3565

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti checkstyle compile cc hbaseprotoc prototool

uname
Linux f5d6a458b2c3 4.15.0-143-generic #147-Ubuntu SMP Wed Apr 14 16:10:11 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / c9383f2c89

Default Java
AdoptOpenJDK-1.8.0_282-b08

checkstyle
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/3/artifact/yetus-general-check/output/diff-checkstyle-hbase-server.txt

Max. process+thread count
86 (vs. ulimit of 30000)

modules
C: hbase-protocol-shaded hbase-client hbase-server U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/3/console

versions
git=2.17.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 59s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 15s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
5m  5s
master passed

+1 :green_heart:
compile
2m 50s
master passed

+1 :green_heart:
shadedjars
9m  4s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 21s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 14s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
4m 45s
the patch passed

+1 :green_heart:
compile
2m 50s
the patch passed

+1 :green_heart:
javac
2m 50s
the patch passed

+1 :green_heart:
shadedjars
9m  2s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 20s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
1m  2s
hbase-protocol-shaded in the patch passed.

+1 :green_heart:
unit
1m 38s
hbase-client in the patch passed.

-1 :x:
unit
203m 43s
hbase-server in the patch failed.

246m 30s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/3/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3565

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 040555e5ffcc 4.15.0-143-generic #147-Ubuntu SMP Wed Apr 14 16:10:11 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / c9383f2c89

Default Java
AdoptOpenJDK-11.0.10+9

unit
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/3/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-hbase-server.txt

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/3/testReport/

Max. process+thread count
3463 (vs. ulimit of 30000)

modules
C: hbase-protocol-shaded hbase-client hbase-server U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/3/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m  6s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 29s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
4m  7s
master passed

+1 :green_heart:
compile
2m 18s
master passed

+1 :green_heart:
shadedjars
9m  7s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m  9s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 13s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
4m  8s
the patch passed

+1 :green_heart:
compile
2m 16s
the patch passed

+1 :green_heart:
javac
2m 16s
the patch passed

+1 :green_heart:
shadedjars
9m  5s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 11s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
0m 47s
hbase-protocol-shaded in the patch passed.

+1 :green_heart:
unit
1m 28s
hbase-client in the patch passed.

-1 :x:
unit
217m  9s
hbase-server in the patch failed.

256m 49s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/3/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3565

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux adc1867d94c3 4.15.0-142-generic #146-Ubuntu SMP Tue Apr 13 01:11:19 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / c9383f2c89

Default Java
AdoptOpenJDK-1.8.0_282-b08

unit
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/3/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-hbase-server.txt

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/3/testReport/

Max. process+thread count
3183 (vs. ulimit of 30000)

modules
C: hbase-protocol-shaded hbase-client hbase-server U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/3/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 25s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+0 :ok:
prototool
0m  0s
prototool was not available.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+0 :ok:
mvndep
0m 17s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
4m 20s
master passed

+1 :green_heart:
compile
5m 56s
master passed

+1 :green_heart:
checkstyle
1m 53s
master passed

+1 :green_heart:
spotbugs
7m 38s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 12s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
4m  8s
the patch passed

+1 :green_heart:
compile
5m 58s
the patch passed

+1 :green_heart:
cc
5m 58s
the patch passed

+1 :green_heart:
javac
5m 58s
the patch passed

-0 :warning:
checkstyle
1m 15s
hbase-server: The patch generated 3 new + 230 unchanged - 0 fixed = 233 total (was 230)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
20m 58s
Patch does not cause any errors with Hadoop 3.1.2 3.2.1 3.3.0.

+1 :green_heart:
hbaseprotoc
2m  7s
the patch passed

+1 :green_heart:
spotbugs
7m 56s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 33s
The patch does not generate ASF License warnings.

74m 38s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/4/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3565

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti checkstyle compile cc hbaseprotoc prototool

uname
Linux 1eb07f8d96ff 4.15.0-143-generic #147-Ubuntu SMP Wed Apr 14 16:10:11 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / c9383f2c89

Default Java
AdoptOpenJDK-1.8.0_282-b08

checkstyle
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/4/artifact/yetus-general-check/output/diff-checkstyle-hbase-server.txt

Max. process+thread count
86 (vs. ulimit of 30000)

modules
C: hbase-protocol-shaded hbase-client hbase-server U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/4/console

versions
git=2.17.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 32s
Docker mode activated.

-0 :warning:
yetus
0m  4s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 16s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
3m 50s
master passed

+1 :green_heart:
compile
2m 11s
master passed

+1 :green_heart:
shadedjars
8m 14s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 17s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 17s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
3m 41s
the patch passed

+1 :green_heart:
compile
2m 14s
the patch passed

+1 :green_heart:
javac
2m 14s
the patch passed

+1 :green_heart:
shadedjars
8m 14s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 13s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
0m 47s
hbase-protocol-shaded in the patch passed.

+1 :green_heart:
unit
1m 15s
hbase-client in the patch passed.

-1 :x:
unit
151m  2s
hbase-server in the patch failed.

187m 51s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/4/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3565

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 29172abf9d7a 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / c9383f2c89

Default Java
AdoptOpenJDK-1.8.0_282-b08

unit
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/4/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-hbase-server.txt

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/4/testReport/

Max. process+thread count
4078 (vs. ulimit of 30000)

modules
C: hbase-protocol-shaded hbase-client hbase-server U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/4/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m  1s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 14s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
4m 58s
master passed

+1 :green_heart:
compile
2m 50s
master passed

+1 :green_heart:
shadedjars
8m 59s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 22s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 14s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
4m 48s
the patch passed

+1 :green_heart:
compile
2m 52s
the patch passed

+1 :green_heart:
javac
2m 52s
the patch passed

+1 :green_heart:
shadedjars
9m  4s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 19s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
1m  2s
hbase-protocol-shaded in the patch passed.

+1 :green_heart:
unit
1m 37s
hbase-client in the patch passed.

-1 :x:
unit
205m 52s
hbase-server in the patch failed.

248m 29s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/4/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3565

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 51f89de49551 4.15.0-143-generic #147-Ubuntu SMP Wed Apr 14 16:10:11 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / c9383f2c89

Default Java
AdoptOpenJDK-11.0.10+9

unit
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/4/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-hbase-server.txt

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/4/testReport/

Max. process+thread count
3299 (vs. ulimit of 30000)

modules
C: hbase-protocol-shaded hbase-client hbase-server U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/4/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

Want to make this same as the branch-2 pushed PR @bbeaudreault  and then I'll commit (I couldn't forward port the branch-2 PR w/o conflict.

@saintstack I updated the PR to include the change from branch-2. Also see https://github.com/apache/hbase/pull/3576, with an addendum for branch-2

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 36s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 18s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
5m 12s
master passed

+1 :green_heart:
compile
2m 43s
master passed

+1 :green_heart:
shadedjars
8m 21s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 24s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 15s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
4m 15s
the patch passed

+1 :green_heart:
compile
2m 42s
the patch passed

+1 :green_heart:
javac
2m 42s
the patch passed

+1 :green_heart:
shadedjars
8m 11s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 22s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
0m 59s
hbase-protocol-shaded in the patch passed.

+1 :green_heart:
unit
1m 25s
hbase-client in the patch passed.

-1 :x:
unit
11m 40s
hbase-server in the patch failed.

51m 27s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/5/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3565

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 9023486b4691 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 5e8a269b1a

Default Java
AdoptOpenJDK-11.0.10+9

unit
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/5/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-hbase-server.txt

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/5/testReport/

Max. process+thread count
1475 (vs. ulimit of 30000)

modules
C: hbase-protocol-shaded hbase-client hbase-server U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/5/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
3m  9s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 28s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
4m 14s
master passed

+1 :green_heart:
compile
2m 16s
master passed

+1 :green_heart:
shadedjars
9m  7s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 12s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 15s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
4m  5s
the patch passed

+1 :green_heart:
compile
2m 17s
the patch passed

+1 :green_heart:
javac
2m 17s
the patch passed

+1 :green_heart:
shadedjars
9m  1s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 10s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
0m 48s
hbase-protocol-shaded in the patch passed.

+1 :green_heart:
unit
1m 28s
hbase-client in the patch passed.

-1 :x:
unit
16m 28s
hbase-server in the patch failed.

57m 40s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/5/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3565

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 9eeec456c6da 4.15.0-142-generic #146-Ubuntu SMP Tue Apr 13 01:11:19 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 5e8a269b1a

Default Java
AdoptOpenJDK-1.8.0_282-b08

unit
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/5/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-hbase-server.txt

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/5/testReport/

Max. process+thread count
954 (vs. ulimit of 30000)

modules
C: hbase-protocol-shaded hbase-client hbase-server U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/5/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
6m  4s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+0 :ok:
prototool
0m  0s
prototool was not available.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+0 :ok:
mvndep
0m 28s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
4m 13s
master passed

+1 :green_heart:
compile
5m 46s
master passed

+1 :green_heart:
checkstyle
1m 53s
master passed

+1 :green_heart:
spotbugs
7m 19s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 13s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
4m  7s
the patch passed

+1 :green_heart:
compile
5m 46s
the patch passed

+1 :green_heart:
cc
5m 46s
the patch passed

+1 :green_heart:
javac
5m 46s
the patch passed

-0 :warning:
checkstyle
1m 13s
hbase-server: The patch generated 2 new + 230 unchanged - 0 fixed = 232 total (was 230)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
20m 10s
Patch does not cause any errors with Hadoop 3.1.2 3.2.1 3.3.0.

+1 :green_heart:
hbaseprotoc
2m  4s
the patch passed

+1 :green_heart:
spotbugs
8m  3s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 34s
The patch does not generate ASF License warnings.

77m 36s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/5/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3565

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti checkstyle compile cc hbaseprotoc prototool

uname
Linux 9e80e51544a0 4.15.0-143-generic #147-Ubuntu SMP Wed Apr 14 16:10:11 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 5e8a269b1a

Default Java
AdoptOpenJDK-1.8.0_282-b08

checkstyle
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/5/artifact/yetus-general-check/output/diff-checkstyle-hbase-server.txt

Max. process+thread count
86 (vs. ulimit of 30000)

modules
C: hbase-protocol-shaded hbase-client hbase-server U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/5/console

versions
git=2.17.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 11s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 16s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
4m 18s
master passed

+1 :green_heart:
compile
2m 20s
master passed

+1 :green_heart:
shadedjars
9m 15s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 18s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 14s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
4m 21s
the patch passed

+1 :green_heart:
compile
2m 24s
the patch passed

+1 :green_heart:
javac
2m 24s
the patch passed

+1 :green_heart:
shadedjars
9m  0s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 10s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
0m 48s
hbase-protocol-shaded in the patch passed.

+1 :green_heart:
unit
1m 31s
hbase-client in the patch passed.

+1 :green_heart:
unit
225m  2s
hbase-server in the patch passed.

265m 44s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/6/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3565

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 2f86b87ab383 4.15.0-142-generic #146-Ubuntu SMP Tue Apr 13 01:11:19 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 5e8a269b1a

Default Java
AdoptOpenJDK-1.8.0_282-b08

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/6/testReport/

Max. process+thread count
2916 (vs. ulimit of 30000)

modules
C: hbase-protocol-shaded hbase-client hbase-server U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/6/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 21s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+0 :ok:
prototool
0m  0s
prototool was not available.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+0 :ok:
mvndep
0m 15s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
4m 20s
master passed

+1 :green_heart:
compile
5m 43s
master passed

+1 :green_heart:
checkstyle
1m 55s
master passed

+1 :green_heart:
spotbugs
7m 44s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 12s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
4m  4s
the patch passed

+1 :green_heart:
compile
5m 44s
the patch passed

+1 :green_heart:
cc
5m 44s
the patch passed

+1 :green_heart:
javac
5m 44s
the patch passed

-0 :warning:
checkstyle
1m 14s
hbase-server: The patch generated 2 new + 231 unchanged - 0 fixed = 233 total (was 231)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
20m 31s
Patch does not cause any errors with Hadoop 3.1.2 3.2.1 3.3.0.

+1 :green_heart:
hbaseprotoc
2m  3s
the patch passed

+1 :green_heart:
spotbugs
7m 56s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 32s
The patch does not generate ASF License warnings.

73m 17s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3565

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti checkstyle compile cc hbaseprotoc prototool

uname
Linux 4bef142bf32d 4.15.0-143-generic #147-Ubuntu SMP Wed Apr 14 16:10:11 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 5e8a269b1a

Default Java
AdoptOpenJDK-1.8.0_282-b08

checkstyle
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/1/artifact/yetus-general-check/output/diff-checkstyle-hbase-server.txt

Max. process+thread count
86 (vs. ulimit of 30000)

modules
C: hbase-protocol-shaded hbase-client hbase-server U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/1/console

versions
git=2.17.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 26s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 17s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
3m 55s
master passed

+1 :green_heart:
compile
2m 15s
master passed

+1 :green_heart:
shadedjars
8m 13s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 15s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 16s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
3m 43s
the patch passed

+1 :green_heart:
compile
2m 15s
the patch passed

+1 :green_heart:
javac
2m 15s
the patch passed

+1 :green_heart:
shadedjars
8m 19s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 16s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
0m 48s
hbase-protocol-shaded in the patch passed.

+1 :green_heart:
unit
1m 16s
hbase-client in the patch passed.

+1 :green_heart:
unit
146m 31s
hbase-server in the patch passed.

183m 29s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3565

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux e35a2b31decb 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 5e8a269b1a

Default Java
AdoptOpenJDK-1.8.0_282-b08

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/1/testReport/

Max. process+thread count
3848 (vs. ulimit of 30000)

modules
C: hbase-protocol-shaded hbase-client hbase-server U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
6m 30s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 16s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
4m 59s
master passed

+1 :green_heart:
compile
2m 51s
master passed

+1 :green_heart:
shadedjars
9m  1s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 20s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 14s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
4m 48s
the patch passed

+1 :green_heart:
compile
2m 55s
the patch passed

+1 :green_heart:
javac
2m 55s
the patch passed

+1 :green_heart:
shadedjars
9m  2s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 19s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
1m  2s
hbase-protocol-shaded in the patch passed.

+1 :green_heart:
unit
1m 38s
hbase-client in the patch passed.

+1 :green_heart:
unit
197m 24s
hbase-server in the patch passed.

245m 38s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3565

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 225bb8d110c3 4.15.0-143-generic #147-Ubuntu SMP Wed Apr 14 16:10:11 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 5e8a269b1a

Default Java
AdoptOpenJDK-11.0.10+9

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/1/testReport/

Max. process+thread count
2620 (vs. ulimit of 30000)

modules
C: hbase-protocol-shaded hbase-client hbase-server U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3565/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

Hey @saintstack, sorry I've been distracted by internal stuff the last few work days. But will be getting back to this this week. Yes, this PR needs to be updated based on our convo in the other PR.

Do we really need to add such complexity to Gets?
Get is anyways a special Scan only at server end.  Now that we have
scan.withStopRow( byte[] key, boolean inclusive)
Also the openScan can open a scanner session at server and also create results and close and then return (all in one RPC like Get).
Scan next will have both size based and time based checks possible to return partial results.
So when a usage needs to get very large rows, instead of Get, user can use Scan only with all these relevant stuff set on Scan?  Just raising for a second thinking.

Thank you for your comment Anoop. The problem we face is that we have hundreds of engineers using hbase in innumerable ways, on multi-tenant clusters. People shift teams, leave, get hired, etc. When you set out to read a row from hbase, you don't always know how large that row will be.
Yes I agree with you, if someone knew they were going to fetch a large row, they should do a Scan. But given the above, people often don't. This new feature acts as a guardrail against causing RegionServer pain in those cases.
The way we've used this feature is we have our own TableFactory that everyone must use to get tables. The returned Table objects are wrapped. When a Get is submitted, the wrapper uses this to limit the max result size. When data is returned, we inspect the Result and throw an exception if it's a partial result.
At that point the user can rewrite their query to use a Scan or add a filter, etc. We also have an escape hatch for urgent issues to allow them to go through, but that is audited.
That said, I just noticed hbase.table.max.rowsize. Surprisingly that was added shortly after our original patch of this and we haven't noticed it since. The default value looks too large, but given we've already been enforcing this I think we might be able to move to that instead.

Given the above and the fact that changes are needed here anyway, I'm going to close this for now while we investigate setting hbase.table.max.rowsize instead. If it doesn't pan out I'll open a new PR with the fixes mentioned.

authors: Apache-HBase anoopsjohn bbeaudreault saintstack