terminusdb / terminusdb/terminusdb-client-python
All database errors are DatabaseErrors
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 80
- Forks
- 62
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
Almost all database errors return the DatabaseError exception. They should return a specific exception instead as a DatabaseException can mean almost anything and catch code for specific errors can become very dirty.
See for instance: https://github.com/terminusdb/terminusdb-client-python/blob/0779fe4d3f21ff536b7ca09153bc8764d74c951a/terminusdb_client/client/Client.py#L1452-L1456
This could would be more elegant if DocumentNotFound was a specific exception.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginnen Sie mit der Überprüfung der in Client.py in den Zeilen 1452-1456 referenzierten Ausnahmebehandlung und ermitteln Sie, wie Datenbankfehler derzeit nach außen gegeben werden. Definieren Sie spezifische Ausnahmen für unterschiedliche Fehlerfälle, einschließlich DocumentNotFound, und überprüfen Sie, dass Aufrufer diese Fälle abfangen können, ohne auf die generische DatabaseError angewiesen zu sein.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- api
- Issue-Typ
- Refactoring
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100