PaperMC / PaperMC/Paper

TrialSpawnerConfig Registry

Open
#11,722 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

status: accepted
Dominant language
Java
Stars
12.7k
Forks
3.5k
Avg merge
3d 13h
Merged PRs (30d)
11

Description

Is your feature request related to a problem?

Upstream's TrialSpawnerConfiguration is poorly designed and thought out. It should be a registry-related type that is immutable. The current design makes it impossible to store a registered config without serializing the whole thing to the trial spawner instead of just the reference key.

Describe the solution you'd like.

We should just deprecate that type and associated API for a new immutable type with registry entry builders for modification.

Describe alternatives you've considered.

N/A

Other

No response

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by locating TrialSpawnerConfiguration and its associated API, then trace how registered configurations are serialized into trial spawners. Done means the old type and API are deprecated and a new immutable registry-related type supports registry entry builders and reference-key storage without serializing the whole configuration.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
api, backend-api-design
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.