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.

Sign up for GitHub

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

Jump to bottom

Postgres diskspace usage, indexing on TPH #1650

Open
oskogstad opened this issue Jan 6, 2025 · 1 comment
Open

Postgres diskspace usage, indexing on TPH #1650

oskogstad opened this issue Jan 6, 2025 · 1 comment
Labels
needs consideration Requires additional consideration

Comments

@oskogstad
Copy link
Collaborator

No description provided.

@oskogstad oskogstad added the needs consideration Requires additional consideration label Jan 6, 2025
@elsand elsand mentioned this issue Jan 8, 2025
Open
@elsand
Copy link
Member

elsand commented Jan 8, 2025 •
edited
Loading

Refinement:

See comment on #1497.

Actors do also carry alot of duplication, as the name of the actor is resolved immediately and stored in full. A better approach would be to have a internal "ActorName" table, which is referred to by id from each actor object within a dialog aggregate.

Name resolution should still happen immediately, but instead of storing to the actor-entity directly, a lookup for that identifier/name pair is performed, and if it matches, a reference to that actorname is stored. If no match is found (either the actor is previously unseen, or has changed name), a new entry is stored and referred to by the actor-object. This will effectively normalize the name property of the actor entity, while maintaining immutability with regards to name changing etc.

Edit: added a separate issue for this: #1669

@elsand elsand closed this as completed Jan 9, 2025
@elsand elsand reopened this Jan 9, 2025
@elsand elsand moved this from Done to Backlog in Dialogporten / Arbeidsflate Jan 13, 2025
@elsand elsand mentioned this issue Feb 5, 2025
Closed
@oskogstad oskogstad mentioned this issue Feb 26, 2025
Merged
4 tasks
oskogstad added a commit that referenced this issue Feb 28, 2025
@oskogstad
feat: Add reverse indexes on Localizations and SearchTags (#1971)
7a506b3 
<!--- Provide a general summary of your changes in the Title above -->

## Description

<!--- Describe your changes in detail -->

## Related Issue(s)

- #[1650](#1650)

## Verification

- [x] **Your** code builds clean without any errors or warnings
- [x] Manual testing done (required)
- [ ] Relevant automated test added (if you find this hard, leave it and
we'll help out)

## Documentation

- [ ] Documentation is updated (either in `docs`-directory, Altinnpedia
or a separate linked PR in
[altinn-studio-docs.](https://github.com/Altinn/altinn-studio-docs), if
applicable)

Co-authored-by: Bjørn Dybvik Langfors <[email protected]>
Co-authored-by: Magnus Sandgren
<[email protected]>
This was referenced Feb 28, 2025
Merged
Merged
oskogstad added a commit that referenced this issue Feb 28, 2025
@oskogstad
fix: Increase migration job timeout (#1985)
393f151 
<!--- Provide a general summary of your changes in the Title above -->

## Description

<!--- Describe your changes in detail -->

## Related Issue(s)

- #1650 

## Verification

- [ ] **Your** code builds clean without any errors or warnings
- [ ] Manual testing done (required)
- [ ] Relevant automated test added (if you find this hard, leave it and
we'll help out)

## Documentation

- [ ] Documentation is updated (either in `docs`-directory, Altinnpedia
or a separate linked PR in
[altinn-studio-docs.](https://github.com/Altinn/altinn-studio-docs), if
applicable)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
needs consideration Requires additional consideration
Projects
Dialogporten / Arbeidsflate
Status: Product Backlog
Milestone
No milestone
Development

No branches or pull requests
2 participants
@elsand @oskogstad