spuru9 commented on PR #1114:
URL:
https://github.com/apache/flink-kubernetes-operator/pull/1114#issuecomment-4517765679
2.
### Verification — jackson-bom 2.15.0 → 2.21.3
**Scope.** The bump affects the unshaded Jackson chain used by CRD types,
the admission webhook, and operator-owned serialization (`StatusRecorder`,
`SpecUtils`, `ConfigObjectNodeDeserializer`). It does
*not* affect the autoscaler state-store path — `AutoScalerSerDeModule`,
`KubernetesAutoScalerStateStore`, and `JdbcAutoScalerStateStore` all import
from `org.apache.flink.shaded.jackson2.*` (Flink-shaded
Jackson at 2.14.2-17.0, separate package namespace). Existing autoscaler
ConfigMap / JDBC state is therefore unaffected by this bump.
**Convergence.** `mvn dependency:tree -Dincludes='com.fasterxml.jackson*'`
across all 13 modules: `jackson-core` / `databind` / `dataformat-yaml` /
`dataformat-xml` / `datatype-jsr310` / `datatype-joda` /
`module-jsonSchema` all on **2.21.3**; `jackson-annotations` on **2.21**
(Jackson's canonical label for the annotations artifact). fabric8 7.3.1 and
Beam 2.62 / 2.73 both accept the BOM override. No version
mismatch in any module.
**API audit.** Exhaustive grep across `src/main` + `src/test` for patterns
that historically break between Jackson minors — **zero** hits on each:
| Pattern | Hits |
|---|---|
| `MapperFeature` / `SerializationFeature` / `DeserializationFeature`
configure | 0 |
| `enableDefaultTyping` / `activateDefaultTyping` /
`PolymorphicTypeValidator` (biggest 2.x breaking-change surface) | 0 |
| `StreamReadConstraints` / `StreamWriteConstraints` overrides | 0 |
| ServiceLoader / `findAndRegisterModules` | 0 |
| `ObjectMapper` subclasses | 0 |
| `@JsonDeserialize(builder=)`, exotic annotations (`@JsonValue`,
`@JsonAny*`, `@JsonNaming`, `@JsonFormat`, `@JsonAlias`, `@JsonMerge`,
`@JsonUnwrapped`, `@JsonView`, `@JsonIdentityInfo`, `@JsonAutoDetect`,
…) | 0 |
| Jdk8Module / ParameterNamesModule / version-pinned JSON test fixtures |
0 |
Annotation surface in production is limited to `@JsonInclude`,
`@JsonProperty`, `@JsonIgnore`, `@JsonIgnoreProperties`, `@JsonDeserialize`
(empty marker and `using=` form), `@JsonCreator` — all unchanged
2.15 → 2.21. The single production custom deserializer
(`ConfigObjectNodeDeserializer` on `AbstractFlinkSpec.flinkConfiguration`) uses
only `JsonParser.readValueAsTree()` and standard `ObjectNode` iteration.
**Tests.** Full reactor `mvn -B -ntp test`: **2,568 / 0 failures / 0
errors / 0 skipped**. Includes every test that exercises unshaded-Jackson paths
— `CrdCompatibilityCheckerTest`, `SpecUtilsTest`,
`ConfigObjectNodeTest`, `FlinkOperatorWebhookTest`,
`AdmissionHandlerTest`, `DefaultRequestMutatorTest`,
`FlinkResourceExceptionUtilsTest`.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
