Skip to content
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.

Sign up for GitHub

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

Jump to bottom

ui: Reset SQL Stats button doesn't clear persisted stats. #70529

Closed
Azhng opened this issue Sep 21, 2021 · 0 comments
Closed

ui: Reset SQL Stats button doesn't clear persisted stats. #70529

Azhng opened this issue Sep 21, 2021 · 0 comments
Assignees
@Azhng
Labels
A-sql-observability Related to observability of the SQL layer C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. GA-blocker

Comments

@Azhng
Copy link
Contributor

Azhng commented Sep 21, 2021

Clear SQL Stats button in Statement/Transaction Page doesn't clear persisted statistics. To reproduce, run

SELECT 1; SELECT 1,1;
SET CLUSTER SETTING sql.stats.flush.interval = '1s';
RESET CLUSTER SETTING sql.stats.flush.interval;

Then click on the Clear SQL Stats button. It shows that the SQL Stats in the system table will not be cleared.

This does not affected CLI builtin.
@Azhng Azhng added C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. release-blocker Indicates a release-blocker. Use with branch-release-2x.x label to denote which branch is blocked. A-sql-observability Related to observability of the SQL layer T-sql-observability labels Sep 21, 2021
@Azhng Azhng self-assigned this Sep 21, 2021
@Azhng Azhng mentioned this issue Sep 22, 2021
Merged
Azhng added a commit to Azhng/cockroach that referenced this issue Sep 22, 2021
@Azhng
sql,server: enable tenant status server to work with persisted SQL stats
c2678d8 
Previsouly, tenant status server was not able to work with the new
persisted SQL Stats. This result in SQL Stats RPC not able to return
or reset persisted SQL stats.
This commit addresses this issue to update tenant status server's
implementation to be able to work with persisted SQL Stats.

Resolves cockroachdb#70585 cockroachdb#70529 cockroachdb#68177

Release Justification: Bug fixes and low-risk updates to new functionality

Release note (bug fix): Statement/Transaction Page now is able to
display and reset persisted SQL Stats.
Azhng added a commit to Azhng/cockroach that referenced this issue Sep 23, 2021
@Azhng
sql,server: enable tenant status server to work with persisted SQL stats
38098fa 
Previsouly, tenant status server was not able to work with the new
persisted SQL Stats. This result in SQL Stats RPC not able to return
or reset persisted SQL stats.
This commit addresses this issue to update tenant status server's
implementation to be able to work with persisted SQL Stats.

Resolves cockroachdb#70585 cockroachdb#70529 cockroachdb#68177

Release Justification: Bug fixes and low-risk updates to new functionality

Release note (bug fix): Statement/Transaction Page now is able to
display and reset persisted SQL Stats.
@maryliag maryliag removed the release-blocker Indicates a release-blocker. Use with branch-release-2x.x label to denote which branch is blocked. label Sep 23, 2021
Azhng added a commit to Azhng/cockroach that referenced this issue Sep 23, 2021
@Azhng
sql,server: enable tenant status server to work with persisted SQL stats
7d0e182 
Previsouly, tenant status server was not able to work with the new
persisted SQL Stats. This result in SQL Stats RPC not able to return
or reset persisted SQL stats.
This commit addresses this issue to update tenant status server's
implementation to be able to work with persisted SQL Stats.

Resolves cockroachdb#70585 cockroachdb#70529 cockroachdb#68177

Release Justification: Bug fixes and low-risk updates to new functionality

Release note (bug fix): Statement/Transaction Page now is able to
display and reset persisted SQL Stats.
Azhng added a commit to Azhng/cockroach that referenced this issue Sep 23, 2021
@Azhng
sql,server: enable tenant status server to work with persisted SQL stats
09993f4 
Previsouly, tenant status server was not able to work with the new
persisted SQL Stats. This result in SQL Stats RPC not able to return
or reset persisted SQL stats.
This commit addresses this issue to update tenant status server's
implementation to be able to work with persisted SQL Stats.

Resolves cockroachdb#70585 cockroachdb#70529 cockroachdb#68177

Release Justification: Bug fixes and low-risk updates to new functionality

Release note (bug fix): Statement/Transaction Page now is able to
display and reset persisted SQL Stats.
Azhng added a commit to Azhng/cockroach that referenced this issue Sep 24, 2021
@Azhng
sql,server: enable tenant status server to work with persisted SQL stats
4a220ab 
Previously, tenant status server was not able to work with the new
persisted SQL Stats. This result in SQL Stats RPC not able to return
or reset persisted SQL stats.
This commit addresses this issue by:
* updating tenant status server's Statements endpoint to be able to work
  with persisted SQL Stats.
* introducing additional parameter in ResetSQLStatsRequest to specify
  whether persisted SQL Stats need to be reset.

Resolves cockroachdb#70585 cockroachdb#70529 cockroachdb#68177

Release note (bug fix): Statement/Transaction Page now is able to
display and reset persisted SQL Stats.
Azhng added a commit to Azhng/cockroach that referenced this issue Sep 24, 2021
@Azhng
sql,server: enable tenant status server to work with persisted SQL stats
92e1967 
Previously, tenant status server was not able to work with the new
persisted SQL Stats. This result in SQL Stats RPC not able to return
or reset persisted SQL stats.
This commit addresses this issue by:
* updating tenant status server's Statements endpoint to be able to work
  with persisted SQL Stats.
* introducing additional parameter in ResetSQLStatsRequest to specify
  whether persisted SQL Stats need to be reset.

Resolves cockroachdb#70585 cockroachdb#70529 cockroachdb#68177

Release note (bug fix): Statement/Transaction Page now is able to
display and reset persisted SQL Stats.
craig bot pushed a commit that referenced this issue Sep 25, 2021
@Azhng @sumeerbhola
Merge #70586 #70707
9ba8499 
70586: sql,server: enable tenant status server to work with persisted SQL stats r=maryliag,dhartunian a=Azhng

Previsouly, tenant status server was not able to work with the new
persisted SQL Stats. This result in SQL Stats RPC not able to return
or reset persisted SQL stats.
This commit addresses this issue to update tenant status server's
implementation to be able to work with persisted SQL Stats.

Resolves #70585 #70529 #68177

Release Justification: Bug fixes and low-risk updates to new functionality

Release note (bug fix): Statement/Transaction Page now is able to
display and reset persisted SQL Stats.

70707: clusterversion,storage: cluster version that uses Pebble's SetWithDelete r=sumeerbhola a=sumeerbhola

This is a backwards incompatible change in Pebble to provide
SingleDelete semantics that are clean and robust to programming
error.

This is being done post v21.2-beta but before GA since this
was deemed a GA blocker and not a release blocker. It allows
for upgrading beta clusters to the GA version. For more on these
discussions see the following threads
cockroachdb/pebble#1255 (comment),
https://cockroachlabs.slack.com/archives/C4A9ALLRL/p1631213490022600
(Cockroach Labs internal link).

Informs cockroachdb/pebble#1255
Informs #69891

Release note: None

Release justification: high-severity bug fix

Co-authored-by: Azhng <archer.xn@gmail.com>
Co-authored-by: sumeerbhola <sumeer@cockroachlabs.com>
maryliag pushed a commit to maryliag/cockroach that referenced this issue Sep 27, 2021
@Azhng @maryliag
sql,server: enable tenant status server to work with persisted SQL stats
8bfdeb6 
Previously, tenant status server was not able to work with the new
persisted SQL Stats. This result in SQL Stats RPC not able to return
or reset persisted SQL stats.
This commit addresses this issue by:
* updating tenant status server's Statements endpoint to be able to work
  with persisted SQL Stats.
* introducing additional parameter in ResetSQLStatsRequest to specify
  whether persisted SQL Stats need to be reset.

Resolves cockroachdb#70585 cockroachdb#70529 cockroachdb#68177

Release note (bug fix): Statement/Transaction Page now is able to
display and reset persisted SQL Stats.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Azhng Azhng

Labels
A-sql-observability Related to observability of the SQL layer C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. GA-blocker
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@maryliag @Azhng