lance-format/lance

Python: fix broken fragment APIs

Open

#1006 aperta il 22 giu 2023

Vedi su GitHub
 (0 commenti) (0 reazioni) (1 assegnatario)Rust (695 fork)github user discovery
A-pythongood first issue

Metriche repository

Star
 (6582 star)
Metriche merge PR
 (Merge medio 6g 1h) (219 PR mergiate in 30 g)

Descrizione

The APIs around getting and manipulating fragments are inconsistent with their return types.

In many cases, the type hint is wrong. For example, LanceFragment.create() claims to return LanceFragment (which would make sense), but instead returns _FragmentMetadata. Also odd that _FragmentMetadata has an underscore even though it is part of a public API.

We should refactor this to be consistent. Would also be nice to start enforcing the type hints with mypy.

Guida contributor

Direzione di ricerca
Esamina le classi API dei frammenti (LanceFragment, FragmentMetadata) e i loro metodi per identificare tipi di ritorno e suggerimenti di tipo incoerenti. Determina il tipo di ritorno corretto per ogni metodo e garantisci coerenza. Quindi aggiorna i suggerimenti di tipo e rifattorizza il codice.
Tech stack
python
Dominio
backend
Tipo issue
Refactoring
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
Pythonmypy
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60