Allow apps to register columns for bigint migrations
Nobody has claimed this yet.
- Dominant language
- PHP
- Stars
- 36.9k
- Forks
- 5.2k
- Avg merge
- 2d 3h
- Merged PRs (30d)
- 713
Description
Similar to the AddMissingColumns and AddMissingIndices commands we should allow apps to register columns that shall be migrated to bigint with the occ db:convert-filecache-bigint command. We'd probably rename that to occ db:convert-to-bigint then to remove the filecache indications, since it does more than that.
Therefore we would introduce another event set similar to the existing ones:
https://github.com/nextcloud/server/blob/fe46149560ede7a071557e4f6b2b32a9d286327a/lib/public/IDBConnection.php#L51-L54
As discussed with @nickvergessen in https://github.com/nextcloud/richdocuments/pull/1243
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the IDBConnection event definitions and the existing AddMissingColumns and AddMissingIndices commands, then trace occ db:convert-filecache-bigint. Determine how apps should register columns and how the command can be generalized beyond filecache. Done means registered app columns are migrated to bigint and the command naming reflects its broader scope.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- php
- Domain
- database
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100