Skip to content

sql, server: add SQLStatsResetter to distsql and internalExecutor#62617

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
Azhng:plumb-sql-stats-reset
Mar 26, 2021
Merged

sql, server: add SQLStatsResetter to distsql and internalExecutor#62617
craig[bot] merged 1 commit intocockroachdb:masterfrom
Azhng:plumb-sql-stats-reset

Conversation

@Azhng
Copy link
Copy Markdown
Contributor

@Azhng Azhng commented Mar 25, 2021

Closes #62587.

Release note: None

@Azhng Azhng requested review from a team and yuzefovich March 25, 2021 19:18
@Azhng Azhng requested a review from a team as a code owner March 25, 2021 19:18
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@Azhng Azhng requested a review from asubiotto March 25, 2021 19:37
Copy link
Copy Markdown
Member

@yuzefovich yuzefovich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM but probably worth waiting for Alfonso's feedback too.

Reviewed 5 of 5 files at r1.
Reviewable status: :shipit: complete! 0 of 0 LGTMs obtained (waiting on @asubiotto and @Azhng)


pkg/sql/execinfra/server_config.go, line 156 at r1 (raw file):

	HydratedTables *hydratedtables.Cache

	// SQLStatsResetter is an interface used to reset SQL stats without introduce

nit: "without introduction" or "without the need to introduce".

@Azhng Azhng force-pushed the plumb-sql-stats-reset branch 3 times, most recently from a5a4269 to c16570b Compare March 25, 2021 22:22
@Azhng
Copy link
Copy Markdown
Contributor Author

Azhng commented Mar 25, 2021

Rebased to pick up fix on TestScanBatch

@Azhng Azhng force-pushed the plumb-sql-stats-reset branch from c16570b to d43090e Compare March 26, 2021 00:25
Copy link
Copy Markdown
Contributor

@asubiotto asubiotto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

Reviewed 3 of 3 files at r2.
Reviewable status: :shipit: complete! 1 of 0 LGTMs obtained (waiting on @Azhng)

@Azhng
Copy link
Copy Markdown
Contributor Author

Azhng commented Mar 26, 2021

TFTR!

bors r=asubiotto,yuzefovich

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Mar 26, 2021

Build succeeded:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

roachtest: sqlsmith: crdb_internal.reset_sql_stats(): sql stats resetter not set

4 participants