quickwit-oss/quickwit

Doc processing error with an object mapping and null value

Open

#3 670 ouverte le 20 juil. 2023

Voir sur GitHub
 (3 commentaires) (0 réactions) (1 assigné)Rust (553 forks)github user discovery
buggood first issue

Métriques du dépôt

Stars
 (11 300 stars)
Métriques de merge PR
 (Métriques PR en attente)

Description

While indexing GH archive dataset, I got this error:

2023-07-20T07:24:14.984Z  WARN quickwit_indexing::actors::doc_processor: error=ValueError("payload.pull_request.head.repo", "Expected an JSON Object, got null")

Here is the mapping:

doc_mapping:
  field_mappings:
    - name: id
      type: text
    - name: type
      type: text
      fast: true
    - name: actor
      type: object
      field_mappings:
      - name: login
        fast: true
        type: text
    - name: payload
      type: object
      field_mappings:
      - name: description
        type: text
        tokenizer: default
      - name: pull_request
        type: object
        field_mappings:
        - name: head
          type: object
          field_mappings:
            - name: repo
              type: object
              field_mappings:
                - name: description
                  type: text
                  tokenizer: default

I would expect that we just ignore the null value and still index the document.

Guide contributeur

Direction de recherche
Examinez le code du doc processor où il lance ValueError pour des objets JSON null alors qu'un objet est attendu. Modifiez le code pour ignorer les valeurs null au lieu d'échouer.
Stack technique
rust
Domaine
backend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
RustGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70