Support Decimal128 literals in predicate pushdown
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 35/100
Línea de trabajo
Start in the predicate pushdown path where scalar_value_to_datum converts DataFusion scalar values into Iceberg datums, and find where Decimal128 currently falls through as unsupported. Read the linked #59 cast-stripping discussion first, since the issue says this should merge after it. Done means exact conversion of representable Decimal128 literals, with tests for comparisons, IN lists, negatives, NULLs, precision boundaries, and casts that are left to DataFusion.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Hi there,
Decimal128 literals are currently unsupported by
scalar_value_to_datum, so decimal comparisons are omitted from
the predicate passed to Iceberg.
For a price DECIMAL(18,2) column, for example:
SELECT id FROM prices
WHERE price > CAST('12.34' AS DECIMAL(18,2));
DataFusion evaluates the filter, but Iceberg cannot use that
decimal predicate for pruning.
Support exact conversion of representable Decimal128 literals,
preserving their value, precision, and scale. Leave unsupported
literals to DataFusion.
Cover comparisons and IN predicates, negative values, NULLs,
precision boundaries, and conservative handling of casts.
This should merge after #59: adding decimal conversion exposes
the existing cast-stripping problem for comparisons that require
type coercion.
I have a draft implementation and would be happy to contribute it
- Lenguaje dominante
- Rust
- Estrellas
- 30
- Forks
- 30
- Merge medio
- 3 d 19 h
- PR fusionados (30 d)
- 15
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 apache/datafusion-iceberg
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
apache/datafusion-iceberg#100 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
apache/datafusion-iceberg#85 · 1 comentario · 5 reacciones ·
Los mantenedores suelen responder en 1 día
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
apache/datafusion-iceberg#84 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
apache/datafusion-iceberg#75 ·
Los mantenedores suelen responder en 1 día
-
Slow CI runnersAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 18/100
apache/datafusion-iceberg#74 ·
Los mantenedores suelen responder en 1 día
Todos los issues de apache/datafusion-iceberg
Issues similares
-
bug i18n priority: medium ui
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
storytold/craft-anything#983 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
`swf` upgrade to 0.4.0Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
Dinnerbone/Rascal#91 · 2 reacciones ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
higress-group/HiRoute#405 ·
Los mantenedores suelen responder en 1 día
-
rust_analyzer flycheck: no CLI way to enable clippy from `check.overrideCommand`Posiblemente ocupada @TrapsterDK la tomó hoy. Abiertoneeds-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
bazelbuild/rules_rust#4392 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día