bug: main does not compile, MinHash calls build_prefilter without the metrics argument

Cerrado Apto para principiantes
  • ¿Algo no está bien?
#9,506 1 comentario 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 2 días

@lance-gatefixer ya está trabajando en esto.

Desde el 23/9/2026.

  • #9508 de @lance-gatefixer — abierto

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
75/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
rust
Área
build-system

Línea de trabajo

El problema está en rust/lance/src/io/exec/minhash.rs en la línea 235, donde se llama a build_prefilter con seis argumentos en lugar de siete. Mira la firma de la función en rust/lance/src/io/exec/utils.rs:370 para ver los parámetros requeridos. Examina cómo rust/lance/src/io/exec/fts.rs pasa el argumento metrics y replica eso. Después de hacer el cambio, ejecuta el compilador de Rust para verificar que la compilación sea exitosa.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

bug

main at 6f843df81 does not compile. build_prefilter takes seven parameters, the last one metrics: &ExecutionPlanMetricsSet (rust/lance/src/io/exec/utils.rs:370), while the call site added for the MinHash scalar index passes six (rust/lance/src/io/exec/minhash.rs:235):

error[E0061]: this function takes 7 arguments but 6 arguments were supplied
   --> rust/lance/src/io/exec/minhash.rs:235
error: could not compile `lance` (lib) due to 1 previous error

pub mod minhash; is declared unconditionally in rust/lance/src/io/exec/mod.rs:18, so every default build hits it: linux-build, windows, the Java jobs, the Python wheels, clippy and rustdoc all fail on PRs that merge current main.

The two changes are individually fine and conflict only semantically, so git merged both without a textual conflict:

  • #9471 added the metrics parameter and updated the call sites that existed at the time.
  • #9114 added minhash.rs with a new build_prefilter call, branched before #9471.

The fix is to pass a metrics set at that call site the way rust/lance/src/io/exec/fts.rs does.

Worth noting separately: nothing caught this before merge, because branch protection tests each PR against its own base rather than against the merge result. A merge-queue or a required rebase would have.

Lenguaje dominante
Rust
Estrellas
7.3k
Forks
924
Merge medio
3 d 19 h
PR fusionados (30 d)
546

Preparar el entorno

Abrir en Codespaces

Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de lance-format/lance

Todos los issues de lance-format/lance

Issues similares

Más issues de Rust

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.