2 2 9 upversion #1467
Merged
2 2 9 upversion #1467
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Validate number of RETURNs in UNION query * Improve error handling and messages with invalid WHERE clauses * Revert changes to optimize_cartesian_product * Add run-time error for non-boolean predicate constants in runtime values (cherry picked from commit ebfc507)
* improve graph loading time by noting multi-edge matices * compute multi-edge only once (cherry picked from commit 514fa25)
(cherry picked from commit 5d6fdfc)
* Disallow property assignment to complex data types * Address PR comments * Capture errors before committing updates Co-authored-by: Roi Lipman <[email protected]> (cherry picked from commit 524f171)
* Don't assert on failed MERGE updates * Update op_merge.c Co-authored-by: Roi Lipman <[email protected]> (cherry picked from commit b2dadd8)
* Disallow traversal patterns outside of the appropriate clauses * Address PR comments Co-authored-by: Roi Lipman <[email protected]> (cherry picked from commit 0f53447)
* add redis-cli parameter example I could not find how to set parameters on `redis-cli` on the documentation, so I added it here. Is there a better place? maybe https://oss.redislabs.com/redisgraph/commands/? * * add a second parameter to make it clear how to use more than a single parameter Co-authored-by: Guy Korland <[email protected]> (cherry picked from commit 3f19bd1)
* Fix warnings reported by fb-infer * validate algebraic expression structure on evaluation Co-authored-by: Roi Lipman <[email protected]> Co-authored-by: swilly22 <[email protected]> (cherry picked from commit eefb8ef)
(cherry picked from commit 17663a6)
CPP is the environment variable for the preprocessor executable, CXX specifies the compiler path. (cherry picked from commit 88048ee)
* Disallow MERGE of entities with null properties * Fix memory leak * Move error-handling functions to errors.c * Fix NULL serialization logic * Address PR comments * Address PR comments * Move all error-handling logic to errors.c * Fix memory leak * Fix unit test * Initialize ErrorCtx thread-local key * reuse TLS error context, set and raise error in one call * remove unused include * Reset error breakpoint after every query * Don't delete property update maps on encountering NULL values * Add Valgrind suppression * Expose ErrorCtx getter for identical logic in handler macro Co-authored-by: Roi Lipman <[email protected]> Co-authored-by: swilly22 <[email protected]> (cherry picked from commit c972f72)
* new php client redislabs-redisgraph-php added * columns are aligned. * Update clients.md * Update clients.md Co-authored-by: Roi Lipman <[email protected]> (cherry picked from commit bbd5993)
Co-authored-by: Guy Korland <[email protected]> Co-authored-by: Roi Lipman <[email protected]> (cherry picked from commit 081c863)
* validate clauses following a RETURN clause * relocated return clause sequance validation * Update ast_validations.c (cherry picked from commit 2157919)
|
This pull request fixes 1 alert when merging 7249f24 into 4490185 - view on LGTM.com fixed alerts:
|
(cherry picked from commit 682a5ba)
* switch from assert to debug assert * Continue converting to debug assert * Revert changes to RG.h * switch from system assert to RedisModule_Assert * re-order include sequence Co-authored-by: Jeffrey Lovitz <[email protected]> (cherry picked from commit 8345677)
jeffreylovitz
force-pushed
the
2-2-9-upversion
branch
from
7249f24 to
f4918c6
Compare
* Add stars to clients * Update clients.md (cherry picked from commit b3e4222)
jeffreylovitz
force-pushed
the
2-2-9-upversion
branch
from
f4918c6 to
369a6bf
Compare
jeffreylovitz
force-pushed
the
2-2-9-upversion
branch
from
0d6e3f2 to
270040c
Compare
|
This pull request fixes 1 alert when merging 270040c into 4490185 - view on LGTM.com fixed alerts:
|
* Don't recurse into previous scopes when optimizing Cartesian Products * Fix crash on migrating cartesian products with constant filters * Don't assert on non-filter branch head * Avoid memory leak * Fix memory leak (cherry picked from commit 9b82f8b)
jeffreylovitz
force-pushed
the
2-2-9-upversion
branch
from
270040c to
819a1be
Compare
|
This pull request fixes 1 alert when merging 819a1be into 6bac990 - view on LGTM.com fixed alerts:
|
swilly22
approved these changes
Dec 2, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR for service pack 2.2.9