Skip to content

Fix #157#356

Open
mocanew wants to merge 2 commits intorootio:masterfrom
mocanew:feature/speedup-radio-schedule
Open

Fix #157#356
mocanew wants to merge 2 commits intorootio:masterfrom
mocanew:feature/speedup-radio-schedule

Conversation

@mocanew
Copy link
Copy Markdown

@mocanew mocanew commented Sep 16, 2019

Improved loading time for /radio page by merging analytics SQL queries into a single one that is parallelised by PostgreSQL.
On our machines, we've seen ~3x pageload time speedups.

Improved loading time for /radio page by merging analytics SQL queries into a single one that is parallelised by PostgreSQL.
@pep8speaks
Copy link
Copy Markdown

pep8speaks commented Sep 16, 2019

Hello @rontav! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 26:80: E501 line too long (90 > 79 characters)
Line 54:80: E501 line too long (127 > 79 characters)
Line 54:127: E502 the backslash is redundant between brackets
Line 55:13: E128 continuation line under-indented for visual indent
Line 58:80: E501 line too long (82 > 79 characters)
Line 68:80: E501 line too long (83 > 79 characters)
Line 70:80: E501 line too long (112 > 79 characters)
Line 70:112: E502 the backslash is redundant between brackets
Line 71:9: E128 continuation line under-indented for visual indent

Line 227:52: E251 unexpected spaces around keyword / parameter equals
Line 227:54: E251 unexpected spaces around keyword / parameter equals

Comment last updated at 2019-09-16 18:30:53 UTC

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