Skip to content
This repository has been archived by the owner on Dec 6, 2024. It is now read-only.

Commit

Permalink
Make Unary gRPC the only recommended mode
Browse files Browse the repository at this point in the history
Experiments have shown that Streaming implementation
results in significantly more complex code. The performance
benefits of Streaming mode were only visible in very
small batches and are not applicable to real life scenarios
when under high load we aim to have large batches.

In addition, one of the potential benefits that we could
get with Streaming - the ability to optimize and avoid
repetition of Resource does not seem to have much real
life usage.

As a result Streaming mode was removed from the RFC and
Unary mode is the only mode that is now part of the spec.
  • Loading branch information
Tigran Najaryan committed Sep 25, 2019
1 parent 715a046 commit 9fe913b
Show file tree
Hide file tree
Showing 9 changed files with 425 additions and 515 deletions.
515 changes: 0 additions & 515 deletions text/0000-opentelemetry-protocol.md

This file was deleted.

425 changes: 425 additions & 0 deletions text/0035-opentelemetry-protocol.md

Large diffs are not rendered by default.

Binary file added text/images/otlp-concurrent.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file removed text/images/otlp-load-balancing-timeline.png
Binary file not shown.
Binary file removed text/images/otlp-load-balancing.png
Binary file not shown.
Binary file modified text/images/otlp-multi-destination.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added text/images/otlp-sequential.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file removed text/images/otlp-streaming.png
Binary file not shown.
Binary file removed text/images/otlp-unary.png
Binary file not shown.

0 comments on commit 9fe913b

Please sign in to comment.