Skip to content

Conversation

@simhadri-g
Copy link
Member

@simhadri-g simhadri-g commented Jul 19, 2023

…ut should be a constant.

What changes were proposed in this pull request?

Patched-iceberg-core pom 'version' contains an expression but should be a constant.
Future Maven versions might no longer support building such pom files.

Why are the changes needed?

[WARNING]
[WARNING] Some problems were encountered while building the effective model for org.apache.hive:patched-iceberg-api:jar:patched-1.3.0-4.0.0-beta-1-SNAPSHOT
[WARNING] 'version' contains an expression but should be a constant. @ org.apache.hive:patched-iceberg-api:patched-${iceberg.version}-${project.parent.version}, /Users/simhadri.govindappa/Documents/apache/hive/iceberg/patched-iceberg-api/pom.xml, line 12, column 12
[WARNING]
[WARNING] Some problems were encountered while building the effective model for org.apache.hive:patched-iceberg-core:jar:patched-1.3.0-4.0.0-beta-1-SNAPSHOT
[WARNING] 'version' contains an expression but should be a constant. @ org.apache.hive:patched-iceberg-core:patched-${iceberg.version}-${project.parent.version}, /Users/simhadri.govindappa/Documents/apache/hive/iceberg/patched-iceberg-core/pom.xml, line 12, column 12
[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]

Does this PR introduce any user-facing change?

No

Is the change a dependency upgrade?

No

How was this patch tested?

manual

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants