Skip to content

Conversation

@Gsantomaggio
Copy link
Member

@Gsantomaggio Gsantomaggio commented Jun 29, 2021

add smart publish to aggregate the messages.

  • add batch size parameter on publish side
  • add queue size parameter on publish side

closes #52

pair with @MarcialRosales

aggredate the messages
work on ha producer
@codecov-commenter
Copy link

codecov-commenter commented Jul 4, 2021

Codecov Report

Merging #53 (f3615cc) into main (3356a45) will increase coverage by 0.50%.
The diff coverage is 73.75%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #53      +/-   ##
==========================================
+ Coverage   74.31%   74.81%   +0.50%     
==========================================
  Files          15       15              
  Lines        1600     1644      +44     
==========================================
+ Hits         1189     1230      +41     
- Misses        328      329       +1     
- Partials       83       85       +2     
Impacted Files Coverage Δ
pkg/stream/constants.go 77.55% <ø> (-4.09%) ⬇️
pkg/stream/enviroment.go 62.82% <ø> (-0.48%) ⬇️
pkg/stream/server_frame.go 66.53% <18.75%> (-7.28%) ⬇️
pkg/stream/producer.go 79.19% <77.31%> (+13.91%) ⬆️
pkg/stream/client.go 77.20% <87.50%> (+0.79%) ⬆️
pkg/stream/coordinator.go 95.13% <100.00%> (+1.56%) ⬆️
pkg/stream/socket.go 78.57% <0.00%> (+7.14%) ⬆️

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 3356a45...f3615cc. Read the comment docs.

work on ha producer
work on ha producer
work on ha producer
@Gsantomaggio Gsantomaggio changed the title WIP add smart publish to add smart publish to Jul 5, 2021
work on ha producer.

closes #52
@Gsantomaggio Gsantomaggio merged commit 033ffd8 into main Jul 5, 2021
@Gsantomaggio Gsantomaggio deleted the publish branch July 19, 2021 05:40
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.

Smart publish

3 participants