Allow multiple `do_node_execute` adapters
I maintainer di solito rispondono entro 2 giorni
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 30/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Da chiarire
- Stato di attività
- Ferma
- Stack tecnologico
- python
- Ambito
- data-engineering
Direzione di ricerca
Inizia tracciando l’istanziazione del driver Hamilton e la gestione degli adapter che implementano l’hook do_node_execute. Il lavoro è completato quando è possibile fornire più adapter di questo tipo con un ordine di wrapping definito e il comportamento è coperto da test e documentato per gli utenti.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Is your feature request related to a problem? Please describe.
I am trying to use Hamilton caches with a custom implementation of the GracefulErrorAdapter already available in Hamilton. Since both caching and the error handling adapter use the do_node_execute hook, driver instantiation fails when specifying a cache and the error handling adapter.
Describe the solution you'd like
I would like to be able to instantiate a Hamilton driver with more than one adapter using the do_node_execute hook. The behavior of the driver would then depend on some precedence for the order in which the adapters wrap the node. This precedence would likely be determined by considerations within Hamilton, but should be fairly obvious to the user.
Describe alternatives you've considered
Subclassing the base Hamilton driver myself and implementing my custom error adapter within this subclass.
+Unblocks me right away-Docs indicate we should avoid making custom drivers.-Defeats the purpose of providing thewith_adapter(s)interfaces on the drivers. Adapters should be "chainable"/composable.
Not using caching
-Makes my pipelines slower on reruns, no benefit from code versioning etc.+No maintenance cost or complexity for metadata store or result store.
Not using an error handling adapter
-Any failure within one node in the pipeline means no results for any nodes. This is awful when we have nodes which will fail under certain conditions but should not cause the pipeline to fail.-Achieving "graceful" error handling otherwise (i.e., error checking in every node) increases code complexity with little gain.
Additional context
Less relevant to the feature request:
The primary change my GracefulErrorAdapter implementation makes from the Hamilton implementation is passing context about the error forward instead of just a sentinel value. I'm making a Hamilton pipeline that has lots of finnicky computer vision code within which is usually not worth writing complex error checking for. I wouldn't mind contributing this adapter to the project so that I can get more eyes on it.
- Lingua principale
- Jupyter Notebook
- Stelle
- 2.6k
- Fork
- 218
- Merge medio
- 2g 13h
- PR unite (30g)
- 18
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di apache/hamilton
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
I maintainer di solito rispondono entro 2 giorni
-
Engine crashes when client sends nan or inf float to UIForse di nuovo libera @Exudev l’ha presa 232 giorni fa e non c’è nessuna pull request aperta. Apertabug SDK
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
apache/hamilton#1458 · 5 commenti ·
I maintainer di solito rispondono entro 2 giorni
-
More detailed type annotations in graph visualizationsForse di nuovo libera @Nishant-Manocha l’ha presa 367 giorni fa e non c’è nessuna pull request aperta. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
apache/hamilton#1383 · 4 commenti ·
I maintainer di solito rispondono entro 2 giorni
-
Documentation: add explainer for FastAPIForse di nuovo libera @Aditya-Deshwal23 l’ha presa 186 giorni fa e non c’è nessuna pull request aperta. Apertadocumentation
Difficoltà 2/5 1-2 giorni Idoneità per principianti 65/100
apache/hamilton#1304 · 5 commenti ·
I maintainer di solito rispondono entro 2 giorni
-
Link back to github from reference API docsForse già presa @naikvaib l’ha presa 24 giorni fa. Apertadocumentation good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
apache/hamilton#572 · 3 commenti ·
I maintainer di solito rispondono entro 2 giorni
Tutte le issue di apache/hamilton
Issue simili
-
write_iceberg ignores the table's write.parquet.compression-codec propertyForse già presa @YuangGao l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
Eventual-Inc/Daft#7606 ·
I maintainer di solito rispondono entro 1 giorno
-
requires-triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
apache/datafusion-comet#7001 ·
I maintainer di solito rispondono entro 1 giorno
-
data good first issue hacktoberfest
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
pacocartones/free-llm-api-hub#326 ·
I maintainer di solito rispondono entro 1 giorno
-
data docs
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
pacocartones/market-data-sanity-checker#143 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
neuroinformatics-unit/movement#1155 ·
I maintainer di solito rispondono entro 3 giorni