jackson-module-jsonSchema
jackson-module-jsonSchema copied to clipboard
Use BigDecimal instead of Double in NumberFormat
JSON can have number values outside the range of double. Also, it's important that multipleOf have an exact representation to be interpreted correctly. BigDecimal is better for these reasons.
I can see why this could be an improvement, but it would also be a backwards-incompatible change, and possibly break existing code. Is there anything we could do to alleviate those problems? Otherwise simple upgrade of module to 2.7 (where this could be merged) could break existing deployments.
We could have two functions like:
private BigDecimal minimum;
public Double getMinimum() { ... }
@JsonIgnore
public BigDecimal getMinimumDecimal() { ... }
This would avoid breaking existing code, at the cost of carrying a little bit of legacy cruft forever.