Hacker News new | ask | show | jobs
by alberth 874 days ago
This an incorrect title.

Per this comment:

https://sqlite.org/forum/forumpost/b6f940e87dd28cf8

> Fix your code with CASTs instead of complaining about it. You didn't follow the documentation

Essentially, the person was using json incorrectly (as per documentation even prior to this release) and now this version of SQLite is causing their incorrect code to not behave as they wanted.

But because SQLite doesn’t want to break backward compatibility (even for a prior unknown bug), they are considering persisting the bug as seen here:

https://sqlite.org/forum/forumpost/bcec95d836ad4048

3 comments

This response seems misleading, the "fix your code" reply isn't from the author of sqlite.

He considers it a bug (in an EARLIER version of having unspecified behavior, not a later version, but there's no difference in practice):

That you could use a BLOB as the left-hand side argument to -> in 3.44.0 is a previously unreported bug in 3.44.0.

This bug in prior versions of SQLite was unknown to the developers. If I had known about it, I would have done something about it. I probably would have tried to preserve the buggy behavior - to maintain "bug compatibility". But I didn't know.

So there is a bug -- which is fine, all software has bugs -- but it's not a USER error, which you're implying.

Yes, the response could be more patient, but ultimately, that's absolutely a user error.

If the documentation clearly says that something should not work, the same thing does not work in other DBs (-> semantics follow Postgres for example), and the definition of the underlying DB types does not really allow for this to work, then the smart thing is to not rely on the behavior (and open a bug report).

Personally, I don't think that "you're doing it wrong" by calling the behaviour previously expected to be a bug is a good excuse, and in fact is an example of being user-hostile. Imagine being on the receiving end. When the documentation says one thing but the actual behaviour is different, people will sensibly follow reality.
The GP seems to have a quoted a commenter in the SQLite thread, who not a maintainer or a representative of the community. The actual maintainer wrote:

> This bug in prior versions of SQLite was unknown to the developers. If I had known about it, I would have done something about it. I probably would have tried to preserve the buggy behavior - to maintain "bug compatibility". But I didn't know.

That comment is much more patient, and less easily misconstrued by drive-by commenters.

The person you responded to also didn't claim it was the maintainers who said it. This passer-by is still user-hostile and may still infuriate the person submitting the ticket who are trying to get their stuff to work again.
> But because SQLite doesn’t want to break backward compatibility (even for a prior unknown bug)

Having been bitten by them breaking compatibility by fixing a "bug" that worked for a long time[0], I would dispute this (albeit mildly.)

[0] Using CTEs with views.