HBASE-30202 [hbase-thirdparty] Bump netty4 to 4.1.135.Final

[alta404]

No description provided.

[Apache-HBase]

💔 -1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	2m 9s		Docker mode activated.
			_ Prechecks _
+1 💚	dupname	0m 0s		No case conflicting files found.
+0 🆗	codespell	0m 0s		codespell was not available.
+0 🆗	detsecrets	0m 0s		detect-secrets was not available.
+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.
			_ master Compile Tests _
-1 ❌	mvninstall	0m 31s	/branch-mvninstall-root.txt	root in master failed.
-1 ❌	compile	0m 3s	/branch-compile-root.txt	root in master failed.
-1 ❌	javadoc	0m 17s	/branch-javadoc-root.txt	root in master failed.
			_ Patch Compile Tests _
-1 ❌	mvninstall	0m 15s	/patch-mvninstall-root.txt	root in the patch failed.
-1 ❌	compile	0m 3s	/patch-compile-root.txt	root in the patch failed.
-1 ❌	javac	0m 3s	/patch-compile-root.txt	root in the patch failed.
+1 💚	blanks	0m 0s		The patch has no blanks issues.
+1 💚	xmllint	0m 0s		No new issues.
-1 ❌	javadoc	0m 18s	/patch-javadoc-root.txt	root in the patch failed.
			_ Other Tests _
-1 ❌	unit	0m 17s	/patch-unit-root.txt	root in the patch failed.
+0 🆗	asflicense	0m 17s		ASF License check generated no output?
		4m 34s

Subsystem	Report/Notes
Docker	ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-161/1/artifact/yetus-precommit-check/output/Dockerfile
GITHUB PR	#161
Optional Tests	dupname asflicense javac javadoc unit codespell detsecrets xmllint compile
uname	Linux 646840ac6116 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	/home/jenkins/jenkins-home/workspace/Base-Thirdparty-PreCommit_PR-161/yetus-precommit-check/src/.yetus/personality.sh
git revision	master / b3084b9
Default Java	Eclipse Adoptium-17.0.11+9
Test Results	https://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-161/1/testReport/
Max. process+thread count	62 (vs. ulimit of 2000)
modules	C: . U: .
Console output	https://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-161/1/console
versions	git=2.34.1 maven=called xmllint=20913
Powered by	Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

[szlta]

@alta404 - thanks for your work on this. In order to resolve the conflicts and build issues can you rebase your patch please?

[Apache-HBase]

🎊 +1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	1m 41s		Docker mode activated.
			_ Prechecks _
+1 💚	dupname	0m 0s		No case conflicting files found.
+0 🆗	codespell	0m 0s		codespell was not available.
+0 🆗	detsecrets	0m 0s		detect-secrets was not available.
+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.
			_ master Compile Tests _
+1 💚	mvninstall	1m 1s		master passed
+1 💚	compile	0m 17s		master passed
+1 💚	javadoc	0m 7s		master passed
			_ Patch Compile Tests _
+1 💚	mvninstall	0m 43s		the patch passed
+1 💚	compile	0m 18s		the patch passed
+1 💚	javac	0m 18s		the patch passed
+1 💚	blanks	0m 0s		The patch has no blanks issues.
+1 💚	xmllint	0m 0s		No new issues.
+1 💚	javadoc	0m 7s		the patch passed
			_ Other Tests _
+1 💚	unit	0m 48s		root in the patch passed.
+1 💚	asflicense	0m 7s		The patch does not generate ASF License warnings.
		5m 19s

Subsystem	Report/Notes
Docker	ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-161/2/artifact/yetus-precommit-check/output/Dockerfile
GITHUB PR	#161
Optional Tests	dupname asflicense javac javadoc unit codespell detsecrets xmllint compile
uname	Linux dc60a5375306 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	/home/jenkins/jenkins-home/workspace/Base-Thirdparty-PreCommit_PR-161/yetus-precommit-check/src/.yetus/personality.sh
git revision	master / bd50787
Default Java	Eclipse Adoptium-17.0.11+9
Test Results	https://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-161/2/testReport/
Max. process+thread count	419 (vs. ulimit of 2000)
modules	C: . U: .
Console output	https://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-161/2/console
versions	git=2.34.1 maven=called xmllint=20913
Powered by	Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

[szlta]

+1 (non-binding)

@pankaj72981@NihalJain could you take a look as well please?

[Apache-HBase]

🎊 +1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	0m 27s		Docker mode activated.
			_ Prechecks _
+1 💚	dupname	0m 0s		No case conflicting files found.
+0 🆗	codespell	0m 0s		codespell was not available.
+0 🆗	detsecrets	0m 0s		detect-secrets was not available.
+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.
			_ master Compile Tests _
+1 💚	mvninstall	0m 43s		master passed
+1 💚	compile	0m 18s		master passed
+1 💚	javadoc	0m 7s		master passed
			_ Patch Compile Tests _
+1 💚	mvninstall	0m 41s		the patch passed
+1 💚	compile	0m 19s		the patch passed
+1 💚	javac	0m 19s		the patch passed
+1 💚	blanks	0m 0s		The patch has no blanks issues.
+1 💚	xmllint	0m 0s		No new issues.
+1 💚	javadoc	0m 6s		the patch passed
			_ Other Tests _
+1 💚	unit	0m 46s		root in the patch passed.
+1 💚	asflicense	0m 5s		The patch does not generate ASF License warnings.
		3m 43s

Subsystem	Report/Notes
Docker	ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-161/3/artifact/yetus-precommit-check/output/Dockerfile
GITHUB PR	#161
Optional Tests	dupname asflicense javac javadoc unit codespell detsecrets xmllint compile
uname	Linux 7efd77d1eb0f 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	/home/jenkins/jenkins-home/workspace/Base-Thirdparty-PreCommit_PR-161/yetus-precommit-check/src/.yetus/personality.sh
git revision	master / c45888b
Default Java	Eclipse Adoptium-17.0.11+9
Test Results	https://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-161/3/testReport/
Max. process+thread count	404 (vs. ulimit of 2000)
modules	C: . U: .
Console output	https://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-161/3/console
versions	git=2.34.1 maven=called xmllint=20913
Powered by	Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

[PDavid]

Thanks, looks good to me. 👍

[szlta]

@alta404 thanks for this PR, I think you can go ahead and merge it

[alta404]

@szlta looks like I don't have access to merge in this repo. Could you please merge it?

[PDavid]

Many thanks for your contribution @alta404! Also thanks @pankaj72981 and @szlta for your reviews!