Hacktoberfest 2026: as issues que os mantenedores marcaram para outubro, abertas e boas para iniciantes. Ver issues do Hacktoberfest

API change: Column methods should throw exceptions consistently

Aberta
#243 1 comentário 0 reações 1 responsável Ver no GitHub

Mantenedores costumam responder em até 1 dia

@Enmk já está trabalhando nisso.

Desde 24/11/2022.

Avaliação

Esta issue ainda não foi avaliada.

Descrição

3.0

There are multiple places throughout the library that are really relaxed about input values\types, like:

  • overrides of Column::Append(ColumnRef column) that just do nothing if column is of not expected type
  • overrides of Column::Load that return false if column failed to load, sometimes eating an exception that could point to the root cause.
  • etc.

Fixes scheduled for next API-breaking release (3.0?):

  • Column::Append overrides should throw an exception if the input column is of the wrong type.
  • Column::Load should change return type to void and throw an exception on error.
Linguagem predominante
C
Estrelas
385
Forks
211
Merge médio
18h 22min
PRs com merge (30d)
4

Preparar o ambiente

Este projeto não oferece contêiner de desenvolvimento, Dockerfile nem guia de contribuição, então a configuração fica por sua conta: comece pelo README e veja nosso guia da primeira contribuição para os passos gerais.

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de ClickHouse/clickhouse-cpp

Todas as issues de ClickHouse/clickhouse-cpp

Issues semelhantes

Mais issues de C

Receba novas issues na sua caixa de entrada

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