prestodb/presto

[native] Remove DuckDB dependency for TaskManagerTest and from Prestissimo

Aperta

#22.923 aperta il 5 giu 2024

Vedi su GitHubGuida contributor
 (1 commento) (2 reazioni) (1 assegnatario)Java (5240 fork)batch import
feature requestgood first issue

Metriche repository

Star
 (15.558 stelle)
Metriche merge PR
 (Merge medio 34g 14h) (120 PR mergiate in 30 g)

Descrizione

TaskManagerTest.cpp currently uses DuckDB to validate the results. This adds an additional dependency to Prestissimo. We must remove this. We must either modify the test to compare with expected values or use a Presto verifier.

Guida contributor

Direzione di ricerca
Ispeziona il file di test TaskManagerTest.cpp e sostituisci l'uso di DuckDB con confronti di valori previsti o un verificatore Presto.
Tech stack
cpp
Dominio
backend
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++GitTesting frameworks
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70