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
infoschema, util/stmtsummary: enhance statements_summary. #25031
infoschema, util/stmtsummary: enhance statements_summary. #25031
Changes from 118 commits
1a96cbb
0c8920a
25ae806
529e346
38fa019
d78ca46
1cddd8f
33d865f
2d75e37
05703d8
eaf1caa
0163613
0935f75
c0227b6
0a168fe
b447b86
8e8792b
2a8ed11
12b6a87
373d344
0f24624
e2a18b1
6b7da7e
ebad974
3fcd196
073ff06
32e8df9
1092e72
7e8a8d8
252b43a
015b145
e061027
d3bf818
f59155b
efcad35
80167a3
dbc94c7
756c53e
43a4adc
be4b4e4
c6ce248
a8be817
fdec81a
01a92a4
9c503e5
b2278b2
d3ea973
a3ed839
9b243a5
eaffa1c
413ed0d
c91e5e3
0bd1f5e
f282833
16c83f9
cca919b
52cb27b
a31059d
4c9e65f
b7a87e1
5725827
b0776f5
b50b213
3b9fa88
cafa16a
f880324
f740b34
6c69920
d8b32ad
4629224
184b359
39655fc
51a56b1
4d633c4
528db85
9f2783f
7500c7c
af77055
3b13b59
6af7eed
2935d60
1b10912
3bfd7e1
53423a3
322d938
0802bd5
f516fd3
03e7d21
a0601b6
5101038
3952c98
e9779fa
d0b4fde
27ee905
2faf184
2ea3d3f
fc01d92
b5a3aac
f71f3ab
49985cd
0f920b2
c19b20a
fbb9f79
b95f833
4de43c7
6544e56
9eccd82
4936831
b823bb7
7044744
4aefe6f
9889df9
46620d1
ab0cbd8
5824872
39b1f06
337161b
39daceb
8f0b757
ce03218
de53695
32787d4
e7b8aff
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
Will
addWith
be modified at the same time?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.
this struct is huge, so I passed its address to this function.
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.
No, my point is locking. Will it cause data race?
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.
addWith
has been locked outside the function, instmtSummaryByDigestEvicted.AddEvicted()
at line 90 in util/evicted.go, so no lock needed here.