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

Async processing -- choose a side! #5331

Open
jisantuc opened this issue Feb 28, 2020 · 1 comment
Open

Async processing -- choose a side! #5331

jisantuc opened this issue Feb 28, 2020 · 1 comment
Labels
maintenance Things we need to do that aren't bugs or new features, but keep us sane and help the app

Comments

@jisantuc
Copy link
Contributor

Problem description

Our async processing mixes API and DB interaction in odd ways. This mixture requires us to keep extra configuration of secrets lying around and also makes it unclear what we can add to different jobs, since it's tough to remember which jobs are configured for what. For example, recently, the upload processing job was updated to add a database interaction step via the batch jar, but it didn't have the configured env variables to be able to talk to the database.

Historically our python-based interaction facilitated easier work with AWS resources, like EMR. The downside was that it required additional copies of sections of the datamodel. With the mixture we went to after that, we now have additional copies of the datamodel in python and additional configuration so we can do two kinds of interaction.

We should choose a side and remove superfluous configuration / powers.

Expected Behavior or Output

Async jobs run in python and interact only with the API or run in Scala and interact only with the database

@jisantuc
Copy link
Contributor Author

jisantuc commented Mar 9, 2020

For a recent example of new places where this causes friction, see this comment on improving error-handling with async task grid generation

@jisantuc jisantuc added the maintenance Things we need to do that aren't bugs or new features, but keep us sane and help the app label Mar 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Things we need to do that aren't bugs or new features, but keep us sane and help the app
Development

No branches or pull requests

1 participant