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.

Sign up for GitHub

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

Jump to bottom

Unify JSON library of Java packages. #2378

Closed
2 tasks
siyuan0322 opened this issue Jan 11, 2023 · 0 comments · Fixed by #2384
Closed
2 tasks

Unify JSON library of Java packages. #2378

siyuan0322 opened this issue Jan 11, 2023 · 0 comments · Fixed by #2384
Assignees
@siyuan0322 @zhanglei1949
Labels
component:gae component:gie

Comments

@siyuan0322
Copy link
Collaborator

siyuan0322 commented Jan 11, 2023 •
edited

We have multiple JSON libraries spreaded in a few packages (fastjson, Jackson, orjson, gson) to parse JSON, which is unnecessary, and I suggests to unified them to Jackson.
@zhanglei1949 zhanglei1949 mentioned this issue Jan 13, 2023
Merged
@siyuan0322 siyuan0322 mentioned this issue Jan 16, 2023
Merged
siyuan0322 added a commit that referenced this issue Jan 18, 2023
@siyuan0322
Clean up and fix Java dependency problems (#2384)
7836d85 
- Make dependencies and plugins in sub module use version defined in
parent pom.
- Resolve a lot of problems detected by `mvn dependency: analyze`
- Get rid of `org.json` and `fastjson`, unify json libraries to Jackson.
- Bump up protoc versions in order to download aarch binaries.
- Scope of odps related jars set to `provided`
- Make some test-only dependencies use scope `test`
- Remove some unused files.

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

@siyuan0322 siyuan0322

@zhanglei1949 zhanglei1949

Labels
component:gae component:gie
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Clean up and fix Java dependency problems siyuan0322/GraphScope
3 participants
@sighingnow @siyuan0322 @zhanglei1949