Skip to content

Conversation

@hawk9821
Copy link
Contributor

Purpose of this pull request

mvn clean package -pl :seatunnel-dist -am -D"skip.ui"=true -DskipTests -Prelease throw error

[INFO] Scanning for projects...
[ERROR] [ERROR] Some problems were encountered while processing the POMs:
[WARNING] 'profiles.profile[seatunnel].dependencies.dependency.(groupId:artifactId:type:classifier)' must be unique: org.apache.seatunnel:connector-doris:jar -> duplicate declaration of version ${project.version} @ org.apache.seatunnel:seatunnel-dist:${revision}, E:\gitwork\seatunnel\seatunnel-dist\pom.xml, line 487, column 29
[ERROR] 'dependencies.dependency.version' for com.facebook.presto:presto-jdbc:jar must be a valid version but is '${presto.version}'. @ org.apache.seatunnel:seatunnel-dist:${revision}, E:\gitwork\seatunnel\seatunnel-dist\pom.xml, line 990, column 30
[ERROR] 'dependencies.dependency.version' for io.trino:trino-jdbc:jar must be a valid version but is '${trino.version}'. @ org.apache.seatunnel:seatunnel-dist:${revision}, E:\gitwork\seatunnel\seatunnel-dist\pom.xml, line 996, column 30
 @
[ERROR] The build could not read 1 project -> [Help 1]
[ERROR]
[ERROR]   The project org.apache.seatunnel:seatunnel-dist:2.3.12-SNAPSHOT (E:\gitwork\seatunnel\seatunnel-dist\pom.xml) has 2 errors
[ERROR]     'dependencies.dependency.version' for com.facebook.presto:presto-jdbc:jar must be a valid version but is '${presto.version}'. @ org.apache.seatunnel:seatunnel-dist:${revision}, E:\gitwork\seatunnel\seatunnel-dist\pom.xml, line 990, column 30
[ERROR]     'dependencies.dependency.version' for io.trino:trino-jdbc:jar must be a valid version but is '${trino.version}'. @ org.apache.seatunnel:seatunnel-dist:${revision}, E:\gitwork\seatunnel\seatunnel-dist\pom.xml, line 996, column 30
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/ProjectBuildingException

Does this PR introduce any user-facing change?

How was this patch tested?

Check list

@github-actions github-actions bot added the core SeaTunnel core module label Jun 13, 2025
@nielifeng nielifeng requested a review from Copilot June 13, 2025 06:36
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR fixes the release packaging exception by removing duplicate dependency declarations using invalid version placeholders.

  • Removed the presto and trino version properties.
  • Removed the presto-jdbc and trino-jdbc dependency blocks that used those properties.
Comments suppressed due to low confidence (2)

seatunnel-dist/pom.xml:111

  • Removing the 'presto.version' property resolves the version placeholder error; please verify that no build configuration relies on this property.
<presto.version>0.279</presto.version>

seatunnel-dist/pom.xml:984

  • The removal of the presto-jdbc and trino-jdbc dependency blocks eliminates duplicate declaration issues and invalid version references; ensure these dependencies are not required for release functionality.
<dependency> block for presto-jdbc and trino-jdbc

<classifier>optional</classifier>
<scope>provided</scope>
</dependency>
<dependency>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

move to profile id seatunnel?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

modified

@Hisoka-X
Copy link
Member

waiting test case passes.

@Hisoka-X Hisoka-X merged commit a7675d1 into apache:dev Jun 19, 2025
6 checks passed
@hawk9821 hawk9821 deleted the devlop branch June 20, 2025 09:14
chncaesar pushed a commit to chncaesar/seatunnel that referenced this pull request Jun 30, 2025
dybyte pushed a commit to dybyte/seatunnel that referenced this pull request Jul 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved core SeaTunnel core module reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants