This is an automated email from the ASF dual-hosted git repository.

dongjoon pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/master by this push:
     new 591a60df788a [SPARK-49602][BUILD] Fix `assembly/pom.xml` to use 
`{project.version}` instead of `{version}`
591a60df788a is described below

commit 591a60df788ae72226375f2d3e85c203200b4b93
Author: Dongjoon Hyun <[email protected]>
AuthorDate: Wed Sep 11 13:03:25 2024 -0700

    [SPARK-49602][BUILD] Fix `assembly/pom.xml` to use `{project.version}` 
instead of `{version}`
    
    ### What changes were proposed in this pull request?
    
    This PR aims to fix `assembly/pom.xml` to use `{project.version}` instead 
of `{version}`.
    
    The original change was introduced recently by
    - #47402
    
    ### Why are the changes needed?
    
    **BEFORE**
    ```
    $ mvn clean | head -n9
    [INFO] Scanning for projects...
    [WARNING]
    [WARNING] Some problems were encountered while building the effective model 
for org.apache.spark:spark-assembly_2.13:pom:4.0.0-SNAPSHOT
    [WARNING] The expression ${version} is deprecated. Please use 
${project.version} instead.
    [WARNING]
    [WARNING] It is highly recommended to fix these problems because they 
threaten the stability of your build.
    [WARNING]
    [WARNING] For this reason, future Maven versions might no longer support 
building such malformed projects.
    [WARNING]
    ```
    
    **AFTER**
    ```
    $ mvn clean | head -n9
    [INFO] Scanning for projects...
    [INFO] 
------------------------------------------------------------------------
    [INFO] Detecting the operating system and CPU architecture
    [INFO] 
------------------------------------------------------------------------
    [INFO] os.detected.name: osx
    [INFO] os.detected.arch: aarch_64
    [INFO] os.detected.version: 15.0
    [INFO] os.detected.version.major: 15
    [INFO] os.detected.version.minor: 0
    ```
    
    ### Does this PR introduce _any_ user-facing change?
    
    No, this is a dev-only change for building distribution.
    
    ### How was this patch tested?
    
    Manual test.
    
    ### Was this patch authored or co-authored using generative AI tooling?
    
    No.
    
    Closes #48081 from dongjoon-hyun/SPARK-49602.
    
    Authored-by: Dongjoon Hyun <[email protected]>
    Signed-off-by: Dongjoon Hyun <[email protected]>
---
 assembly/pom.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/assembly/pom.xml b/assembly/pom.xml
index 8b21f7e808ce..4b074a88dab4 100644
--- a/assembly/pom.xml
+++ b/assembly/pom.xml
@@ -200,7 +200,7 @@
             <configuration>
               <executable>cp</executable>
               <arguments>
-                
<argument>${basedir}/../connector/connect/client/jvm/target/spark-connect-client-jvm_${scala.binary.version}-${version}.jar</argument>
+                
<argument>${basedir}/../connector/connect/client/jvm/target/spark-connect-client-jvm_${scala.binary.version}-${project.version}.jar</argument>
                 
<argument>${basedir}/target/scala-${scala.binary.version}/jars/connect-repl</argument>
               </arguments>
             </configuration>


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

Reply via email to