-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
stats: integrate real symbol table into stats system #4980
Merged
mattklein123
merged 186 commits into
envoyproxy:master
from
jmarantz:integration-symtab
Jun 28, 2020
Merged
Changes from 177 commits
Commits
Show all changes
186 commits
Select commit
Hold shift + click to select a range
ed31d09
checkpoint
jmarantz 1cd7a7b
got heap_stat_data_test working.
jmarantz d62f533
all tests working; but still taking locks in hot-path.
jmarantz b3cad8a
format
jmarantz 63138e8
Merge branch 'master' into integration-symtab
jmarantz 524cf02
format
jmarantz bb84b90
fix botched merge.
jmarantz f9c0441
perf test and tweaks
jmarantz 7c5ac2f
formatting, comments, and tests.
jmarantz 5e1c713
Merge branch 'master' into integration-symtab
jmarantz bdbf340
use std::make_unique for clang-tidy
jmarantz d70cb18
Make a class for Http::CodeUtility::chargeResponseTiming et al, plumb…
jmarantz 9c49dec
Use join() rather than fmt::format, and hold static strings in string…
jmarantz df7d7a1
fix router test (empty prefix) and use const string_view for member v…
jmarantz da648b3
add speed-test.
jmarantz ec66aaf
Remove libraries not needed by speed test.
jmarantz 461471a
Merge branch 'master' into integration-symtab
jmarantz 32ff476
Merge branch 'http-code-stats-as-object' into integration-symtab
jmarantz b711b1b
checkpoint
jmarantz 7723309
checkpoint
jmarantz a892f70
partially symbolize http-response-code stats.
jmarantz 5fadb96
Virtualize SymbolTable (but not symbols or StatName) for use in mocks…
jmarantz 3dacb8e
Merge branch 'master' into integration-symtab
jmarantz aaca920
Pre-allocate symbols for 200s and 404s, alloc file-system stats per f…
jmarantz e25f7cd
Finishing plumbing up Filesystem::Instance as an object via API and g…
jmarantz ec78dca
Merge branch 'master' into integration-symtab
jmarantz ae0d345
format
jmarantz 1727daa
Merge branch 'master' into integration-symtab
jmarantz 739bf06
revert API change - make an IsolatedStoreImpl in the RawConnectionDri…
jmarantz 8336650
force the SymbolTable backing MockIsolatedStore to be a singleton.
jmarantz c17cca9
format
jmarantz 04cbc9c
comment out singleton symbol-table usage in MockIsolatedStatsStore.
jmarantz 101e913
use globals
jmarantz 09d1b6e
merge in globals
jmarantz cee54fe
move file placed in wrong dir
jmarantz 6883d6d
add missing file.
jmarantz c353a34
Merge branch 'master' into integration-symtab
jmarantz 1601f6f
Merge branch 'master' into integration-symtab
jmarantz ac03dd9
All tests working again.
jmarantz 0a2fbe8
format
jmarantz 2c92ce7
Merge branch 'master' into integration-symtab
jmarantz 4bb7c0d
format
jmarantz b7acdd5
prune dead code.
jmarantz 6b4cbb8
Merge branch 'master' into integration-symtab
jmarantz 0cf3f92
Merge branch 'master' into integration-symtab
jmarantz 0eb83f6
Merge branch 'master' into integration-symtab
jmarantz d087924
Merge branch 'master' into integration-symtab
jmarantz af1d8d0
Merge branch 'master' into integration-symtab
jmarantz 4014dde
Remove the symbol-table interface; just call symbol table directly.
jmarantz 346e9e5
move toString to SymbolTable.
jmarantz 1343181
Merge branch 'master' into integration-symtab
jmarantz 889f819
format
jmarantz cc87f92
fix tsan error (from a stale version of a test due to a botched git m…
jmarantz f81f794
Re-order member variables in the hope of improving test flakes due to…
jmarantz 42bd6b8
Merge branch 'master' into integration-symtab
jmarantz c375cf6
Merge branch 'master' into integration-symtab
jmarantz 5fe26f5
Move the aggregated stat-name storage mechanism out of MetricImpl and…
jmarantz 3bdbdc2
Merge branch 'master' into integration-symtab
jmarantz 0b39814
Got compile to work.
jmarantz 665897f
Get tests to work.
jmarantz 7c9471e
Merge branch 'master' into integration-symtab
jmarantz ef23a21
Merge branch 'master' into integration-symtab
jmarantz 40046c7
Merge branch 'master' into integration-symtab
jmarantz 1028dbe
format
jmarantz 45ef917
all tests passing.
jmarantz fd8db0e
Use FakeSymbolTable in prod instances.
jmarantz c6ec598
format
jmarantz 88f1f4d
Use FakeSymbolTableImpl in most tests too (except the SymbolTable uni…
jmarantz 94c5205
Don't pass SymbolTable in to stat Mocks but use the global; factor ou…
jmarantz b8cbed3
Remove comments and dead code.
jmarantz d53d7f6
Merge branch 'master' into integration-symtab
jmarantz 4a0d37d
Go back to using real symbol tables in this client, and fix a content…
jmarantz b2a1c9f
Merge branch 'master' into integration-symtab (hystrix_test fails)
jmarantz 8bd401b
format
jmarantz 494f366
All tests working.
jmarantz aa60638
spelling
jmarantz 8626e51
Merge branch 'master' into integration-symtab
jmarantz 9ffa66f
Merge branch 'master' into integration-symtab
jmarantz b312ecd
Backed out a few changes that are not needed.
jmarantz a0f0c9e
simplify the router changes.
jmarantz 156434e
Clean up router changes slightly.
jmarantz aa6ecff
checkpoint merge; does not compile.
jmarantz 54d9307
Merge branch 'master' into integration-symtab
jmarantz 9e13376
everything compiles, all tests pass except StatsThreadLocalStoreTest.…
jmarantz 59a36d7
Merge branch 'master' into integration-symtab
jmarantz 33f36f3
format
jmarantz 64d94fa
Fix tests and resync with #6161 and #6504
jmarantz b1a7db0
Merge branch 'master' into integration-symtab
jmarantz a57a859
Merge branch 'master' into integration-symtab
jmarantz 5ad3312
Merge branch 'master' into integration-symtab
jmarantz 4f48459
Merge branch 'master' into integration-symtab
jmarantz 7082921
Merge branch 'master' into integration-symtab
jmarantz 4c89a6f
fix functional issues from merge.
jmarantz 788efb6
Merge branch 'master' into integration-symtab
jmarantz 27b9c06
remove some detritus
jmarantz d27394c
Merge branch 'master' into integration-symtab
jmarantz 214d3a0
Merge branch 'master' into integration-symtab
jmarantz 60b82cf
get build to work again.
jmarantz 9d2d673
fix merge conflicts.
jmarantz 9e3986c
Merge branch 'master' into integration-symtab
jmarantz 49b8078
Remove stale test file.
jmarantz 3206f30
Merge branch 'master' into integration-symtab
jmarantz aa15d73
Merge branch 'master' into integration-symtab
jmarantz 47a3219
Merge branch 'master' into integration-symtab
jmarantz 81b9779
Merge branch 'master' into integration-symtab
jmarantz f242443
Merge branch 'master' into integration-symtab
jmarantz afbb94a
clean up commented-out code.
jmarantz dfbe16b
Merge branch 'master' into integration-symtab
jmarantz 76ac2aa
Merge branch 'master' into integration-symtab
jmarantz 95c82b2
Skip over empty StatNames.
jmarantz 64f8099
Remove superfluous changes.
jmarantz 801b1b6
Remove superfluous comments.
jmarantz 843a35a
Merge branch 'master' into integration-symtab
jmarantz 86aa06c
Merge branch 'master' into integration-symtab
jmarantz 647fd6d
Merge branch 'master' into integration-symtab
jmarantz ce7417d
get CI mostly working; I think there's an unrelated problem in hotres…
jmarantz 713d97d
format
jmarantz 5980a10
updated memory usage
jmarantz 199ef47
fix comment
jmarantz 137315e
Merge branch 'master' into integration-symtab
jmarantz 8028807
Merge branch 'master' into integration-symtab
jmarantz fe153be
Merge branch 'master' into integration-symtab
jmarantz 15034e9
Merge branch 'master' into integration-symtab
jmarantz e0c4711
Merge branch 'master' into integration-symtab
jmarantz c831e37
cleanup tests.
jmarantz 3cb2b91
fix expected value.
jmarantz 3e931f6
Merge branch 'master' into integration-symtab
jmarantz 1e69649
Merge branch 'master' into integration-symtab
jmarantz bd95076
Merge branch 'master' into integration-symtab
jmarantz 1016ae6
shared the main symbol-table with the isolated stats used for cluster…
jmarantz e45e905
Update memory usage to account for isolated stats sharing symbol tables.
jmarantz e1787cb
pedantic spelling
jmarantz 600977e
update PR #
jmarantz 5f48598
Merge branch 'shared-cluster-isolated-stats' into integration-symtab
jmarantz 44bd5a5
format
jmarantz 9b075bb
Merge branch 'master' into integration-symtab
jmarantz d4744bd
Add option to switch between fake and real symbol-tables on the comma…
jmarantz 313d591
Add command-line option to switch between real/fake symbol tables, an…
jmarantz bee7b67
Merge branch 'master' into symbol-table-option
jmarantz 56ab602
use configured symbol-table impl in integration tests.
jmarantz a0e1d7c
clang-tidy error
jmarantz 92442ef
Cover both real & fake symbol tables in symbol-table memory test.
jmarantz 96f4e85
cleanups
jmarantz f9c4f82
Merge branch 'master' into symbol-table-option
jmarantz 2a2f1a0
Merge branch 'master' into integration-symtab
jmarantz 53ba649
Merge branch 'symbol-table-option' into integration-symtab
jmarantz 08f444e
convert more tests to use the symbol-table creator rather than hard-c…
jmarantz ef57468
Merge branch 'symbol-table-option' into integration-symtab
jmarantz 1c08f0a
format
jmarantz 25367f6
use real symbol tables by default.
jmarantz 6551dee
format
jmarantz 8a6521c
swap default
jmarantz 347cb3c
Merge branch 'master' into integration-symtab
jmarantz a17e17c
fix test
jmarantz bf6fcd1
back out some earlier attempts at patching in the options PR.
jmarantz dc690b2
update byte-size golds.
jmarantz 71e0fcd
back out the sharing of symbol tables for upstream cluster isolated s…
jmarantz f5d1dbb
Merge branch 'master' into integration-symtab
jmarantz b9a9ccc
Merge branch 'master' into integration-symtab
jmarantz b9b26d4
Merge branch 'master' into integration-symtab
jmarantz bf022f1
Merge branch 'master' into integration-symtab
jmarantz 8be749b
Merge branch 'master' into integration-symtab
jmarantz b855889
Merge branch 'master' into integration-symtab
jmarantz 91f0f92
Merge branch 'master' into integration-symtab
jmarantz 5ff494a
remove superfluous check.
jmarantz eb79c95
Merge branch 'master' into integration-symtab
jmarantz b8dfdc0
switch another default to use real symbol tables.
jmarantz 141e711
Merge branch 'master' into integration-symtab
jmarantz f834e4e
move version history where it belongs.
jmarantz 52eca10
Merge branch 'master' into integration-symtab
jmarantz 7c3ce33
Merge branch 'master' into integration-symtab
jmarantz febd650
format
jmarantz 9c79dc2
Merge branch 'master' into integration-symtab
jmarantz 16077ab
Instantiate a TestSymboLTable so that mocks reference the same one.
jmarantz dcabf96
reformat with clang-10.
jmarantz c0e45c6
improve assert message for dynamic/symbolic mismatch, and don't check…
jmarantz f954e35
clean up the config_test so it has minimal documnted changes from the…
jmarantz c3bf36d
Merge branch 'master' into integration-symtab
jmarantz 3193beb
Merge branch 'master' into integration-symtab
jmarantz 55e2060
Merge branch 'master' into integration-symtab
jmarantz 4d3043e
Merge branch 'master' into integration-symtab
jmarantz 5695a17
try to avoid race by waiting for a timeout stat rather than just stro…
jmarantz 9d33cce
Declare ads_integregion_test as enormous to try to avoid tsan timeouts.
jmarantz 96e2c6c
remove superfluous line, that probably was from a merge.
jmarantz 141c716
back out accidental commit
jmarantz fabc533
Merge branch 'master' into integration-symtab
jmarantz File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
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.
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.
Merge issue?