Skip to content

Commit

Permalink
Fix typo in FailedNotRetryable (#243)
Browse files Browse the repository at this point in the history
  • Loading branch information
tigrannajaryan authored and bogdandrutu committed Sep 7, 2019
1 parent 9896098 commit 68007e4
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions specification/sdk-exporter.md
Original file line number Diff line number Diff line change
Expand Up @@ -32,15 +32,15 @@ ExportResult is one of:

- Success - batch is successfully exported. For protocol exporters this typically means that the data is sent over the wire and delivered to the destination server.

- FailedNoneRetryable - exporting failed. The caller must not retry exporting the same batch. The batch must be dropped. This for example can happen when the batch contains bad data and cannot be serialized.
- FailedNotRetryable - exporting failed. The caller must not retry exporting the same batch. The batch must be dropped. This for example can happen when the batch contains bad data and cannot be serialized.

- FailedRetryable - cannot export to the destination. The caller should record the error and may retry exporting the same batch after some time. This for example can happen when the destination is unavailable, there is a network error or endpoint does not exist.

### Shutdown()

Shuts down the exporter. Called when SDK is shut down. This is an opportunity for exporter to do any cleanup required.

`Shutdown` should be called only once for each `Exporter` instance. After the call to `Shutdown` subsequent calls to `Export` are not allowed and should return FailedNoneRetryable error.
`Shutdown` should be called only once for each `Exporter` instance. After the call to `Shutdown` subsequent calls to `Export` are not allowed and should return FailedNotRetryable error.

`Shutdown` should not block indefinitely (e.g. if it attempts to flush the data and the destination is unavailable). Language library authors can decide if they want to make the shutdown timeout to be configurable.

Expand Down Expand Up @@ -71,7 +71,7 @@ type ExportResultCode int

const (
Success ExportResultCode = iota
FailedNoneRetryable
FailedNotRetryable
FailedRetryable
)
```
Expand All @@ -81,7 +81,7 @@ const (
```java
public interface SpanExporter {
public enum ResultCode {
Success, FailedNoneRetryable, FailedRetryable
Success, FailedNotRetryable, FailedRetryable
}

ResultCode export(Collection<ExportableSpan> batch);
Expand Down

0 comments on commit 68007e4

Please sign in to comment.