Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Replace checked JsonProcessingException with unchecked JacksonException #661

Closed
cowtowncoder opened this issue Jan 12, 2021 · 0 comments · Fixed by #666
Closed

Replace checked JsonProcessingException with unchecked JacksonException #661

cowtowncoder opened this issue Jan 12, 2021 · 0 comments · Fixed by #666
Labels
3.x Issues to be only tackled for Jackson 3.x, not 2.x

Comments

@cowtowncoder
Copy link
Member

As described in more detail under JSTEP-4, it'd be good to change existing Jackson exception hierarchy (now based on JacksonException, added in 2.12) to extend unchecked RuntimeException instead of checked IOException. This is a major backwards-incompatible change and will need to go in Jackson 3.0.
@cowtowncoder cowtowncoder added the 3.x Issues to be only tackled for Jackson 3.x, not 2.x label Jan 12, 2021
cowtowncoder added a commit that referenced this issue Jan 12, 2021
@cowtowncoder
Start work on #661 (JacksonException -> unchecked); reader-side conve…
e43c86e 
…rted
cowtowncoder added a commit that referenced this issue Jan 12, 2021
@cowtowncoder
Second step for #661: remove IOException from generators
cde9443
cowtowncoder added a commit that referenced this issue Jan 12, 2021
@cowtowncoder
Third part of #661: remove IOException from factories too. Close to f…
e9bf9b6 
…ull eradication for core
@cowtowncoder cowtowncoder mentioned this issue Jan 12, 2021
Closed
cowtowncoder added a commit that referenced this issue Jan 12, 2021
@cowtowncoder
Bit more work on #661 to centralize IOException wrapping with helper …
6ad2009 
…methods
This was referenced Jan 16, 2021
Closed
Closed
Merged
cowtowncoder added a commit that referenced this issue Jan 20, 2021
@cowtowncoder
Implement #661: change JacksonException (and all its subtypes) to u…
491b6d9 
…nchecked (#666)

Change `JacksonException` to extend `RuntimeException`; make necessary API changes wrt `IOException`.
cowtowncoder added a commit that referenced this issue Jan 20, 2021
@cowtowncoder
More clean up wrt #661
30b3c81
@renannprado renannprado mentioned this issue Jun 29, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
3.x Issues to be only tackled for Jackson 3.x, not 2.x
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Implement #661: change JacksonException (and all its subtypes) to unchecked FasterXML/jackson-core
1 participant
@cowtowncoder