-
Notifications
You must be signed in to change notification settings - Fork 0
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
LLM POC #31
Conversation
YouTube API does not allow getting more than 50 videos at once, so we need to iteratively request data if the provided list with ids has more than 50 items.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Amaaaazing job, really
I've left a few small comments for clairfication purposes mosly)
volumes: | ||
- ypm-db:/var/lib/postgresql/data | ||
|
||
volumes: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just out of curiosity - why do we need a volume here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So the data is persistent even if we stop and start a container with PostgreSQL. But I see a point in not making it persistent
Squashed and merged, thanks! |
Pull Request
Description
LibraryImportController
)compose.yml
to spin up PostgreSQLgetVideoData
toVideoController
and other methods in consequent services to retrieve video data from YouTube APIgetToken
to theTokenService
to retrieve the token from Spring's authorized client. This should allow us not to pass@RegisteredOAuth2AuthorizedClient OAuth2AuthorizedClient authClient
as a parameter to controllersRelated Issue
Issue #30
Checklist
Please review and check all applicable items:
Additional Notes
How to guide:
To spin up the DB you'd need to run the command in the terminal:
To make things work in IntelliJ IDEA you'd need to set the environment variables in the run configuration:
Screenshots (if applicable)