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

Convert Aggregator tasks to one-aggregator-per-task #29

Open
thcrock opened this issue Jun 30, 2017 · 0 comments
Open

Convert Aggregator tasks to one-aggregator-per-task #29

thcrock opened this issue Jun 30, 2017 · 0 comments

Comments

@thcrock
Copy link
Contributor

thcrock commented Jun 30, 2017

Our aggregation tasks are too big. They are currently scoped to everything needed to produce a particular dataset, which sometimes ends up being multiple classifiers, a skill extractor, etc. This can end up using a lot of memory. And if we want to output multiple datasets that use the same aggregation task, all that work ends up being done again by each dataset generator.

We should really split each of these aggregation tasks into its own Airflow task, which can then be merged together afterwards.

We can do most of this here in the Airflow repository, by splitting tasks like GeoTitleCount into smallers ones that output smaller CSVs, and adding a new merge task that combines the CSVs into the output currently produced.

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

No branches or pull requests

1 participant