Skip to content

PHOENIX-7600 Replace commons-logging with slf4j#160

Open
meszinorbi wants to merge 1 commit intoapache:masterfrom
meszinorbi:PHOENIX-7600
Open

PHOENIX-7600 Replace commons-logging with slf4j#160
meszinorbi wants to merge 1 commit intoapache:masterfrom
meszinorbi:PHOENIX-7600

Conversation

@meszinorbi
Copy link
Copy Markdown

No description provided.

@stoty
Copy link
Copy Markdown
Contributor

stoty commented Jun 2, 2025

Please check the test logs

[INFO] Running org.apache.phoenix.hive.HiveMapReduceIT
Error: Tests run: 1, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 0.009 s <<< FAILURE! -- in org.apache.phoenix.hive.HiveMapReduceIT
Error: org.apache.phoenix.hive.HiveMapReduceIT -- Time elapsed: 0.002 s <<< ERROR!
java.lang.RuntimeException: java.lang.NoClassDefFoundError: org/apache/commons/logging/impl/Log4JLogger
at org.apache.phoenix.query.BaseTest.initMiniCluster(BaseTest.java:552)
at org.apache.phoenix.query.BaseTest.setUpTestCluster(BaseTest.java:451)
at org.apache.phoenix.query.BaseTest.checkClusterInitialized(BaseTest.java:437)

Looks like you need to add log4j-1.2-api to the test classpath.

@stoty
Copy link
Copy Markdown
Contributor

stoty commented Jun 2, 2025

Hmm. Maybe the problem is different.

It's missing the commons-logging classes, not the log4j12 ones.

In any case the test classpath is broken.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants