Skip to content

Conversation

@Gsantomaggio
Copy link
Member

reduce the scope of the mutex. Optimize producer/consumer closing in case of
remove stream.

reduce the scope of the mutex. Optimize producer/consumer closing in case of
remove stream.
@codecov-commenter
Copy link

codecov-commenter commented Jun 11, 2021

Codecov Report

Merging #45 (6f4cb64) into main (72ed104) will increase coverage by 0.03%.
The diff coverage is 91.11%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #45      +/-   ##
==========================================
+ Coverage   74.83%   74.86%   +0.03%     
==========================================
  Files          15       15              
  Lines        1518     1520       +2     
==========================================
+ Hits         1136     1138       +2     
  Misses        305      305              
  Partials       77       77              
Impacted Files Coverage Δ
pkg/stream/client.go 76.85% <ø> (ø)
pkg/stream/enviroment_debug.go 0.00% <ø> (ø)
pkg/stream/producer.go 63.12% <ø> (ø)
pkg/stream/enviroment.go 66.12% <90.24%> (-0.11%) ⬇️
pkg/stream/coordinator.go 93.57% <100.00%> (+0.14%) ⬆️
pkg/stream/server_frame.go 74.34% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 72ed104...6f4cb64. Read the comment docs.

@Gsantomaggio Gsantomaggio merged commit f5e8f51 into main Jun 14, 2021
@Gsantomaggio Gsantomaggio deleted the refactor_coordinator branch June 14, 2021 08:18
Zerpet pushed a commit that referenced this pull request Jul 17, 2023
* Implement SubEntryBatch publish

closes: Gsantomaggio/rabbitmq-stream-go-client#44
Signed-off-by: Gabriele Santomaggio <G.santomaggio@gmail.com>
Zerpet pushed a commit that referenced this pull request Jul 17, 2023
* Implement SubEntryBatch publish

closes: Gsantomaggio/rabbitmq-stream-go-client#44
Signed-off-by: Gabriele Santomaggio <G.santomaggio@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.

3 participants