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

New Locking, Various Bugfixes, UI Improvements #502

Merged
merged 23 commits into from
Jul 28, 2024
Merged

New Locking, Various Bugfixes, UI Improvements #502

merged 23 commits into from
Jul 28, 2024

Conversation

datomo
Copy link
Member

@datomo datomo commented Jul 5, 2024 •
edited

Summary

This PR requires the PR #98 from Polypheny-UI.

This PR fixes various bugs and inconsistencies. The main features are adjustment to the LockManager and evaluation of new constraints upon creation.

Fixes:

  • Wrong transaction times #5]
  • Error on startup -> added missing restore logic
  • Materialized views from constants
  • Improved robustness of statistics to remove random exceptions
  • Fixed errors in CURRENT and LOCAL time functions for SQL
  • Fix for namespace queries in MQL (DMLs, DQLs)
  • Adjustment to integrity checks

Changes

  • New LockManager
  • Attaching new constraint to transaction for evaluation during commit

datomo added 18 commits June 24, 2024 14:57
@datomo
safety in statistic to prevent random race-conditions
1a6a259
@datomo
fixed notebooks queries
40a1ae5
@datomo
adjusting of locking mechanism
297d0e8
@datomo
removed non-thread safe locking, replaced with conservative locking
42602df
@datomo
added missing case, where xLock already is held by tx
6162f18
@datomo
reverted to default store
8ab9092
@datomo
fixed extraction of partitions
1b3c093
@datomo
finally fixed deadlock scenario
2cb3fdb
@datomo
fixed wrong transaction times
3199905
@datomo
added more robust locking, with starving protection
8bca347
@datomo
fix for materialized views from derived values
cfa8dda
@datomo
fixed some cases where lock already acquired
479d266
@datomo
fix for namespace syntax in mql
bfa7f63
@datomo
stop creation of document views
08e0238
@datomo
adjusted waiting of threads
7c97562
@datomo
fix for local and current time and date functions
f2998e7
@datomo
added enforcement of new constraints for during creation and during r…
5832df0 
…untime
@datomo
fixed incorrect remove of table for constraint
c1fe145
@datomo datomo mentioned this pull request Jul 5, 2024
Merged
vogti
vogti approved these changes Jul 28, 2024
View reviewed changes
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

This comment was marked as duplicate.

Sign in to view

@vogti vogti merged commit 18d3cce into master Jul 28, 2024
32 checks passed
@vogti vogti deleted the workload-fixes branch July 28, 2024 11:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vogti vogti vogti approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@datomo @vogti @gartens