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

Performance Improvement for Documents, Interval Adjustements, Adjustements for UI Logic and Various Fixes #483

Merged
merged 68 commits into from
Apr 8, 2024

Conversation

datomo
Copy link
Member

@datomo datomo commented Mar 25, 2024

Summary

This PR adjusts the value representation for intervals to be more generic and fit to the previously introduced PolyValues.
Additionally it adjusts the serialization logic for PolyDocuments to improve the general document performance.
Further it fixes various bugs and adds missing logic to the webui module.

Changes

  • PolyInterval streamlined and adjustments
  • PolyDocument performance
  • Cleanup and streamlining of PolyValues

Bug Fixes

  • Filtered Source queries let to errors
  • Fixed Source deleting
  • Source fixes for removal of columns
  • Multimedia INSERT and UPDATE
  • Snapshot rollbacks leading to new snapshots -> infinite loop
  • ConcurrentModificationException on webui broadcast
  • Deployment of Placements and Constraints via UI and CLI
  • Various missing error handling and rollback errors
  • Transmitting of Config errors on enable to webui

datomo and others added 23 commits March 17, 2024 01:21
…ary substitution for documents to improve performance
@datomo datomo marked this pull request as ready for review April 8, 2024 07:30
Copy link
Member

@vogti vogti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thx, @datomo, for this PR!

@vogti vogti merged commit 250079c into master Apr 8, 2024
38 checks passed
@vogti vogti deleted the document-fix branch April 8, 2024 15:09
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