Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(cli): raise python error if pipeline error #3

Conversation

aballiet
Copy link
Collaborator

@aballiet aballiet commented Dec 20, 2024

CLI raises exit code 1 if pipeline error

Copy link

linear bot commented Dec 20, 2024

@aballiet aballiet self-assigned this Dec 20, 2024
@aballiet aballiet merged commit 08c5e9a into gorgias Dec 20, 2024
1 check passed
@aballiet aballiet deleted the antoineballiet/daae-4584-raise-python-erorr-from-main-thread-if-pipeline-error branch December 20, 2024 14:59
anaselmhamdi added a commit that referenced this pull request Dec 20, 2024
* feat: implement failure handling for pipeline (#1)

* wip

* wip

* Update gorgias.yml

* Update gorgias.yml

* wip

* feat: add destination unnesting for file

* feat: assert schema is provided if unnest set to True

* add test for config parsing unnest

* Update process.py

* chore: test shutdown wait true pending

* chore(fix): cast bizon date columns as string in bq_streaming

* feat: implement stop events, runner status, refacto time partitioning

* fix test

* delete gha

---------

Co-authored-by: Anas El Mhamdi <anas.elmhamdi@gmail.com>

* feat(cli): raise python error if pipeline error (#3)

* fix: multiprocessing event type

* feat: cli return exist code 1 if pipeline error

* cli raises clickException

* always run pytest on PR

---------

Co-authored-by: Anas El Mhamdi <anas.elmhamdi@gmail.com>
aballiet added a commit that referenced this pull request Dec 20, 2024
* feat(cli): raise python error if pipeline error (#3)

* fix: multiprocessing event type

* feat: cli return exist code 1 if pipeline error
---------

Co-authored-by: Anas El Mhamdi <anas.elmhamdi@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant