apache / apache/paimon-cpp

[Bug] Nullability validation incorrectly checks children masked by null parents

Fechada
#339 0 comentários 0 reações 0 responsáveis Ver no GitHub
bug
Linguagem predominante
C++
Estrelas
65
Forks
25
Merge médio
2d 9h
PRs com merge (30d)
82

Descrição

### Search before asking

- [x] I searched in the [issues](https://github.com/apache/paimon-cpp/issues) and found nothing similar.

### Paimon-cpp version

master

### Minimal reproduce step

`ArrowUtils::CheckNullabilityMatch` recursively checks complete Arrow child arrays.

For nested Arrow types, a null parent may still have physical child values. These
children are logically hidden and should not participate in nullability validation.
However, if a hidden child is null and its field is non-nullable, the current
implementation rejects the batch.

This affects nested STRUCT, LIST, MAP, and VECTOR fields and is inconsistent with
Java's logical row semantics.

### What doesn't meet your expectations?

Expected behavior:

- Ignore child values masked by a null parent.
- Reject null children when the parent is valid and the child is logically visible.

### Anything else?

_No response_

### Are you willing to submit a PR?

- [x] I'm willing to submit a PR!

Guia de contribuição

Abrir o guia de contribuição

Direção de pesquisa

Comece localizando ArrowUtils::CheckNullabilityMatch e os testes que abrangem arrays STRUCT, LIST, MAP e VECTOR aninhados. Verifique os casos em que pais nulos contêm filhos fisicamente nulos e os casos em que pais válidos expõem filhos nulos; considera-se concluído quando filhos mascarados são ignorados, enquanto filhos visíveis não anuláveis são rejeitados.

Escrita pelo modelo de indexação a partir do texto da issue.

Avaliação

Stack de tecnologia
cpp
Domínio
data-engineering
Tipo de issue
Bug
Dificuldade
3/5
Tempo estimado
1-2 dias
Status de atividade
Ativa
Clareza
Razoavelmente clara
Facilidade para iniciantes
58/100

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.