Skip to content

fix(it): restore runITs compileability after dependency/API drift#14

Open
dronsv wants to merge 1 commit into
mainfrom
fix/it-harness-compile-drift
Open

fix(it): restore runITs compileability after dependency/API drift#14
dronsv wants to merge 1 commit into
mainfrom
fix/it-harness-compile-drift

Conversation

@dronsv
Copy link
Copy Markdown
Owner

@dronsv dronsv commented May 5, 2026

Summary

Restores src/it/java compileability under runITs after dependency/API drift.

Changes:

  • Migrates lingering IT imports from org.apache.commons.lang.StringUtils to org.apache.commons.lang3.StringUtils.
  • Updates two FastBatchingCellReader.BatchLoader IT call sites to use the current 6-argument constructor.

Scope

Maintenance-only. No production behavior changes.

Verification

  • runITs IT-classes compile step now passes.
  • Previous blockers are gone:
    • missing legacy commons-lang
    • stale 4-arg BatchLoader constructor calls

Runtime IT execution is still blocked by an environment issue:

  • MySQL spinup / docker-in-docker socket problem

Context

This unblocks later runtime verification of Phase 8d's XMLA Discover telemetry IT:

mvn verify -DrunITs \
  -Dit.test=XmlaDiscoverNativeSqlTelemetryTest \
  -Dmaven.test.failure.ignore=true

- Migrate lingering IT imports from commons-lang to commons-lang3.
- Update BatchLoader IT call sites for the current 6-arg constructor.

This is a maintenance-only branch to unblock runtime verification of
Phase 8d's XMLA Discover telemetry IT.
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.

1 participant