HBASE-29454: Update hbase-examples scripts to be compatible with Python 3

[kgeisz]

https://issues.apache.org/jira/browse/HBASE-29454

Some Python scripts in hbase-examples were never updated from Python 2 to Python 3. This pull request updates those scripts to be compatible with Python 3. The scripts were updated using 2to3 -w <script>. They were verified using python3 -m compileall <script>. Python 3.11.12 was used for the conversion and verification.

This pull request also renames the gen-py directory found in hbase-examples/src/main/python/thrift1 and hbase-examples/src/main/python/thrift2 to gen_py. This is done because the gen_py module needs to be added to the import statement in the Python scripts in order to prevent import errors. Python does not allow dashes in module names, so the dash in the module name must be replaced with an underscore.

[taklwu]

LGTM, mostly just add those missing parentheses for print function

[taklwu]

can you also check if hbase-server/src/main/python/hbase/merge_conf.py has anything to do with python3?

[Copilot]

Pull Request Overview

This pull request updates Python scripts in the HBase examples to be compatible with Python 3. The changes were made using the 2to3 tool and address syntax differences between Python 2 and Python 3.

• Updates print statements from Python 2 syntax to Python 3 function calls
• Converts exception handling from Python 2 to Python 3 syntax
• Updates string formatting to use modern f-string syntax
• Replaces deprecated print statement redirection with modern file parameter

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File	Description
hbase-server/src/main/python/hbase/merge_conf.py	Updates stderr print redirection to use file parameter
hbase-examples/src/main/python/thrift2/DemoClient.py	Converts print statements to function calls
hbase-examples/src/main/python/thrift1/demo_hbase_thrift_over_http_tls.py	Updates print statements and string formatting to Python 3 syntax
hbase-examples/src/main/python/thrift1/DemoClient.py	Comprehensive Python 3 conversion including print statements, exception handling, and error raising

need another round of review

[Kota-SH]

LGTM.

nit: Consider updating the shebangs for consistency with other py scripts.

[kgeisz]

retest

[Apache-HBase]

🎊 +1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	0m 33s		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.
+0 🆗	shelldocs	0m 1s		Shelldocs was not available.
+1 💚	@author	0m 0s		The patch does not contain any @author tags.
			_ master Compile Tests _
+0 🆗	mvndep	0m 32s		Maven dependency ordering for branch
+1 💚	mvninstall	4m 9s		master passed
+1 💚	spotless	0m 56s		branch has no errors when running spotless:check.
			_ Patch Compile Tests _
+0 🆗	mvndep	0m 11s		Maven dependency ordering for patch
+1 💚	mvninstall	3m 36s		the patch passed
+1 💚	blanks	0m 0s		The patch has no blanks issues.
-0 ⚠️	pylint	1m 24s	/results-pylint.txt	The patch generated 3 new + 2022 unchanged - 4 fixed = 2025 total (was 2026)
+1 💚	shellcheck	0m 0s		No new issues.
+1 💚	spotless	0m 51s		patch has no errors when running spotless:check.
			_ Other Tests _
+1 💚	asflicense	0m 46s		The patch does not generate ASF License warnings.
		14m 47s

Subsystem	Report/Notes
Docker	ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7161/6/artifact/yetus-general-check/output/Dockerfile
GITHUB PR	#7161
JIRA Issue	HBASE-29454
Optional Tests	dupname asflicense codespell detsecrets spotless javac pylint shellcheck shelldocs
uname	Linux 7f87da62357d 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	dev-support/hbase-personality.sh
git revision	master / 6a8dcb2
Default Java	Eclipse Adoptium-17.0.11+9
Max. process+thread count	83 (vs. ulimit of 30000)
modules	C: hbase-server hbase-examples . U: .
Console output	https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7161/6/console
versions	git=2.34.1 maven=3.9.8 pylint=2.15.5 shellcheck=0.8.0
Powered by	Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

[Apache-HBase]

🎊 +1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	0m 28s		Docker mode activated.
-0 ⚠️	yetus	0m 2s		Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck
			_ Prechecks _
			_ master Compile Tests _
+0 🆗	mvndep	0m 10s		Maven dependency ordering for branch
+1 💚	mvninstall	3m 14s		master passed
+1 💚	javadoc	2m 35s		master passed
			_ Patch Compile Tests _
+0 🆗	mvndep	0m 12s		Maven dependency ordering for patch
+1 💚	mvninstall	3m 2s		the patch passed
+1 💚	javadoc	2m 36s		the patch passed
			_ Other Tests _
+1 💚	unit	285m 18s		root in the patch passed.
		304m 43s

Subsystem	Report/Notes
Docker	ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7161/6/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
GITHUB PR	#7161
JIRA Issue	HBASE-29454
Optional Tests	javac javadoc unit
uname	Linux f83cf3250189 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	dev-support/hbase-personality.sh
git revision	master / 6a8dcb2
Default Java	Eclipse Adoptium-17.0.11+9
Test Results	https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7161/6/testReport/
Max. process+thread count	8619 (vs. ulimit of 30000)
modules	C: hbase-server hbase-examples . U: .
Console output	https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7161/6/console
versions	git=2.34.1 maven=3.9.8
Powered by	Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.