aleksis/AlekSIS-Core

Fork 0

WIP: Draft: Add default GIN index for extended_data #2145

Draft

nik wants to merge 3 commits from feature/gin-index-on-ext-data into master

nik commented

2021-02-21 23:12:11 +01:00

Owner

PostgreSQL provides the GIN index for composite data types, like the JSONB type. It allows the index to match the keys of JSON objects, so searching in extended_data (e.g. for an import reference) can be done using an index scan.

nik commented

2021-02-21 23:15:41 +01:00

Author

Owner

added 2 commits

9dd6d8b1 - Add system check for ExtensibleModel Meta classes
6e7faba0 - Add migration to create GIN indexes

Compare with previous version

added 2 commits <ul><li>9dd6d8b1 - Add system check for ExtensibleModel Meta classes</li><li>6e7faba0 - Add migration to create GIN indexes</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-Core/-/merge_requests/500/diffs?diff_id=4948&start_sha=3be589a06af020efbbc0ff81c6c9b9ec75af62eb)

nik commented

2021-02-21 23:18:42 +01:00

Author

Owner

marked this merge request as draft

marked this merge request as **draft**

nik commented

2021-02-21 23:19:01 +01:00

Author

Owner

Left as draft so we do not forget to update all apps.

nik commented

2021-05-20 12:22:24 +02:00

Author

Owner

marked this merge request as ready

marked this merge request as **ready**

nik commented

2021-05-20 12:22:53 +02:00

Author

Owner

@hansegucker Please merge

We will check and clean up all migrations before tagging the release anyway.

@hansegucker Please merge We will check and clean up all migrations before tagging the release anyway.

hansegucker commented

2021-05-20 13:06:46 +02:00

Owner

But doesn't include this also an update of all Meta classes?

But doesn't include this also an update of all `Meta` classes?

nik commented

2021-11-12 16:45:23 +01:00

Author

Owner

marked this merge request as draft

marked this merge request as **draft**

nik commented

2021-11-12 16:45:59 +01:00

Author

Owner

This might be outdated because of #440

This pull request has changes conflicting with the target branch.

aleksis/core/checks.py
aleksis/core/mixins.py
aleksis/core/models.py

View command line instructions

Manual merge helper

Use this merge commit message when completing the merge manually.

Merge commit title

Merge pull request 'WIP: Draft: Add default GIN index for extended_data' (#2145) from feature/gin-index-on-ext-data into master

Merge commit body

Reviewed-on: https://code.teckids.org/aleksis/AlekSIS-Core/pulls/2145

Merge pull request 'WIP: Draft: Add default GIN index for extended_data' (#2145) from feature/gin-index-on-ext-data into master

Reviewed-on: https://code.teckids.org/aleksis/AlekSIS-Core/pulls/2145

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin feature/gin-index-on-ext-data:feature/gin-index-on-ext-data

git switch feature/gin-index-on-ext-data

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch master

git merge --no-ff feature/gin-index-on-ext-data

git switch feature/gin-index-on-ext-data

git rebase master

git switch master

git merge --ff-only feature/gin-index-on-ext-data

git switch feature/gin-index-on-ext-data

git rebase master

git switch master

git merge --no-ff feature/gin-index-on-ext-data

git switch master

git merge --squash feature/gin-index-on-ext-data