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

merge dev to main #230

Merged
merged 21 commits into from
Mar 21, 2024
Merged

merge dev to main #230

merged 21 commits into from
Mar 21, 2024

Conversation

aatmanvaidya
Copy link
Collaborator

No description provided.

duggalsu and others added 21 commits March 19, 2024 02:40
feat: hash worker can add data to postgresql
- Updated urllib3 manually for botocore compatibility
- Updated groupadd, useradd and mkdir commands to handle failure
- Added setting python user to debug image
- Added venv volume to feluda api in docker compose file
- Added build args for feluda indexer and reporter
- Added volumes in docker compose file
…irements

fix: Update urllib3 package and fix feluda core dockerfile
- Created trigger function for updating timestamp
- Modified create table function with conditional for table
- Added create table for storing CRC
- Added function to create trigger for table
- Modified store and update functions to store worker_column
- Added postgres init and call to storing video CRC
- Pinned images in docker compose
feat: Add video CRC to postgres
- Modified video worker column value
feat: Added audio CRC write to postgres
chore: adding context manager to audio operator
fix: postgres store and update func in hash worker
# Conflicts:
#	src/core/store/postgresql.py
- Replaced all dynamic SQL with prepared statements
- Removed all f-strings
- Added str conversion to hash function on db store as failsafe
@aatmanvaidya aatmanvaidya requested a review from duggalsu March 21, 2024 12:19
@aatmanvaidya aatmanvaidya merged commit ed75278 into main Mar 21, 2024
5 checks passed
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.

3 participants