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

Consider pulling user icontime out of metadata into the users_entity table (Trac #4373) #4373

Closed
elgg-gitbot opened this issue Feb 16, 2013 · 5 comments
Closed

Consider pulling user icontime out of metadata into the users_entity table (Trac #4373) #4373

elgg-gitbot opened this issue Feb 16, 2013 · 5 comments
Labels
engine feature

Comments

@elgg-gitbot
Copy link

Original ticket http://trac.elgg.org/ticket/4373 on 42130203-08-02 by brettp, assigned to unknown.

Elgg version: 1.7

This is used often and could benefit from being an object property instead of metadata. See discussion at http://community.elgg.org/pg/forum/topic/844549/elgg-website-crashed-database-overload-due-to-incorrect-queries/
@elgg-gitbot
Copy link
Author

ewinslow wrote on 42365184-09-26

I think we should come to agreement on our discussion about how to better store file uploads/icons before we consider something like this.

@elgg-gitbot
Copy link
Author

Milestone changed to Long Term Discussion by ewinslow on 42365184-09-26

@mrclay
Copy link
Member

mrclay commented Jun 6, 2013

This should not be a problem with MD prefetch and cache

@mrclay mrclay closed this as completed Jun 6, 2013
@ewinslow
Copy link
Contributor

ewinslow commented Jun 6, 2013

Even if we did this I would think it should go all the way to the entities table. Users aren't the only ones with icons, after all.

@mrclay
Copy link
Member

mrclay commented Jun 6, 2013

Might be useful to base this on some real data on the most common metadata. With name/description I see big wins in eliminating joins, not so much with metadata since we do that all with one query (MD cache populateFromEntities) we have to perform anyway.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
engine feature
Milestone
No milestone
Development

No branches or pull requests
3 participants
@mrclay @ewinslow @elgg-gitbot