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

Fix missing primary keys and optional columns #114

Merged
merged 1 commit into from
Jun 11, 2021

Conversation

stephdl
Copy link
Contributor

@stephdl stephdl commented Jun 10, 2021

After the migration we have some fix to do to the database

The database is missing some primary keys. Due to the fact that adding primary keys on big tables could take some time they were not added automatically. 
By running "occ db:add-missing-primary-keys" those missing primary keys could be added manually while the instance keeps running.
Missing primary key on table "oc_federated_reshares".
Missing primary key on table "oc_systemtag_object_mapping".
Missing primary key on table "oc_comments_read_markers".
Missing primary key on table "oc_collres_resources".
Missing primary key on table "oc_collres_accesscache".
Missing primary key on table "oc_filecache_extended".

The database is missing some optional columns. Due to the fact that adding columns on big tables could take some time they were not added automatically when they can be optional. 
By running "occ db:add-missing-columns" those missing columns could be added manually while the instance keeps running. Once the columns are added some features might improve responsiveness or usability.
Missing optional column "reference_id" in table "oc_comments".

relative to this NethServer/dev#6506 (comment)

@stephdl stephdl requested a review from gsanchietti June 10, 2021 16:07
@nethbot
Copy link
Member

nethbot commented Jun 10, 2021

in 7.9.2009/autobuild:

@stephdl
Copy link
Contributor Author

stephdl commented Jun 10, 2021

@gsanchietti gsanchietti merged commit a1d6fe1 into NethServer:master Jun 11, 2021
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