Skip to content

deps: enable async otlp exporting#278

Closed
santigimeno wants to merge 1 commit intonode-v22.x-nsolid-v5.xfrom
santi/grpc_async
Closed

deps: enable async otlp exporting#278
santigimeno wants to merge 1 commit intonode-v22.x-nsolid-v5.xfrom
santi/grpc_async

Conversation

@santigimeno
Copy link
Copy Markdown
Member

Stop using synchronous exporting as it may block for a substantial period of time on network disconnections.

@santigimeno santigimeno self-assigned this Mar 13, 2025
Copy link
Copy Markdown
Contributor

@juanarbol juanarbol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

Stop using synchronous exporting as it may block for a substantial
period of time on network disconnections.
santigimeno added a commit that referenced this pull request Mar 24, 2025
Stop using synchronous exporting as it may block for a substantial
period of time on network disconnections.

PR-URL: #278
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
@santigimeno
Copy link
Copy Markdown
Member Author

Landed in 3ddc4ed

@santigimeno santigimeno deleted the santi/grpc_async branch March 24, 2025 09:26
santigimeno added a commit that referenced this pull request May 1, 2025
Stop using synchronous exporting as it may block for a substantial
period of time on network disconnections.

PR-URL: #278
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
santigimeno added a commit that referenced this pull request May 2, 2025
Stop using synchronous exporting as it may block for a substantial
period of time on network disconnections.

PR-URL: #278
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
santigimeno added a commit that referenced this pull request May 7, 2025
Stop using synchronous exporting as it may block for a substantial
period of time on network disconnections.

PR-URL: #278
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
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.

2 participants