Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[HOTFIX] fix compile paimon error by broken maven version for net.minidev:json-smart #6442

Closed
wants to merge 4 commits into from

Conversation

FANNG1
Copy link
Contributor

@FANNG1 FANNG1 commented Feb 12, 2025

What changes were proposed in this pull request?

Temporarily locking in to avoid netplex/json-smart-v2#240

Why are the changes needed?

Couldn't compile Gravitino code

Execution failed for task ':catalogs:catalog-lakehouse paimon:compileJava'.

> Could not resolve all files for configuration ':catalogs:catalog-lakehouse-paimon: compileclasspath'

> Could not find any version that matches net.minidev:json-smart:[1.3.1,2.3].

Versions that do not match: 2.5.2

Searched in the following locations:
- https: //repo.maven.apache.org/maven2/net/minidev/json-smart/maven-metadata.xml

file:/home/runner/.m2/repository/net/minidev/json-smart/

Required by:
project :catalogs:catalog-lakethouse-paimon > org.apache.hadop:hadop-comon:2.10.2 > org.apache.hadoop:hadop-auth2.10.2 > com.nimbusds:nimbus-jose-jwt:7.g

Does this PR introduce any user-facing change?

no

How was this patch tested?

compile success in local

@FANNG1 FANNG1 changed the title [HOT_FIX] fix compile paimon error by broken maven version for net.minidev:json-smart [HOTFIX] fix compile paimon error by broken maven version for net.minidev:json-smart Feb 12, 2025
@jerryshao
Copy link
Contributor

Do we still need this PR? @FANNG1

@FANNG1
Copy link
Contributor Author

FANNG1 commented Feb 13, 2025

Do we still need this PR? @FANNG1

The upstream has fixed the broken maven state and Gravitino works normally now, close this PR.

@FANNG1 FANNG1 closed this Feb 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants