Skip to content
This repository has been archived by the owner on Jul 1, 2022. It is now read-only.

PEF: performance enhancement. Load classes selectively. #1241

Merged
merged 1 commit into from Aug 28, 2018

Conversation

ygerasimov
Copy link
Collaborator

There was a problem on YMCA Brandywine with loading data for all their classes (more than 30k records). After loading them selectively performance issue was resolved.

Copy link
Contributor

@podarok podarok left a comment

Choose a reason for hiding this comment

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

Code looks good

@gundevel
Copy link
Collaborator

Build comment file:

Environment Link
Fresh OpenY installation - Rose theme http://openy.cibox.tools/build831
Fresh OpenY installation - Lily theme http://lily.openy.cibox.tools/build831
Upgraded(upgrade path) installation http://upgrade.openy.cibox.tools/build831
Installation process http://install.openy.cibox.tools/build831/install.php

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://openy.cibox.tools:8080/job/PR_BUILDER_COMPOSER/831/

@gundevel
Copy link
Collaborator

Acessibility Sniffer: front page checking WCAG2AA http://openy.cibox.tools/build831/frontWCAG2AAhtmlcs.txt
Acessibility Sniffer: join page checking WCAG2AA http://openy.cibox.tools/build831/joinWCAG2AAhtmlcs.txt
Acessibility Sniffer: locations page checking WCAG2AA http://openy.cibox.tools/build831/locationsWCAG2AAhtmlcs.txt
Acessibility Sniffer: schedules page checking WCAG2AA http://openy.cibox.tools/build831/schedulesWCAG2AAhtmlcs.txt
Acessibility Sniffer: blog page checking WCAG2AA http://openy.cibox.tools/build831/blogWCAG2AAhtmlcs.txt

Copy link
Contributor

@Sanchiz Sanchiz left a comment

Choose a reason for hiding this comment

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

Approved. Works good on Brandywine.

@podarok podarok merged commit 252dbf2 into ymcatwincities:8.x-1.x Aug 28, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants