cube-js/cube

`chartPivot` and `tablePivot` methods of `ResultSet` don't throw error for un-decomposed datasets of `compareDateRange` type queries

Aberta

#8.050 aberto em 28 de mar. de 2024

Ver no GitHubGuia do colaborador
 (2 comentários) (0 reação) (0 responsável)Rust (1.965 forks)batch import
client:coregood first issuehelp wanted

Métricas do repositório

Stars
 (19.563 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

Describe the bug If you run a compareDateRange query, you will get a result set that needs to be "decomposed" first using the decompose method of the client library. Calling the rawData method on it would correctly emit this error:

Error: Method is not supported for a 'compareDateRangeQuery' query type. Please use decompose

However, calling tablePivot and chartPivot would not emit any errors—which is quite confusing for someone who doesn't know about decompose and quirks of compareDateRange.

Expected behavior Calling tablePivot and chartPivot on compareDateRange result set should emit the same error as shown above.

Version: 0.35.4

Guia do colaborador

Direção de pesquisa
Inspecione a classe ResultSet para encontrar os métodos chartPivot e tablePivot e, em seguida, adicione uma verificação para o tipo de consulta compareDateRange semelhante ao método rawData.
Pilha de tecnologia
javascript
Domain
backend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
JavaScript
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75