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

kvserver: remove the old constraint conformance code that's not aware of span configs and secondary tenants #100180

Open
knz opened this issue Mar 30, 2023 · 0 comments
Open

kvserver: remove the old constraint conformance code that's not aware of span configs and secondary tenants #100180

knz opened this issue Mar 30, 2023 · 0 comments
Labels
A-kv-server Relating to the KV-level RPC server C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. T-kv KV Team

Comments

@knz
Copy link
Contributor

knz commented Mar 30, 2023 •
edited
Loading

The code in kvserver/reports/reporter.go is not aware of secondary tenants and span configs. We should not use it any more and it should be removed.

Jira issue: CRDB-26286
Epic: CRDB-26686
@knz knz added C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. A-kv-server Relating to the KV-level RPC server labels Mar 30, 2023
@blathers-crl blathers-crl bot added the T-kv KV Team label Mar 30, 2023
@knz knz mentioned this issue Mar 30, 2023
Open
@exalate-issue-sync exalate-issue-sync bot removed the T-kv KV Team label Mar 30, 2023
@exalate-issue-sync exalate-issue-sync bot added the T-kv KV Team label Apr 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-kv-server Relating to the KV-level RPC server C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. T-kv KV Team
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@knz