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

Add error reporting to integrations POC #396

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

Conversation

mathialo
Copy link
Collaborator

Add an error method to the Extractor class that starts tracking of an error. This can be used in one of two ways, either by manually starting and stopping the error state:

e = extractor.error(...)

# handle error

e.finish()

or by using it as a context

with extractor.error(...):
    # Handle error

You can create an instant error (with no duration) by using the instant() method:

extractor.error(...).instant()

Tracking of start/end times, generating and keeping track of external IDs, reporting in checkins, and so on is all handled automatically.

Add an `error` method to the `Extractor` class that starts tracking of
an error. This can be used in one of two ways, either by manually
starting and stopping the error state:

``` python
e = extractor.error(...)

 # handle error

e.finish()
```

or by using it as a context

``` python
with extractor.error(...):
    # Handle error
```

You can create an instant error (with no duration) by using the
`instant()` method:

``` python
extractor.error(...).instant()
```

Tracking of start/end times, generating and keeping track of external
IDs, reporting in checkins, and so on is all handled automatically.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant