search

mirromutth / r2dbc-mysql

R2DBC MySQL Implementation
Apache License 2.0
656 stars 98 forks source link

Consider JSON as pass-thru #15

Closed mp911de closed 5 years ago

mp911de commented 5 years ago

JSON mapping is typically an application-level concern (JSON-P, JSON-B) because all transformations and bindings are reflected on application or client-level. Having support directly in the driver comes with several drawbacks as drivers need to either commit to a single library or provide support for multiple implementations (Jackso, GSON, Apache Johnzon). This isn't typically the thing you want to deal with. Also, most JSON libraries work with blocking streams (InputStream, OutputStream) that is not Reactive-friendly.

Another approach would be to provide an extensible codec registry and optional codecs that users can register.

mirromutth commented 5 years ago

Ok, I will remove JSON support and type JSON will be treated as a VARCHAR.

mirromutth commented 5 years ago

JSON support has removed. See StringConverter in io.github.mirromutth.r2dbc.mysql.converter.