andygrove opened a new pull request, #3263:
URL: https://github.com/apache/datafusion-comet/pull/3263

   ## Summary
   - Fix TPC-H and TPC-DS verification jobs that fail with "Could not resolve 
dependencies" error
   - The jobs were failing because Maven couldn't find `comet-common` when 
running from the `spark/` subdirectory
   
   ## Root Cause
   The "Generate data" steps run Maven from within the spark/ subdirectory:
   ```bash
   cd spark && MAVEN_OPTS='-Xmx20g' ../mvnw -B -Prelease exec:java ...
   ```
   
   When Maven runs from a submodule directory, it doesn't see the reactor build 
and tries to resolve `comet-common` as a dependency from `~/.m2/repository` or 
remote repos. Since the previous "Build project" step only ran `compile 
test-compile`, the artifact was never installed locally.
   
   ## Fix
   Change `compile test-compile -DskipTests` to `install -DskipTests` so that 
the common module JAR is installed to the local Maven repository before the 
data generation step.
   
   ## Test plan
   - [ ] CI passes on this PR
   - [ ] TPC-H and TPC-DS verification jobs complete successfully
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to