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

Creating a Post microservice #1735

Closed
HyperTHD opened this issue Feb 13, 2021 · 2 comments · Fixed by #1838
Closed

Creating a Post microservice #1735

HyperTHD opened this issue Feb 13, 2021 · 2 comments · Fixed by #1838
Assignees
Labels
area: back-end area: deployment Production or Staging deployment area: docker area: elasticsearch Elasticsearch Related issues and pull requests area: microservices area: redis Redis Database related type: enhancement New feature or request
Milestone

Comments

@HyperTHD
Copy link
Contributor

What would you like to be added:

Telescope is going to undergo a huge overhaul with the backend as we're going to convert it to operate using microservices. As part of that effort, we should have a microservice to handle the posts.

Why would you like this to be added:

A microservice to handle the posts would take away pressure from the feeds service we also plan on incorporating. The migration would be a little easier in that we have such operations currently running in our backend. It would also need to be updated to communicate with the user microservice work currently being done in #1642

As per @humpd's vision of this microservice:

Post Microservice

@HyperTHD HyperTHD added the type: enhancement New feature or request label Feb 13, 2021
@HyperTHD HyperTHD self-assigned this Feb 13, 2021
@HyperTHD HyperTHD added this to the 1.8 Release milestone Feb 13, 2021
@HyperTHD HyperTHD added area: back-end area: web server Issues related to the web server Priority: Medium type: nice to have Feature that'd be nice to have, but not a priority labels Feb 13, 2021
@HyperTHD HyperTHD removed the type: nice to have Feature that'd be nice to have, but not a priority label Feb 13, 2021
@humphd humphd changed the title Adding new Microservice to handle incoming posts Adding new Microservice to handle posts Feb 13, 2021
@HyperTHD HyperTHD added area: deployment Production or Staging deployment and removed area: web server Issues related to the web server labels Feb 13, 2021
@humphd
Copy link
Contributor

humphd commented Feb 13, 2021

I think the way to think about this service is to begin small. Here is my initial suggestion:

We will need to figure out how to connect this to our new User service for feed info. We won't be storing feeds in Redis anymore, so Feed and User info will have to come from the User service. I'm not sure if we should do that in the Posts service (i.e., have it amalgamate that info into the responses) or let the client do calls to both services and aggregate the data there. cc @raygervais for some architectural advice.

Later on I think we'll want to expand this service to include triggering indexing.

@HyperTHD HyperTHD changed the title Adding new Microservice to handle posts Creating a Post microservice Feb 14, 2021
@manekenpix
Copy link
Member

@HyperTHD I'm here if you need an extra hand. This sounds like a nice piece to work on.

@HyperTHD HyperTHD added area: docker area: elasticsearch Elasticsearch Related issues and pull requests area: redis Redis Database related and removed Priority: Medium labels Feb 15, 2021
@HyperTHD HyperTHD linked a pull request Mar 1, 2021 that will close this issue
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: back-end area: deployment Production or Staging deployment area: docker area: elasticsearch Elasticsearch Related issues and pull requests area: microservices area: redis Redis Database related type: enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants