Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[#9666] Fixed realtime to maintain valid value #9978

Merged
merged 1 commit into from
May 22, 2023

Conversation

smilu97
Copy link
Member

@smilu97 smilu97 commented May 22, 2023

When updating flux of activeThreadCount, connection message overrides meaningful payload.

This patch restrict the connection message to be passed only one time per websocket request

@smilu97 smilu97 self-assigned this May 22, 2023
@codecov
Copy link

codecov bot commented May 22, 2023

Codecov Report

Merging #9978 (d28f0e0) into master (b1572be) will decrease coverage by 0.03%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master    #9978      +/-   ##
============================================
- Coverage     38.83%   38.80%   -0.03%     
- Complexity    11994    11995       +1     
============================================
  Files          3626     3626              
  Lines         95132    95142      +10     
  Branches      10521    10522       +1     
============================================
- Hits          36942    36924      -18     
- Misses        55029    55056      +27     
- Partials       3161     3162       +1     
Impacted Files Coverage Δ
...ead/count/dao/ActiveThreadCountFetcherFactory.java 0.00% <0.00%> (ø)
...hread/count/dao/ActiveThreadCountWebDaoConfig.java 0.00% <0.00%> (ø)
...time/activethread/count/dao/OptimisticFetcher.java 0.00% <0.00%> (ø)

... and 9 files with indirect coverage changes

@smilu97 smilu97 merged commit 97fc92a into pinpoint-apm:master May 22, 2023
@smilu97 smilu97 deleted the 9666_redis branch May 22, 2023 10:13
@emeroad emeroad added this to the 2.6.0 milestone May 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Replace fully connected connections between collector and web with Redis
2 participants