readKey drops a null from keys with three or more leading nulls

Abierto
#13 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
3/5
Tiempo estimado
1-2 días
Aptitud para principiantes
67/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
javascript
Área
backend

Línea de trabajo

Start by reading the readKey and writeKey implementations and the tests for composite key round trips. Add cases for the reported keys with three or more leading nulls, including [null,null,null] and [null,null,null,null,5], and consider the related [5] decoder bug. Done means readKey(writeKey(k)) deep-equals each tested key.

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

Descripción

readKey loses a null from composite keys with three or more leading nulls:

  • writeKey([null,null,null,null,5]) gives 0000000000000000140140000000000000, and readKey returns [null,null,null,5].
  • writeKey([null,null,null]) gives 0000000000, and readKey returns [null,null,""].
  • [null,null,null,5] and shorter keys round-trip correctly. The encodings are all distinct, so the fault is in the decoder.

In Harper, a composite primary key of this shape comes back from a scan as an id that encodes to different bytes, so a get by that id can't find the record. Related decoder bug: HarperFast/harper#3777 ([5] decodes as 5); both could be fixed together.

Expected: readKey(writeKey(k)) deep-equals k.

Found while reviewing HarperFast/harper#3938.

🤖 Generated by Claude Opus 5.5 (Claude Code); posted via @kriszyp.

Lenguaje dominante
JavaScript
Estrellas
12
Forks
6
Métricas de merge de PR
Sin PR fusionados en 30 d

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

  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 kriszyp/ordered-binary

Todos los issues de kriszyp/ordered-binary

Issues similares

Más issues de JavaScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.