Skip to content

Conversation

hannesa2
Copy link
Contributor

@hannesa2 hannesa2 commented Jun 21, 2019

In https://fabric.io/ I see a lot of them
image

Because I want to reduce errors there, I want to get rid of it.

And as it looks like (I didn't debug in C code) you throw an error with that text, am I right ?

image

@developernotes
Copy link
Member

Hello @hannesa2 ,

After investigating, the "not an error" exceptions are legitimate problems. The message is being set to "not an error" because the problem did not originate from a low level SQLite error code, but one elsewhere in the JNI library. In other words, even though the message is "not an error" they are real exceptions that need to be handled and should not be suppressed. We do feel that a more helpful general error message should be used, which we will include in the next release. Because of this, we will not merge this change into master. Thanks!

@hannesa2 hannesa2 deleted the IgnoreNotAnError branch June 26, 2019 04:27
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.

2 participants