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.
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
Improve UX of the CDC API #21536
Improve UX of the CDC API #21536
Changes from 17 commits
9c038df
13c3ae8
30ee22c
46e804a
40c56bc
ca92caa
310ed1b
267fa1c
23e1d85
f828229
4848a2e
94f559c
c6a5248
9b12103
6873e0b
534d32d
6c442d9
c240949
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -24,7 +24,7 @@ | |
* @since Jet 4.2 | ||
*/ | ||
@EvolvingApi | ||
public class ParsingException extends Exception { | ||
public class ParsingException extends RuntimeException { | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why this change? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Most of the operations now are not lazy-parsing the json, so no exception is really expected. However, if user already used methods like There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I wonder if we should mark this class as deprecated then? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It is still used in lazy-parsing in RecordPartImpl, although I'm not against changing this to other exception type. |
||
|
||
private static final long serialVersionUID = 1L; | ||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The change record is marked with
@EvolvingApi
so we can remove thethrows ParsingException
if it's not the case anymore.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, done