Skip to content

Fix GlueDeserializer to use DeserializationContext instead of ObjectMapper#211

Merged
Randgalt merged 1 commit into
trinodb:mainfrom
ksobolew:kudi/json-trailing-tokens
Jul 15, 2026
Merged

Fix GlueDeserializer to use DeserializationContext instead of ObjectMapper#211
Randgalt merged 1 commit into
trinodb:mainfrom
ksobolew:kudi/json-trailing-tokens

Conversation

@ksobolew

Copy link
Copy Markdown
Contributor

Calling the parser directly corrupts the parser state in the enclosing parse. As a result, we may end up with an error if the FAIL_ON_TRAILING_TOKENS flag is enabled in the parser.

…apper

Calling the parser directly corrupts the parser state in the enclosing
parse. As a result, we may end up with an error if the
`FAIL_ON_TRAILING_TOKENS` flag is enabled in the parser.
@cla-bot cla-bot Bot added the cla-signed label Jul 13, 2026
@Laonel
Laonel requested a review from Randgalt July 14, 2026 06:15
@ksobolew

Copy link
Copy Markdown
Contributor Author

@Randgalt can you merge please? Looks like I'm not trusted enough to do it ;)

@Randgalt
Randgalt merged commit 1f36216 into trinodb:main Jul 15, 2026
2 checks passed
@ksobolew
ksobolew deleted the kudi/json-trailing-tokens branch July 15, 2026 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants