-
Notifications
You must be signed in to change notification settings - Fork 73
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
Remove redis #1879
Remove redis #1879
Conversation
BulkInvoiceDownloadJob.perform_now( | ||
invoice_ids, | ||
company_logo, | ||
download_id, | ||
root_url, | ||
current_url_options | ||
) |
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.
Instead of calling the Job, can you copy all the contents from the job and do a benchmarking.
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.
I did that and the results are same. Very minor difference. Should I go ahead and delete this file and uninstall redis?
Instead of this, could we exploring removing BulkInvoiceDownloadChannel instead if possible? We could remove ACa completely. If not, then we should explore anycable or using hotwire if possible, or just simple polling. |
@apoorv1316 what's the status on this? Can we wrap this up before picking up other issues? |
Closing this in favor of #1892 |
No description provided.