-
Notifications
You must be signed in to change notification settings - Fork 141
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
PMM-12468 Mongo plan summary and full scan. #3496
Changes from all commits
7aad647
01eedf7
e4ec526
2ca8f05
aa791de
de6f04b
3d02c7f
c254270
f8c9d06
afdf1e9
1519713
1f3d4f8
c70d09f
1b6021c
096ea6b
f543aba
2cf2c97
fab2f5a
13c3bde
44c0221
bc335d1
1c3a4eb
e62b224
cc6d96e
5a62285
a0c1430
9e7f00b
f0b7b0c
17a1f77
5e011b4
b597ed3
a111fdb
367adb7
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -38,7 +38,7 @@ func TestAggregator(t *testing.T) { | |
t.Run("Add", func(t *testing.T) { | ||
t.Run("error if aggregator is not running", func(t *testing.T) { | ||
a := New(time.Now(), "test-agent", logrus.WithField("component", "test"), truncate.GetMongoDBDefaultMaxQueryLength()) | ||
err := a.Add(nil, proto.SystemProfile{}) | ||
err := a.Add(context.TODO(), proto.SystemProfile{}) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Context should not be nil. |
||
assert.EqualError(t, err, "aggregator is not running") | ||
}) | ||
}) | ||
|
Large diffs are not rendered by default.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Large diffs are not rendered by default.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,2 @@ | ||
ALTER TABLE metrics | ||
DROP COLUMN `plan_summary`; |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,2 @@ | ||
ALTER TABLE metrics | ||
ADD COLUMN `plan_summary` LowCardinality(String); |
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.
Unused there.