Skip to content

Handle serde-field that fails to deserialize#3129

Closed
jleibs wants to merge 4 commits intomainfrom
jleibs/safer_serde_json
Closed

Handle serde-field that fails to deserialize#3129
jleibs wants to merge 4 commits intomainfrom
jleibs/safer_serde_json

Conversation

@jleibs
Copy link
Copy Markdown
Contributor

@jleibs jleibs commented Aug 28, 2023

What

arrow2-convert forces us into needing to provide data even if serde fail to deserialize the field. Fall back to default in this case, but specify invalid.

Resolves: #3127

Checklist

@jleibs jleibs marked this pull request as ready for review August 28, 2023 17:51
@jleibs jleibs added this to the 0.8.2 milestone Aug 28, 2023
@jleibs jleibs added the 🟦 blueprint The data that defines our UI label Aug 28, 2023
@jleibs jleibs closed this Aug 28, 2023
@jleibs jleibs deleted the jleibs/safer_serde_json branch August 28, 2023 18:57
@jleibs jleibs removed this from the 0.8.2 milestone Aug 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🟦 blueprint The data that defines our UI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Changes to serde-field data cause crash when loading blueprint

2 participants