-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
roachpb,sql.ui: remove Opt field from StatementStatisticsKey #69597
Conversation
e79df89
to
96c5c29
Compare
Should we try to get this in for this release? Or we should wait for after the branch cut? |
wait for after the branch cut |
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.
Looks good to me! I love the smell of deleted code in the morning.
Reviewed 16 of 16 files at r1, all commit messages.
Reviewable status:complete! 0 of 0 LGTMs obtained (waiting on @Azhng)
-- commits, line 2 at r1:
nit: It might be worth explicitly saying here why this field is safe to delete?
f6a96ad
to
8ca1f86
Compare
This field is safe to delete because we have been filling this field with hard coded values since few releases ago. Resolves cockroachdb#68077 Release note: None Release justification:
8ca1f86
to
08b663b
Compare
TFTR! bors r=matthewtodd |
Build failed: |
Hmm bors flaked, try again bors r=matthewtodd |
Build succeeded: |
69597: roachpb,sql.ui: remove Opt field from StatementStatisticsKey r=matthewtodd a=Azhng Resolves cockroachdb#68077 Release note: None 70364: docs: update readme for cluster-ui r=xinhaoz a=Azhng Previsouly, README in cluster-ui points to an incorrect directory to run yarn command. This commit update the directory mentioned in the README. Release note: None Co-authored-by: Azhng <archer.xn@gmail.com>
Resolves #68077
Release note: None