Work around a bug in boolean values in Haystack's Elasticsearch 7 backend.

Review Request #12882 — Created March 10, 2023 and submitted

Information

Review Board
release-5.0.x

Reviewers

Haystack's Elasticsearch 7 backend has a bug where boolean values were
being serialized as True or False. Starting in Elasticsearch 7,
these must explicitly be true or false.

We ship copies of the Elasticsearch backends, originally intended as
backports for Review Board 4, but this gives us the opportunity to work
around this in our end. Since Haystack hasn't had a release in a year,
and this bug was reported to them about that long ago, we can't rely on
a version bump any time soon.

This change patches our copy of the Elasticsearch 7 backend to properly
serialize booleans.

The Haystack bug report is available at:

https://github.com/django-haystack/django-haystack/issues/1851

Set up Elasticsearch 7 and triggered the bug. Verified that the fix
allowed me to properly set up a search involving booleans.

Summary ID
Work around a bug in boolean values in Haystack's Elasticsearch 7 backend.
Haystack's Elasticsearch 7 backend has a bug where boolean values were being serialized as `True` or `False`. Starting in Elasticsearch 7, these must explicitly be `true` or `false`. We ship copies of the Elasticsearch backends, originally intended as backports for Review Board 4, but this gives us the opportunity to work around this in our end. Since Haystack hasn't had a release in a year, and this bug was reported to them about that long ago, we can't rely on a version bump any time soon. This change patches our copy of the Elasticsearch 7 backend to properly serialize booleans.
596b3d323b72768e8c4cc264c40ab1d38b32cb52
david
  1. Ship It!
  2. 
      
maubin
  1. Ship It!
  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-5.0.x (625e273)
Loading...