Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
feat: common flusher for reporting with mtu handler #4823
feat: common flusher for reporting with mtu handler #4823
Changes from all commits
8674775
9dd764f
283c45c
b7573fa
cb0f33b
a1181eb
83fadce
cefbf6e
d8bc98b
542c3a6
6d9c9f4
fe8ceb2
a0f2398
da7052e
675258a
9f50ef5
ade2b1b
3805fa1
7497ede
8054bf2
fcf4ea8
374bc71
6b6a33e
44de774
da6cb62
7dfa68e
36a94e8
4a4076e
17782bd
340e775
d685ef8
63ce9d0
63a1fb1
0317126
aae4e37
9b80d27
f56437f
28e9be7
9dd9970
76eb527
02500eb
e31f0c3
7199623
cddc53d
aac3db0
624e6af
a60a753
42efbbf
1b49de6
4dc8da0
3a92b2d
cd0e1da
183ea44
dc13987
5d494c7
2ebb993
bedf901
e3425f7
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Check warning on line 12 in enterprise/reporting/flusher/aggregator/nop.go
Codecov / codecov/patch
enterprise/reporting/flusher/aggregator/nop.go#L11-L12
Check warning on line 44 in enterprise/reporting/flusher/aggregator/tracked_users_inapp.go
Codecov / codecov/patch
enterprise/reporting/flusher/aggregator/tracked_users_inapp.go#L44
Check warning on line 55 in enterprise/reporting/flusher/aggregator/tracked_users_inapp.go
Codecov / codecov/patch
enterprise/reporting/flusher/aggregator/tracked_users_inapp.go#L55
Check warning on line 59 in enterprise/reporting/flusher/aggregator/tracked_users_inapp.go
Codecov / codecov/patch
enterprise/reporting/flusher/aggregator/tracked_users_inapp.go#L59
Check warning on line 63 in enterprise/reporting/flusher/aggregator/tracked_users_inapp.go
Codecov / codecov/patch
enterprise/reporting/flusher/aggregator/tracked_users_inapp.go#L63
Check warning on line 67 in enterprise/reporting/flusher/aggregator/tracked_users_inapp.go
Codecov / codecov/patch
enterprise/reporting/flusher/aggregator/tracked_users_inapp.go#L67
Check warning on line 84 in enterprise/reporting/flusher/aggregator/tracked_users_inapp.go
Codecov / codecov/patch
enterprise/reporting/flusher/aggregator/tracked_users_inapp.go#L84
Check warning on line 89 in enterprise/reporting/flusher/aggregator/tracked_users_inapp.go
Codecov / codecov/patch
enterprise/reporting/flusher/aggregator/tracked_users_inapp.go#L89
Check warning on line 101 in enterprise/reporting/flusher/aggregator/tracked_users_inapp.go
Codecov / codecov/patch
enterprise/reporting/flusher/aggregator/tracked_users_inapp.go#L101
Check warning on line 105 in enterprise/reporting/flusher/aggregator/tracked_users_inapp.go
Codecov / codecov/patch
enterprise/reporting/flusher/aggregator/tracked_users_inapp.go#L105
Check warning on line 115 in enterprise/reporting/flusher/aggregator/tracked_users_inapp.go
Codecov / codecov/patch
enterprise/reporting/flusher/aggregator/tracked_users_inapp.go#L115
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is Alias needed? can't we directly use TrackedUsersReport?
Check warning on line 99 in enterprise/reporting/flusher/cron_runner.go
Codecov / codecov/patch
enterprise/reporting/flusher/cron_runner.go#L98-L99
Check warning on line 103 in enterprise/reporting/flusher/cron_runner.go
Codecov / codecov/patch
enterprise/reporting/flusher/cron_runner.go#L103
Check warning on line 111 in enterprise/reporting/flusher/cron_runner.go
Codecov / codecov/patch
enterprise/reporting/flusher/cron_runner.go#L111
Check warning on line 127 in enterprise/reporting/flusher/cron_runner.go
Codecov / codecov/patch
enterprise/reporting/flusher/cron_runner.go#L127