Non-indexed JSONb operations
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 30/100
Línea de trabajo
Comienza en lib/ecto/adapters/postgres/connection.ex alrededor de la línea 1054 y reproduce los dos ejemplos de Repo.to_sql que comparan valores JSONB literales y parametrizados. Revisa cómo se representan las rutas JSONB de PostgreSQL, los valores parametrizados, la igualdad y la contención; se considera terminado cuando los casos no literales propuestos generan SQL compatible con índices GIN sin hacer regresiones en las consultas literales existentes.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Elixir version
Erlang/OTP 27 [erts-15.2.3] [source] [64-bit] [smp:16:16] [ds:16:16:10] [async-threads:1] [jit] Elixir 1.18.3 (compiled with Erlang/OTP 27)
Database and Version
PostgreSQL 17.4 on x86_64-pc-linux-gnu, compiled by gcc (GCC) 14.2.1 20240912 (Red Hat 14.2.1-3), 64-bit
Ecto Versions
3.12.5
Database Adapter and Versions (postgrex, myxql, etc)
0.20.0
Current behavior
When comparing jsonb paths with literals in where, Ecto generates efficient query using @>:
iex(1)> q = from j in Oban.Job, where: j.args["event_id"] == "123"
iex(2)> Repo.to_sql(:all, q) |> elem(0) |> IO.puts()
SELECT ... FROM "oban_jobs" AS o0 WHERE ((o0."args"@>'{"event_id": "123"}'))
However, if doing the same, but interpolating the value, e.g.:
iex(1)> q = from j in Oban.Job, where: j.args["event_id"] == ^"123"
iex(2)> Repo.to_sql(:all, q) |> elem(0) |> IO.puts()
SELECT ... FROM "oban_jobs" AS o0 WHERE ((o0."args"#>'{"event_id"}') = $1)
Ecto generates the query using #> operator, which isn't supported by GIN indexes
Expected behavior
It seems that this inefficiency comes from https://github.com/elixir-ecto/ecto_sql/blob/ad5e31c13b034564ff8bf3783bfb2e8ab7c0e6d0/lib/ecto/adapters/postgres/connection.ex#L1054, which is restricted only to literals, and not interpolations.
My proposal is to add support for pinned vars and other db columns with following cases:
-
data known in elixir:
SELECT ... FROM "oban_jobs" AS o0 WHERE (o0."args" @@ '$.event_id #{operator} #{Jason.encode value}') -
equality comparison:
SELECT ... FROM "oban_jobs" AS o0 WHERE (o0."args" @> jsonb_build_object($1, jsonb_build_object($2, ...)))where $1..$n-1 is path items and $n is the value.
Maybe it would be more optimal case for elixir-data equality comparisons since the query can be prepared -
containment check
iex(1)> q = from j in Oban.Job, where: %{event_id: "123"} in j.args iex(2)> Repo.to_sql(:all, q) |> elem(0) |> IO.puts() SELECT ... FROM "oban_jobs" AS o0 WHERE (o0."args" @> jsonb_build_object('event_id', '123'))The idea is to re-build the object using
jsonb_build_objectandjsonb_build_arraywhich would allow for specific values to be non-literals
- Lenguaje dominante
- Elixir
- Estrellas
- 657
- Forks
- 344
- Merge medio
- 2 h 16 min
- PR fusionados (30 d)
- 2
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de elixir-ecto/ecto_sql
-
Introduce Ecto.SQLRepoAbiertoKind:Enhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
elixir-ecto/ecto_sql#637 · 3 reacciones ·
-
Kind:Bug
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
elixir-ecto/ecto_sql#602 · 4 comentarios ·
Todos los issues de elixir-ecto/ecto_sql
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 92/100
semaphoreio/semaphore#1305 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
QuinnWilton/argus#5 · 1 comentario ·
-
CLI auth policy save crashes with NotFound when the AuthorizationSettings singleton is missingAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
carverauto/serviceradar#5009 ·
Los mantenedores suelen responder en 1 día