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

107 index transformations #130

Merged
merged 27 commits into from
Jan 2, 2018
Merged

Conversation

lucaslavandeira
Copy link
Contributor

@lucaslavandeira lucaslavandeira commented Dec 29, 2017

Closes #107
Closes #118
Closes #121
Closes #59
Reimplementación del módulo de indexación de datos con el objetivo de calcular la mayor cantidad posible de combinaciones de transformaciones, agregaciones y colapsos de tiempo para evitar hacer los cálculos en memoria durante los requests.

La nueva implementación utiliza las colas de trabajos de Redis para la indexación, haciendo posible la paralelización de la misma agregando más workers.

También haciendo esta indexación previa se corrijen bugs relacionados a los datos devueltos

lucaslavandeira and others added 26 commits December 11, 2017 15:43
@poligarcia
Copy link
Contributor

@lucaslavandeira para que el build pase, ahora que los tests tardan más, deberíamos aumentar el tiempo que Travis permite que se ejecuten los tests!

@coveralls
Copy link

Coverage Status

Coverage decreased (-2.9%) to 83.048% when pulling 2a78352 on 107-index-transformations into 684b936 on master.

@lucaslavandeira lucaslavandeira merged commit 6b9e0ad into master Jan 2, 2018
@lucaslavandeira lucaslavandeira deleted the 107-index-transformations branch January 3, 2018 18:38
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.

None yet

3 participants