FasterXML / FasterXML/jackson-annotations

Make `@JsonIgnoreProperties` and `@JsonPropertyOrder` merge (optionally), not just override

Aperta
#82 3 commenti 0 reazioni 0 assegnatari Vedi su GitHub
Lingua principale
Java
Stelle
1.1k
Fork
342
Merge medio
8h 53m
PR unite (30g)
2

Descrizione

A small subset of array-valued annotations would benefit from ability to append/merge their values.
List includes at least:
- `@JsonIgnoreProperties` (default: simple union/merge)
- `@JsonPropertyOrder` (default: simple append?)

In these cases, current behavior of full replacement is probably not what is usually the expected behavior.
But additionally we should probably also allow alternate choice of "override"/"overwrite".

One obvious challenge is that since `Annotation` types can not be instantiated, we may have to figure out whether we could use value object approach (see `@JacksonAnnotationValue` for details) to help here.

The main challenge here is to combine two important aspects:
1. Try to keep annotation-merging/inheritance-handling/resolution code annotation agnostic and fully generic (that is, logic need not know semantics of actual annotation tpes)
2. Fully handle merging so that un-merged set of annotations need not be exposed to calling code.

So: big part of this is to see if merging is practical, and how.

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.