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.

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

Add doc for fail.on.deserialization.error #462

Merged
merged 2 commits into from
Nov 21, 2017
Merged

Conversation

dguy
Copy link
Contributor

@dguy dguy commented Nov 20, 2017

No description provided.

Copy link

@bluemonk3y bluemonk3y left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

docs/examples.md Outdated
@@ -304,6 +304,11 @@ Common configuration properties that you might want to change from their default
- [cache.max.bytes.buffering](https://kafka.apache.org/documentation/#streamsconfigs):
The default value in KSQL is `10000000` (~ 10 MB);

- fail.on.deserialization.error:
When set to true (the default), any errors that occur when deserializing a record will result in the the error being logged and the record being dropped.
If you wish to halt processing on deserialization errors you should set this to false.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

seems a bit counter-intuitive ? i.e. set the "fail on error" flag to false to NOT halt ("fail") on error ? i predict significant user confusion for this...

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

doh! thanks for pointing out. It is the other way around!

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lol - makes much more sense now!

@bluemonk3y
Copy link

@dguy - think you could also add a mention of it on the bottom of this page (where it talks about config)?

https://github.com/confluentinc/ksql/blob/0.1.x/docs/syntax-reference.md#syntax-reference

Copy link
Contributor

@apurvam apurvam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM thanks!

@dguy dguy merged commit 0c51411 into confluentinc:4.0.x Nov 21, 2017
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.

4 participants