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.

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-7484 Optimize mutation plan creation for upserts over tenant connection #2041

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

sanjeet006py
Copy link
Contributor

@sanjeet006py sanjeet006py commented Dec 13, 2024

Upserts to multi-tenant tables over tenant connections were regressed by 5K - 6K % compared to non-tenant connections. But with this patch upserts over tenant connection are as good as non-tenant connection.

}
}

private PTableRef getTableRef(PTableKey key, boolean doNotThrowTableNotFoundException) throws TableNotFoundException {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With this, you also mean to cover PHOENIX-7486?

Copy link
Contributor Author

@sanjeet006py sanjeet006py Dec 13, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually, PHOENIX-7486 is a more of generic JIRA for finding and fixing other occurrences of this anti pattern. I fixed one in here as that was contributing to the regressed numbers. W/o the change in PMetadataImpl I was able to bring down regression to 30-40% from 5K-6K% but need this change to bring it down to 0.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: use skipTNFE instead of doNotThrowTableNotFoundException

@virajjasani
Copy link
Contributor

Very nice find @sanjeet006py!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants