[BEAM-10961] Enable strict dependency checking on sdks/java/io/hadoop-format

[shehzaadn-vd]

Moved here from PR# 13688 per @ibzib's request.

R: @ibzib, please add other folks if needed.

---

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

• Choose reviewer(s) and mention them in a comment (R: @username).
• Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
• Update CHANGES.md with noteworthy changes.
• If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

Post-Commit Tests Status (on master branch)

Lang	SDK	Dataflow	Flink	Samza	Spark	Twister2
Go		---		---		---
Java
Python				---		---
XLang				---		---

Pre-Commit Tests Status (on master branch)

---	Java	Python	Go	Website	Whitespace	Typescript
Non-portable
Portable	---		---	---	---	---

See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

[ibzib]

LGTM, but we should get a second opinion on the log4j thing from someone who knows this module better. Perhaps @aromanenko-dev ?

[aromanenko-dev]

Run Java HadoopFormatIO Performance Test

[aromanenko-dev]

Well, I don't know why specific Log4j version was used for testing (it was added even before my code refactoring) but if all tests pass then, seems, we don't need that anymore. So, it LGTM in general for me.

Maybe @iemejia recalls something on this.

[aromanenko-dev]

@shehzaadn-vd Please, don't forget to add a Jira prefix to commit messages as well.

[iemejia]

Well, I don't know why specific Log4j version was used for testing (it was added even before my code refactoring) but if all tests pass then, seems, we don't need that anymore. So, it LGTM in general for me.

Maybe @iemejia recalls something on this.

No idea but in any case even if used it should have not had that scope but testRuntimeOnly so good to remove save if some test is not happy

[shehzaadn-vd]

@aromanenko-dev I didn't mean to ignore your comment about jira prefixes on commit messages. I just saw it late. Apologies. Hope I can get a pass this time!

[aromanenko-dev]

@shehzaadn-vd Not a problem, I'll squash before merge and fix a comment message