bazel-contrib / bazel-contrib/rules_jvm_external

Use built-in Bazel JSON parser

Open
#472 2 comments 1 reaction 0 assignees View on GitHub
type: feature request
Dominant language
Java
Stars
373
Forks
301
Avg merge
7d 17h
Merged PRs (30d)
3

Description

With our maven_install.json it takes about 10-20 seconds to parse the JSON. Bazel just added a JSON parser so if this package switched to it, that bottleneck would go away.

https://github.com/bazelbuild/bazel/commit/6e47a409914d9f2ccb169482252f142c9fb503b8

Contributor guide

Open the contributing guide

Research direction

Start with the current code path that parses maven_install.json, then read the linked Bazel commit to understand the built-in JSON parser. Compare the existing and replacement paths, and verify that parsing no longer accounts for the reported 10–20 second bottleneck.

Written by the indexing model from the issue text.

Assessment

Tech stack
json
Domain
build-system
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.