2 2 12 upversion by jeffreylovitz · Pull Request #1512 · RedisGraph/RedisGraph · GitHub
Skip to content

2 2 12 upversion#1512

Merged
swilly22 merged 14 commits into
2.2from
2-2-12-upversion
Dec 27, 2020
Merged

2 2 12 upversion#1512
swilly22 merged 14 commits into
2.2from
2-2-12-upversion

Conversation

@jeffreylovitz

Copy link
Copy Markdown
Contributor

No description provided.

rafie and others added 14 commits December 27, 2020 12:00
* Added QA automation support

* fixes 1

* fixes 2

(cherry picked from commit ebdbd6e)
Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>
(cherry picked from commit 0ead11e)
* Refactor logic for invoking aggregate functions

* WIP

* Fix unit tests, stDevP logic

* Fix formatting flag

* reduce and evaluate aggregation functions

* consolidation agg and none agg functions

* Additional tests, comment corrections

Co-authored-by: swilly22 <roi@redislabs.com>
Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>
(cherry picked from commit aa66db8)
* global cache

* Replace the cache-per-thread with a global cache:
-Use rwlock to synchronize access to the cache.
-Replace the hash lookup of queries strings with rax to support safe access of multiple readers.
-When cache hit occurs, do (shallow) memcpy of the query AST along with reference count increase, and store the current execution params in the new AST copy.

* Fix memory leaks.
Bug fix: set AST in thread local storage after the shallow copy.

* Update comments in config files and number of threads in test_cache to fit the new cache design.

* Defending execution plans that are being cloned from evicted by another thread, by adding ref count.

* Change cache implementation so that cache item is copied within the cache get/set functions by using a callback. This ensures that cache item will not be freed when evicted from cache while it is cloned by another thread.

* Change cache implementation so that cache item is copied within the cache get/set functions by using a callback. This ensures that cache item will not be freed when evicted from cache while it is cloned by another thread.

* moved from ulong to long long

* refactor cache unit-test

* refactored cache

* Refactor cache_set and test_cache.

* mostly formatting

* switch to new config get

Co-authored-by: swilly22 <roi@redislabs.com>
Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>
(cherry picked from commit 75d0a99)
* Add support for startNode and endNode functions

* Address PR comments

(cherry picked from commit 584f59b)
* Disallow updating properties after deleting their entities

* Address PR comments

(cherry picked from commit 92125cf)
* Disallow assigning properties to complex types in MERGE SET constructions

* Address PR comments

(cherry picked from commit 6bc3552)
* Consider both filters and labels in selecting a starting point

* refactored traverse order

* Address PR comments

* Update test_traversal_ordering.cpp

Co-authored-by: swilly22 <roi@redislabs.com>
Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>
(cherry picked from commit 818e541)
* optimize expression before accessing it

* algebraic expression transposed should handle directly nested transposes

(cherry picked from commit 78f5310)
* Don't attempt to access non-existent properties

* Address PR comments

* Add additional tests

* Add documentation

* Skip tests that cause memory leaks

* Address PR comments

(cherry picked from commit 50aa868)
* Error on alias references in parameters

* Address PR comments

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>
(cherry picked from commit aed3fba)
@swilly22 swilly22 merged commit efe17d9 into 2.2 Dec 27, 2020
@swilly22 swilly22 deleted the 2-2-12-upversion branch December 27, 2020 18:26
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.

5 participants