cloud-fan opened a new pull request, #55779:
URL: https://github.com/apache/spark/pull/55779

   ### What changes were proposed in this pull request?
   
   In `core/pom.xml`, the `<includeArtifactIds>` list under the 
`maven-dependency-plugin`'s `copy-dependencies` execution still references the 
old artifact ID `jetty-proxy`, while the actual `<dependency>` declared in the 
same file (and the corresponding `<include>` in the `maven-shade-plugin`) both 
use the renamed `jetty-ee10-proxy`. This PR renames `jetty-proxy` → 
`jetty-ee10-proxy` in the `<includeArtifactIds>` list to keep all three 
locations consistent.
   
   ### Why are the changes needed?
   
   Because there is no longer any artifact with id `jetty-proxy` on the 
classpath, the `copy-dependencies` execution silently skips the proxy JAR. With 
`SPARK_PREPEND_CLASSES`, Spark relies on the unshaded original JARs being 
copied to `target/`, so the proxy JAR is missing in that mode. Renaming the 
entry restores the intended behavior.
   
   ### Does this PR introduce _any_ user-facing change?
   
   No. Build-only fix.
   
   ### How was this patch tested?
   
   Existing build. Verified that the dependency block 
(`org.eclipse.jetty.ee10:jetty-ee10-proxy`) and the `maven-shade-plugin` 
`<include>` already use the same renamed artifact ID.
   
   ### Was this patch authored or co-authored using generative AI tooling?
   
   Generated-by: Claude Code (claude-opus-4-7)


-- 
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