Skip to content

avoid creating Jackson parser explicitly#1277

Merged
mkurz merged 1 commit into
playframework:mainfrom
pjfanning:avoid-createParser
Jul 15, 2026
Merged

avoid creating Jackson parser explicitly#1277
mkurz merged 1 commit into
playframework:mainfrom
pjfanning:avoid-createParser

Conversation

@pjfanning

@pjfanning pjfanning commented Dec 24, 2025

Copy link
Copy Markdown
Contributor

Pull Request Checklist

  • Have you read through the contributor guidelines?
  • Have you squashed your commits?
  • Have you added copyright headers to new files?
  • Have you updated the documentation?
  • Have you added tests for any changed functionality?

Fixes

Not needed if #1280 is merged

Purpose

What does this PR do?

Let Jackson create the parser itself under the hood.

Background Context

Why did you take this approach?

References

Are there any relevant issues / PRs / mailing lists discussions?

@gaeljw

gaeljw commented Jan 7, 2026

Copy link
Copy Markdown
Member

Somehow I missed this PR. I think it still makes sense even with #1280, I haven't changed the read methods. But we could merge this PR into the #1280 maybe indeed as it's similar.

@pjfanning
pjfanning force-pushed the avoid-createParser branch from 163d34a to 7ae9a9d Compare July 14, 2026 18:38
@pjfanning

Copy link
Copy Markdown
Contributor Author

@mkurz would it be possible to consider merging this?

@mkurz
mkurz merged commit 1942d66 into playframework:main Jul 15, 2026
15 checks passed
@mkurz

mkurz commented Jul 15, 2026

Copy link
Copy Markdown
Member

@pjfanning
pjfanning deleted the avoid-createParser branch July 15, 2026 10:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants