[BUG] CI shows performance issues with Room

Open Beginner friendly
#4,971 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
78/100
Issue type
Bug
Clarity
Clearly specified
Activity status
Active
Tech stack
android, kotlin
Domain
ci-cd, databases, mobile-dev

Research direction

Start with owncloudData/src/main/java/com/owncloud/android/data/files/db/OCFileEntity.kt and owncloudData/src/main/java/com/owncloud/android/data/spaces/db/SpaceSpecialEntity.kt, then inspect the CI run linked in the issue. Check how the foreign-key columns are declared and how indexes are represented in these entities. Done means the KSP warnings about missing indexes no longer appear in the CI output.

Written by the indexing model from the issue text.

Description

Technical
Actual behaviour

https://github.com/owncloud/android/actions/runs/33727074545/job/100558399903?pr=4967

Image

as text:

> Task :owncloudData:kspDebugKotlin
w: [ksp] /home/runner/work/android/android/owncloudData/src/main/java/com/owncloud/android/data/files/db/OCFileEntity.kt:68: (owner,spaceId) column(s) reference a foreign key but they are not part of an index. This may trigger full table scans whenever parent table is modified so you are highly advised to create an index that covers these columns.
w: [ksp] /home/runner/work/android/android/owncloudData/src/main/java/com/owncloud/android/data/spaces/db/SpaceSpecialEntity.kt:45: (spaces_special_account_name,spaces_special_space_id) column(s) reference a foreign key but they are not part of an index. This may trigger full table scans whenever parent table is modified so you are highly advised to create an index that covers these columns.
Dominant language
Kotlin
Stars
4.2k
Forks
3.1k
Avg merge
1d 12h
Merged PRs (30d)
7

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from owncloud/android

All issues in owncloud/android

Similar issues

More Kotlin issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.