HBASE-24503 : Backport HBASE-24492 Remove infinite loop from ProtobufLogReader#readNext (#1831)

[virajjasani]

Signed-off-by: Duo Zhang zhangduo@apache.org

[Apache-HBase]

💔 -1 overall

Vote	Subsystem	Runtime	Comment
+0 🆗	reexec	6m 44s	Docker mode activated.
		_ Prechecks _
+1 💚	dupname	0m 0s	No case conflicting files found.
+1 💚	hbaseanti	0m 0s	Patch does not have any anti-patterns.
+1 💚	@author	0m 0s	The patch does not contain any @author tags.
-0 ⚠️	test4tests	0m 0s	The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
		_ branch-2.2 Compile Tests _
+1 💚	mvninstall	4m 28s	branch-2.2 passed
+1 💚	compile	1m 8s	branch-2.2 passed
+1 💚	checkstyle	1m 45s	branch-2.2 passed
+1 💚	shadedjars	5m 40s	branch has no errors when building our shaded downstream artifacts.
+1 💚	javadoc	0m 47s	branch-2.2 passed
+0 🆗	spotbugs	4m 28s	Used deprecated FindBugs config; considering switching to SpotBugs.
+1 💚	findbugs	4m 25s	branch-2.2 passed
		_ Patch Compile Tests _
+1 💚	mvninstall	4m 7s	the patch passed
+1 💚	compile	1m 11s	the patch passed
+1 💚	javac	1m 11s	the patch passed
+1 💚	checkstyle	1m 35s	the patch passed
+1 💚	whitespace	0m 0s	The patch has no whitespace issues.
+1 💚	shadedjars	5m 40s	patch has no errors when building our shaded downstream artifacts.
+1 💚	hadoopcheck	33m 50s	Patch does not cause any errors with Hadoop 2.8.5 2.9.2 2.10.0 or 3.1.2 3.2.1.
+1 💚	javadoc	0m 35s	the patch passed
+1 💚	findbugs	3m 12s	the patch passed
		_ Other Tests _
-1 ❌	unit	331m 58s	hbase-server in the patch failed.
+1 💚	asflicense	0m 33s	The patch does not generate ASF License warnings.
		414m 14s

Reason	Tests
Failed junit tests	hadoop.hbase.client.TestSnapshotCloneIndependence
	hadoop.hbase.master.TestDLSFSHLog
	hadoop.hbase.master.TestAssignmentManagerMetrics

Subsystem	Report/Notes
Docker	Client=19.03.13 Server=19.03.13 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2479/1/artifact/out/Dockerfile
GITHUB PR	#2479
Optional Tests	dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
uname	Linux 0d707f663a46 4.15.0-65-generic #74-Ubuntu SMP Tue Sep 17 17:06:04 UTC 2019 x86_64 GNU/Linux
Build tool	maven
Personality	/home/jenkins/jenkins-home/workspace/Base-PreCommit-GitHub-PR_PR-2479/out/precommit/personality/provided.sh
git revision	branch-2.2 / 4bf490c
Default Java	1.8.0_181
unit	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2479/1/artifact/out/patch-unit-hbase-server.txt
Test Results	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2479/1/testReport/
Max. process+thread count	2623 (vs. ulimit of 12500)
modules	C: hbase-server U: hbase-server
Console output	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2479/1/console
versions	git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered by	Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

[Apache-HBase]

💔 -1 overall

Vote	Subsystem	Runtime	Comment
+0 🆗	reexec	0m 38s	Docker mode activated.
		_ Prechecks _
+1 💚	dupname	0m 0s	No case conflicting files found.
+1 💚	hbaseanti	0m 0s	Patch does not have any anti-patterns.
+1 💚	@author	0m 0s	The patch does not contain any @author tags.
-0 ⚠️	test4tests	0m 0s	The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
		_ branch-2.2 Compile Tests _
+1 💚	mvninstall	4m 23s	branch-2.2 passed
+1 💚	compile	1m 12s	branch-2.2 passed
+1 💚	checkstyle	1m 41s	branch-2.2 passed
+1 💚	shadedjars	5m 9s	branch has no errors when building our shaded downstream artifacts.
+1 💚	javadoc	0m 47s	branch-2.2 passed
+0 🆗	spotbugs	4m 33s	Used deprecated FindBugs config; considering switching to SpotBugs.
+1 💚	findbugs	4m 28s	branch-2.2 passed
		_ Patch Compile Tests _
+1 💚	mvninstall	3m 50s	the patch passed
+1 💚	compile	1m 15s	the patch passed
+1 💚	javac	1m 15s	the patch passed
+1 💚	checkstyle	1m 42s	the patch passed
+1 💚	whitespace	0m 0s	The patch has no whitespace issues.
+1 💚	shadedjars	5m 14s	patch has no errors when building our shaded downstream artifacts.
+1 💚	hadoopcheck	32m 48s	Patch does not cause any errors with Hadoop 2.8.5 2.9.2 2.10.0 or 3.1.2 3.2.1.
+1 💚	javadoc	0m 51s	the patch passed
+1 💚	findbugs	5m 43s	the patch passed
		_ Other Tests _
-1 ❌	unit	244m 37s	hbase-server in the patch failed.
+1 💚	asflicense	0m 54s	The patch does not generate ASF License warnings.
		325m 12s

Reason	Tests
Failed junit tests	hadoop.hbase.client.TestZKAsyncRegistry

Subsystem	Report/Notes
Docker	Client=19.03.13 Server=19.03.13 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2479/2/artifact/out/Dockerfile
GITHUB PR	#2479
Optional Tests	dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
uname	Linux 19a47aa830f1 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 GNU/Linux
Build tool	maven
Personality	/home/jenkins/jenkins-home/workspace/Base-PreCommit-GitHub-PR_PR-2479/out/precommit/personality/provided.sh
git revision	branch-2.2 / 4bf490c
Default Java	1.8.0_181
unit	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2479/2/artifact/out/patch-unit-hbase-server.txt
Test Results	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2479/2/testReport/
Max. process+thread count	2734 (vs. ulimit of 12500)
modules	C: hbase-server U: hbase-server
Console output	https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2479/2/console
versions	git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered by	Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.