-
Notifications
You must be signed in to change notification settings - Fork 994
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
PHOENIX-5456 IndexScrutinyTool slow for indexes on multitenant tables… #575
Conversation
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.
The change lgtm.
Btw, is there a way for us to run a scrutiny on multitenant view index, for all tenants?
I'm wondering if we one day get PHOENIX-4657, does this change make such a scrutiny not possible?
PTable pDataTable = pconn.getTable(new PTableKey(null, multiTenantTable)); | ||
PTable pIndexTable = pconn.getTable(new PTableKey(null, indexNameGlobal)); | ||
|
||
//IndexColumnNames columnNames = new IndexColumnNames(pDataTable, pIndexTable); |
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.
delete comment
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.
done
@Parameterized.Parameters public static Collection<Object[]> data() { | ||
return Arrays.asList(new Object[][] { { "CREATE TABLE %s (ID INTEGER NOT NULL PRIMARY KEY, NAME VARCHAR, ZIP INTEGER, EMPLOY_DATE TIMESTAMP, EMPLOYER VARCHAR)", | ||
"CREATE LOCAL INDEX %s ON %s (NAME, EMPLOY_DATE) INCLUDE (ZIP)" }, { "CREATE TABLE %s (ID INTEGER NOT NULL PRIMARY KEY, NAME VARCHAR, ZIP INTEGER, EMPLOY_DATE TIMESTAMP, EMPLOYER VARCHAR) SALT_BUCKETS=2", | ||
"CREATE INDEX %s ON %s (NAME, EMPLOY_DATE) INCLUDE (ZIP)" }, { "CREATE TABLE %s (ID INTEGER NOT NULL PRIMARY KEY, NAME VARCHAR, ZIP INTEGER, EMPLOY_DATE TIMESTAMP, EMPLOYER VARCHAR) SALT_BUCKETS=2", |
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.
I think we should be able to add "MULTI_TENANT=TRUE" without anything failing. But I guess "testColumnsForSelectQueryOnMultiTenantTable" in the other class already tests what we want.
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.
yeah, the multitenant_tenant = true ones are in the other file
LGTM. Thanks |
… and IndexScrutinyIT doesn't run
… and IndexScrutinyIT doesn't run