-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Update the pinot tenants tables api to support returning broker tagged tables #11184
Update the pinot tenants tables api to support returning broker tagged tables #11184
Conversation
Codecov Report
@@ Coverage Diff @@
## master #11184 +/- ##
==========================================
- Coverage 0.11% 0.11% -0.01%
==========================================
Files 2229 2229
Lines 119803 119827 +24
Branches 18126 18130 +4
==========================================
Hits 137 137
- Misses 119646 119670 +24
Partials 20 20
Flags with carried forward coverage won't be shown. Click here to find out more.
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
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.
LGTM with minor comments. Please rebase on the latest master since we added the new auth annotation recently
This is authored by @priyen and is a small
feature
addition.This updates the
tenants/{tenantName}/tables
API to support also specifying type to return tagged tables by the broker tenant. Tables can have separate broker and server tenants, so this allows us to actually get both.Previously:
/tenants/{tenantName}/tables
-> returns tables with that server tagNow:
/tenants/{tenantName}/tables
-> returns tables with that server tag/tenants/{tenantName}/tables?type=server
-> returns tables with that server tag/tenants/{tenantName}/tables?type=broker
-> returns tables with that broker tagWe are already using this successfully in our fork, and unit tests are included