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

Catch RejectedExecutionException #10394

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tobiasKaminsky
Copy link
Member

Needs feedback from reporting customer.

Signed-off-by: tobiasKaminsky tobias@kaminsky.me

  • Tests written, or not not needed

Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
@github-actions
Copy link

Codacy

Lint

TypemasterPR
Warnings8585
Errors00

SpotBugs

CategoryBaseNew
Bad practice3333
Correctness4545
Dodgy code351351
Experimental11
Internationalization99
Multithreaded correctness99
Performance6565
Security2828
Total541541

@nextcloud-android-bot
Copy link
Collaborator

@AlvaroBrey
Copy link
Member

#10338

@AlvaroBrey
Copy link
Member

AlvaroBrey commented Jun 15, 2022

Additionally you may want to use a separate ThreadPoolExecutor, rather than the default one, so that we have a bit more margin if there are other AsyncTasks running (uploads for example). In the end, we should get rid of AsyncTasks , but this is OK for now.

tsdicloud pushed a commit to nextmcloud/android that referenced this pull request Aug 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants