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

V2.0.5 #957

Merged
merged 34 commits into from Feb 24, 2020
Merged

V2.0.5 #957

merged 34 commits into from Feb 24, 2020

Conversation

swilly22
Copy link
Collaborator

No description provided.

jeffreylovitz and others added 30 commits January 30, 2020 12:22
* fixed ast mapping for path filter

* renamed tests. open a new redis graph client per tests

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>
* added mutex per matrix

* fixed PR comments
* Fix memory leaks

* Use original logic for MarkWriter
Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>
* added optimize cp

* fixed a bug. added test

* add optimization_util files

* fixed PR comments

* fixed PR comments

* added multiple branch cp optimization

* fixed PR comment

* added in place replacement at ExecutionPlan_RemoveOp

* added test for cp optimization and semi apply

* fixed PR comments
* Added non existing entity runtime error

* fixed PR comments

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>
* added filter tree compaction

* fixed PR comments

* fixed PR comments

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>
Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>
* Improve client libraries sections on both readme and clients.md
Clients libraries are subject to repetitive edition, it seems better to have a concise formatting

* refferencing my own contribution to clients libraries

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>
* Update README.md

* Update clients.md
Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>
* Deplete Apply op's match stream for every left-hand Record

* Disallow ExpandInto ops on variables beneath Apply ops

* Add tests

* PR fixes

* Argument op holds one Record

* PR fixes

* Improve logic for building OpArgument modifies arrays

* standardize logic for building tmp ExecutionPlans

* Remove sub-ExecutionPlan logic

* Fix CondTraverseReset routine

* Fix ExpandIntoReset

* Fix variable-length QueryGraph pointer

* Always reset match branch

* Add explanatory comment

* PR fixes

* PR fixes

* raxValues returns void pointer array

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>
…ctx (#933)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>
* added arr clone with cb

* fixed PR comments

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>
* Close Redis key handles

* Fix memory leak in checking whether procedures are read-only

* Improve explanatory comment

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>
* Resolve memory leaks on Path SIValues

* Update op_delete.c

* Update op_cond_var_len_traverse.c

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>
* GraphBLAS 3.2.0

* updated makefiles

* Remove unnecessary GraphBLAS build flag

Co-authored-by: Jeffrey Lovitz <jeffrey.lovitz@gmail.com>
* Simplify OpAggregate logic, remove unnecessary struct members

* Update op_aggregate.c

* Update op_aggregate.c

* Update op_aggregate.c

* Update op_aggregate.c

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>
Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>
@swilly22 swilly22 merged commit f3f38d7 into 2.0 Feb 24, 2020
@swilly22 swilly22 deleted the v2.0.5 branch February 24, 2020 06:29
DvirDukhan added a commit that referenced this pull request Mar 15, 2020
* Support float inputs for modulo computations (#895)

* Add references

* Update mkdocs.yml

* Update References.md

* fixed ast mapping for path filter (#896)

* fixed ast mapping for path filter

* renamed tests. open a new redis graph client per tests

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* added mutex per matrix (#898)

* added mutex per matrix

* fixed PR comments

* updated license headers to 2020 (#902)

* Preserve order of op's children array when introducing index scans (#912)

* Fix memory leaks (#917)

* Fix memory leaks

* Use original logic for MarkWriter

* added filter tree clone (#915)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Optimize cp (#906)

* added optimize cp

* fixed a bug. added test

* add optimization_util files

* fixed PR comments

* fixed PR comments

* added multiple branch cp optimization

* fixed PR comment

* added in place replacement at ExecutionPlan_RemoveOp

* added test for cp optimization and semi apply

* fixed PR comments

* Added non existing entity runtime error (#919)

* Added non existing entity runtime error

* fixed PR comments

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* RediSearch query error reporting (#925)

* added filter tree compaction (#922)

* added filter tree compaction

* fixed PR comments

* fixed PR comments

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Handle both possible execution orders in concurrent rename test (#926)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Refferencing a RedisGraph javascript client library (#928)

* Improve client libraries sections on both readme and clients.md
Clients libraries are subject to repetitive edition, it seems better to have a concise formatting

* refferencing my own contribution to clients libraries

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Rust client (#931)

* Update README.md

* Update clients.md

* intoduced fpClone in OpBase (#930)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Apply deplete match stream (#913)

* Deplete Apply op's match stream for every left-hand Record

* Disallow ExpandInto ops on variables beneath Apply ops

* Add tests

* PR fixes

* Argument op holds one Record

* PR fixes

* Improve logic for building OpArgument modifies arrays

* standardize logic for building tmp ExecutionPlans

* Remove sub-ExecutionPlan logic

* Fix CondTraverseReset routine

* Fix ExpandIntoReset

* Fix variable-length QueryGraph pointer

* Always reset match branch

* Add explanatory comment

* PR fixes

* PR fixes

* raxValues returns void pointer array

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* increased allowed parameters count to UNIT_MAX. fixed a bug in query_ctx (#933)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* introduce mapping between record entries and projected columns (#936)

* added arr clone with cb (#937)

* added arr clone with cb

* fixed PR comments

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Leak fixes (#935)

* Close Redis key handles

* Fix memory leak in checking whether procedures are read-only

* Improve explanatory comment

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* label matrix should be fetch right before eval (#938)

* More sensible function signature for SIValue_Free (#941)

* Resolve memory leaks on Path SIValues (#940)

* Resolve memory leaks on Path SIValues

* Update op_delete.c

* Update op_cond_var_len_traverse.c

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* GraphBLAS 3.2.0 (#942)

* GraphBLAS 3.2.0

* updated makefiles

* Remove unnecessary GraphBLAS build flag

Co-authored-by: Jeffrey Lovitz <jeffrey.lovitz@gmail.com>

* Simplify OpAggregate logic, remove unnecessary struct members (#947)

* Simplify OpAggregate logic, remove unnecessary struct members

* Update op_aggregate.c

* Update op_aggregate.c

* Update op_aggregate.c

* Update op_aggregate.c

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Add compile-time error for unsupported AST node types (#944)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* removed unused operations and types (#950)

* remove GraphBLAS cacheing from CI (#953)

* switch from OR AND semiring to ANY PAIR (#955)

* version bump

Co-authored-by: Jeffrey Lovitz <jeffrey.lovitz@gmail.com>
Co-authored-by: Guy Korland <gkorland@gmail.com>
Co-authored-by: DvirDukhan <dvir@redislabs.com>
Co-authored-by: Sceat <11330271+Sceat@users.noreply.github.com>
DvirDukhan added a commit that referenced this pull request Mar 25, 2020
* Support float inputs for modulo computations (#895)

* Add references

* Update mkdocs.yml

* Update References.md

* fixed ast mapping for path filter (#896)

* fixed ast mapping for path filter

* renamed tests. open a new redis graph client per tests

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* added mutex per matrix (#898)

* added mutex per matrix

* fixed PR comments

* updated license headers to 2020 (#902)

* Preserve order of op's children array when introducing index scans (#912)

* Fix memory leaks (#917)

* Fix memory leaks

* Use original logic for MarkWriter

* added filter tree clone (#915)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Optimize cp (#906)

* added optimize cp

* fixed a bug. added test

* add optimization_util files

* fixed PR comments

* fixed PR comments

* added multiple branch cp optimization

* fixed PR comment

* added in place replacement at ExecutionPlan_RemoveOp

* added test for cp optimization and semi apply

* fixed PR comments

* Added non existing entity runtime error (#919)

* Added non existing entity runtime error

* fixed PR comments

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* RediSearch query error reporting (#925)

* added filter tree compaction (#922)

* added filter tree compaction

* fixed PR comments

* fixed PR comments

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Handle both possible execution orders in concurrent rename test (#926)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Refferencing a RedisGraph javascript client library (#928)

* Improve client libraries sections on both readme and clients.md
Clients libraries are subject to repetitive edition, it seems better to have a concise formatting

* refferencing my own contribution to clients libraries

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Rust client (#931)

* Update README.md

* Update clients.md

* intoduced fpClone in OpBase (#930)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Apply deplete match stream (#913)

* Deplete Apply op's match stream for every left-hand Record

* Disallow ExpandInto ops on variables beneath Apply ops

* Add tests

* PR fixes

* Argument op holds one Record

* PR fixes

* Improve logic for building OpArgument modifies arrays

* standardize logic for building tmp ExecutionPlans

* Remove sub-ExecutionPlan logic

* Fix CondTraverseReset routine

* Fix ExpandIntoReset

* Fix variable-length QueryGraph pointer

* Always reset match branch

* Add explanatory comment

* PR fixes

* PR fixes

* raxValues returns void pointer array

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* increased allowed parameters count to UNIT_MAX. fixed a bug in query_ctx (#933)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* introduce mapping between record entries and projected columns (#936)

* added arr clone with cb (#937)

* added arr clone with cb

* fixed PR comments

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Leak fixes (#935)

* Close Redis key handles

* Fix memory leak in checking whether procedures are read-only

* Improve explanatory comment

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* label matrix should be fetch right before eval (#938)

* More sensible function signature for SIValue_Free (#941)

* Resolve memory leaks on Path SIValues (#940)

* Resolve memory leaks on Path SIValues

* Update op_delete.c

* Update op_cond_var_len_traverse.c

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* GraphBLAS 3.2.0 (#942)

* GraphBLAS 3.2.0

* updated makefiles

* Remove unnecessary GraphBLAS build flag

Co-authored-by: Jeffrey Lovitz <jeffrey.lovitz@gmail.com>

* Simplify OpAggregate logic, remove unnecessary struct members (#947)

* Simplify OpAggregate logic, remove unnecessary struct members

* Update op_aggregate.c

* Update op_aggregate.c

* Update op_aggregate.c

* Update op_aggregate.c

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Add compile-time error for unsupported AST node types (#944)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* removed unused operations and types (#950)

* remove GraphBLAS cacheing from CI (#953)

* switch from OR AND semiring to ANY PAIR (#955)

* version bump

Co-authored-by: Jeffrey Lovitz <jeffrey.lovitz@gmail.com>
Co-authored-by: Guy Korland <gkorland@gmail.com>
Co-authored-by: DvirDukhan <dvir@redislabs.com>
Co-authored-by: Sceat <11330271+Sceat@users.noreply.github.com>
swilly22 added a commit that referenced this pull request Apr 7, 2020
* Support float inputs for modulo computations (#895)

* Add references

* Update mkdocs.yml

* Update References.md

* fixed ast mapping for path filter (#896)

* fixed ast mapping for path filter

* renamed tests. open a new redis graph client per tests

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* added mutex per matrix (#898)

* added mutex per matrix

* fixed PR comments

* updated license headers to 2020 (#902)

* Preserve order of op's children array when introducing index scans (#912)

* Fix memory leaks (#917)

* Fix memory leaks

* Use original logic for MarkWriter

* added filter tree clone (#915)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Optimize cp (#906)

* added optimize cp

* fixed a bug. added test

* add optimization_util files

* fixed PR comments

* fixed PR comments

* added multiple branch cp optimization

* fixed PR comment

* added in place replacement at ExecutionPlan_RemoveOp

* added test for cp optimization and semi apply

* fixed PR comments

* Added non existing entity runtime error (#919)

* Added non existing entity runtime error

* fixed PR comments

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* RediSearch query error reporting (#925)

* added filter tree compaction (#922)

* added filter tree compaction

* fixed PR comments

* fixed PR comments

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Handle both possible execution orders in concurrent rename test (#926)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Refferencing a RedisGraph javascript client library (#928)

* Improve client libraries sections on both readme and clients.md
Clients libraries are subject to repetitive edition, it seems better to have a concise formatting

* refferencing my own contribution to clients libraries

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Rust client (#931)

* Update README.md

* Update clients.md

* intoduced fpClone in OpBase (#930)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Apply deplete match stream (#913)

* Deplete Apply op's match stream for every left-hand Record

* Disallow ExpandInto ops on variables beneath Apply ops

* Add tests

* PR fixes

* Argument op holds one Record

* PR fixes

* Improve logic for building OpArgument modifies arrays

* standardize logic for building tmp ExecutionPlans

* Remove sub-ExecutionPlan logic

* Fix CondTraverseReset routine

* Fix ExpandIntoReset

* Fix variable-length QueryGraph pointer

* Always reset match branch

* Add explanatory comment

* PR fixes

* PR fixes

* raxValues returns void pointer array

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* increased allowed parameters count to UNIT_MAX. fixed a bug in query_ctx (#933)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* introduce mapping between record entries and projected columns (#936)

* added arr clone with cb (#937)

* added arr clone with cb

* fixed PR comments

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Leak fixes (#935)

* Close Redis key handles

* Fix memory leak in checking whether procedures are read-only

* Improve explanatory comment

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* label matrix should be fetch right before eval (#938)

* More sensible function signature for SIValue_Free (#941)

* Resolve memory leaks on Path SIValues (#940)

* Resolve memory leaks on Path SIValues

* Update op_delete.c

* Update op_cond_var_len_traverse.c

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* GraphBLAS 3.2.0 (#942)

* GraphBLAS 3.2.0

* updated makefiles

* Remove unnecessary GraphBLAS build flag

Co-authored-by: Jeffrey Lovitz <jeffrey.lovitz@gmail.com>

* Simplify OpAggregate logic, remove unnecessary struct members (#947)

* Simplify OpAggregate logic, remove unnecessary struct members

* Update op_aggregate.c

* Update op_aggregate.c

* Update op_aggregate.c

* Update op_aggregate.c

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Add compile-time error for unsupported AST node types (#944)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* removed unused operations and types (#950)

* remove GraphBLAS cacheing from CI (#953)

* switch from OR AND semiring to ANY PAIR (#955)

* using the structured semiring we can combine relation and relation ma… (#964)

* using the structured semiring we can combine relation and relation mapping into a single matrix

* address PR comments

* Update graph.c

* decoupled result set from execution plan (#929)

* decoupled result set from execution plan

* after rebase merge

* fixed PR comments

* Restored resulte set

* fixed PR comments

* fixed PR comments

* fixed PR comments

* Update ast.c

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Always free emptied space on AlgebraicExpression replacement (#986)

* Fix leak on full-text index queries with syntax error (#985)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Fix leak on index scan IN optimization (#984)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Minor updates to bulk deletion (#994)

* Remove redundant clone from bulk insertion of string props (#993)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Fix memory leaks on RDB-loaded strings (#991)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Add flow test to validate (#967)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Update outdated logic for aggregate groups (#968)

* Update outdated logic for aggregate groups

* Update group.c

* Update op_aggregate.c

* Update op_aggregate.c

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* added all_node_scan clone (#960)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* added op argument clone (#959)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* added cartesian product clone (#963)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* added expand into clone (#973)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* added project op clone (#980)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* added skip op clone (#983)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* added op unwind clone (#988)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* added limit op clone (#976)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* added proc call clone (#979)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* added filter op clone (#974)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* added delete op clone (#970)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* added distinct op clone (#972)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* add hash join op clone (#989)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* added semi apply clone (#982)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* added results op clone (#981)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* added join op clone (#975)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* added apply multiplexer clone (#961)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* added cond var len traverse clone (#966)

* added cond var len traverse clone

* added conditional traverse clone

* fixed PR comments

* Reintroduce logic for freeing memory on run-time errors (#992)

* Reintroduce logic for freeing memory on run-time errors

* Remove VolatileRecord logic for freeing after run-time errors

* Fix memory leaks on run-time errors in OpProject

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Free internal edge arrays on relationship matrix deletion (#997)

* Free internal edge arrays on relationship matrix deletion

* Update graph.c

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Add debug function to print query (#995)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* added clone for create, merge, merge_create, update (#969)

* added clone for create, merge, merge_create, update

* Update ast_shared.c

* Update ast_shared.c

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Agg func fixes (#946)

* Fix memory leaks in Collect function

* Fix leak in aggregate func's SIValue result

* Fix leak in children of aggregate function call

* Simplify variable-length path free logic

* Improve ownership logic in Collect, add explanatory comments

* PR fix

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* added label scan clone (#978)

* added label scan clone

* fixed PR comments

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* added id seek clone (#977)

* added id seek clone

* added better comment on the range clone

* Update op_node_by_id_seek.c

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* added op aggreage clone (#958)

* added op aggreage clone

* fixed PR comments

* Update op_aggregate.c

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Add install instructions for OpenMP (#1006)

* Add install instructions for OpenMP

* Update README

* added op sort clone (#987)

* added op sort clone

* fixed PR comments

* removed free_list logic

* Update execution_plan.c

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* re-enable graph.profile (#1004)

* Re-enable GraphBLAS circle-ci cache (#1007)

* Fix leak on projected heap-allocated graph entities (#996)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* GRAPH.SLOWLOG (#897)

* slowlog WIP

* slowlog per graph

* updated docs

* Add longer-running query to slowlog flow test

* avoid race, log only GRAPH.QUERY

Co-authored-by: Jeffrey Lovitz <jeffrey.lovitz@gmail.com>

* Use NOP label scan if range iterator construction fails (#1001)

* Use NOP label scan if range iterator construction fails

* deplete iterator for invalid range

* dont access op consume function directly

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* removed redisearch optimization (#1010)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Note ReplicaOf memory constraint in Rampfile (#1011)

* Automate leak check (#1002)

* Add make memcheck rule

* Add Circle memcheck job

* Don't use Docker image for automated leak checking

* Enable log names by migrating flow test Env initialization

* Disable invalid TCK scenarios of issue #945

* Run memory test without optimizations after packaging artifacts

* Add suppressions for leaks in flush-vs-shutdown race

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Write longer-running query for slowlog test (#1012)

* OPType bitmask switched to contiguous enum (#1013)

* OPType bitmask switched to contiguous enum

* PR fixes

* Clean up op tree modification code

* Add static array of all scan ops

* Remove unnecessary conditional

* Remove static type array for unoptimized scans

* Validate arity per-command rather than globally (#1023)

* Ar exp param (#990)

* wip

* wip

* after rebase

* updated libcypher parser

* added AR_EXP_PARAM

* fixed PR comments

* fixed PR comments

* fixed PR comments

* fixed PR comments

* Update arithmetic_expression.c

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Fix invalid op sequence when introducing index scans (#1028)

* Fix invalid op sequence when partially replacing filters with index scans

* Simplify op-freeing logic in utilizeIndices

* Automate testing against enterprise v5.4.14

* push down transpose operations (#1032)

* push down transpose operations

* added test for transposed bi-directional edge

* Update algebraic_expression_optimization.c

* allow skip and limit to be parametes. (#1020)

* wip

* test pass

* added skip limit params test

* added failure test

* fixed PR comments

* fixed online review comments

* restored traverse record cap

* fixing memory leaks

* fixed PR comments

* changed redisearch version to 1.6.11 (#1035)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* version bump

* version bump

* Version 2.0.2 (#949)

* Support float inputs for modulo computations (#895)

* Add references

* Update mkdocs.yml

* Update References.md

* fixed ast mapping for path filter (#896)

* fixed ast mapping for path filter

* renamed tests. open a new redis graph client per tests

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* added mutex per matrix (#898)

* added mutex per matrix

* fixed PR comments

* updated license headers to 2020 (#902)

* Preserve order of op's children array when introducing index scans (#912)

* Fix memory leaks (#917)

* Fix memory leaks

* Use original logic for MarkWriter

* added filter tree clone (#915)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Optimize cp (#906)

* added optimize cp

* fixed a bug. added test

* add optimization_util files

* fixed PR comments

* fixed PR comments

* added multiple branch cp optimization

* fixed PR comment

* added in place replacement at ExecutionPlan_RemoveOp

* added test for cp optimization and semi apply

* fixed PR comments

* Added non existing entity runtime error (#919)

* Added non existing entity runtime error

* fixed PR comments

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* RediSearch query error reporting (#925)

* added filter tree compaction (#922)

* added filter tree compaction

* fixed PR comments

* fixed PR comments

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Handle both possible execution orders in concurrent rename test (#926)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Refferencing a RedisGraph javascript client library (#928)

* Improve client libraries sections on both readme and clients.md
Clients libraries are subject to repetitive edition, it seems better to have a concise formatting

* refferencing my own contribution to clients libraries

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Rust client (#931)

* Update README.md

* Update clients.md

* intoduced fpClone in OpBase (#930)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Apply deplete match stream (#913)

* Deplete Apply op's match stream for every left-hand Record

* Disallow ExpandInto ops on variables beneath Apply ops

* Add tests

* PR fixes

* Argument op holds one Record

* PR fixes

* Improve logic for building OpArgument modifies arrays

* standardize logic for building tmp ExecutionPlans

* Remove sub-ExecutionPlan logic

* Fix CondTraverseReset routine

* Fix ExpandIntoReset

* Fix variable-length QueryGraph pointer

* Always reset match branch

* Add explanatory comment

* PR fixes

* PR fixes

* raxValues returns void pointer array

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* increased allowed parameters count to UNIT_MAX. fixed a bug in query_ctx (#933)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* introduce mapping between record entries and projected columns (#936)

* added arr clone with cb (#937)

* added arr clone with cb

* fixed PR comments

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Leak fixes (#935)

* Close Redis key handles

* Fix memory leak in checking whether procedures are read-only

* Improve explanatory comment

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* label matrix should be fetch right before eval (#938)

* More sensible function signature for SIValue_Free (#941)

* Resolve memory leaks on Path SIValues (#940)

* Resolve memory leaks on Path SIValues

* Update op_delete.c

* Update op_cond_var_len_traverse.c

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* GraphBLAS 3.2.0 (#942)

* GraphBLAS 3.2.0

* updated makefiles

* Remove unnecessary GraphBLAS build flag

Co-authored-by: Jeffrey Lovitz <jeffrey.lovitz@gmail.com>

* Simplify OpAggregate logic, remove unnecessary struct members (#947)

* Simplify OpAggregate logic, remove unnecessary struct members

* Update op_aggregate.c

* Update op_aggregate.c

* Update op_aggregate.c

* Update op_aggregate.c

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Add compile-time error for unsupported AST node types (#944)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* version bump 2.0.2

Co-authored-by: Jeffrey Lovitz <jeffrey.lovitz@gmail.com>
Co-authored-by: Guy Korland <gkorland@gmail.com>
Co-authored-by: DvirDukhan <dvir@redislabs.com>
Co-authored-by: Sceat <11330271+Sceat@users.noreply.github.com>

* Version 2.0.3 (#951)

* Support float inputs for modulo computations (#895)

* Add references

* Update mkdocs.yml

* Update References.md

* fixed ast mapping for path filter (#896)

* fixed ast mapping for path filter

* renamed tests. open a new redis graph client per tests

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* added mutex per matrix (#898)

* added mutex per matrix

* fixed PR comments

* updated license headers to 2020 (#902)

* Preserve order of op's children array when introducing index scans (#912)

* Fix memory leaks (#917)

* Fix memory leaks

* Use original logic for MarkWriter

* added filter tree clone (#915)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Optimize cp (#906)

* added optimize cp

* fixed a bug. added test

* add optimization_util files

* fixed PR comments

* fixed PR comments

* added multiple branch cp optimization

* fixed PR comment

* added in place replacement at ExecutionPlan_RemoveOp

* added test for cp optimization and semi apply

* fixed PR comments

* Added non existing entity runtime error (#919)

* Added non existing entity runtime error

* fixed PR comments

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* RediSearch query error reporting (#925)

* added filter tree compaction (#922)

* added filter tree compaction

* fixed PR comments

* fixed PR comments

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Handle both possible execution orders in concurrent rename test (#926)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Refferencing a RedisGraph javascript client library (#928)

* Improve client libraries sections on both readme and clients.md
Clients libraries are subject to repetitive edition, it seems better to have a concise formatting

* refferencing my own contribution to clients libraries

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Rust client (#931)

* Update README.md

* Update clients.md

* intoduced fpClone in OpBase (#930)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Apply deplete match stream (#913)

* Deplete Apply op's match stream for every left-hand Record

* Disallow ExpandInto ops on variables beneath Apply ops

* Add tests

* PR fixes

* Argument op holds one Record

* PR fixes

* Improve logic for building OpArgument modifies arrays

* standardize logic for building tmp ExecutionPlans

* Remove sub-ExecutionPlan logic

* Fix CondTraverseReset routine

* Fix ExpandIntoReset

* Fix variable-length QueryGraph pointer

* Always reset match branch

* Add explanatory comment

* PR fixes

* PR fixes

* raxValues returns void pointer array

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* increased allowed parameters count to UNIT_MAX. fixed a bug in query_ctx (#933)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* introduce mapping between record entries and projected columns (#936)

* added arr clone with cb (#937)

* added arr clone with cb

* fixed PR comments

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Leak fixes (#935)

* Close Redis key handles

* Fix memory leak in checking whether procedures are read-only

* Improve explanatory comment

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* label matrix should be fetch right before eval (#938)

* More sensible function signature for SIValue_Free (#941)

* Resolve memory leaks on Path SIValues (#940)

* Resolve memory leaks on Path SIValues

* Update op_delete.c

* Update op_cond_var_len_traverse.c

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* GraphBLAS 3.2.0 (#942)

* GraphBLAS 3.2.0

* updated makefiles

* Remove unnecessary GraphBLAS build flag

Co-authored-by: Jeffrey Lovitz <jeffrey.lovitz@gmail.com>

* Simplify OpAggregate logic, remove unnecessary struct members (#947)

* Simplify OpAggregate logic, remove unnecessary struct members

* Update op_aggregate.c

* Update op_aggregate.c

* Update op_aggregate.c

* Update op_aggregate.c

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Add compile-time error for unsupported AST node types (#944)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* removed unused operations and types (#950)

* version bump 2.0.3

Co-authored-by: Jeffrey Lovitz <jeffrey.lovitz@gmail.com>
Co-authored-by: Guy Korland <gkorland@gmail.com>
Co-authored-by: DvirDukhan <dvir@redislabs.com>
Co-authored-by: Sceat <11330271+Sceat@users.noreply.github.com>

* V2.0.4 (#954)

* Support float inputs for modulo computations (#895)

* Add references

* Update mkdocs.yml

* Update References.md

* fixed ast mapping for path filter (#896)

* fixed ast mapping for path filter

* renamed tests. open a new redis graph client per tests

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* added mutex per matrix (#898)

* added mutex per matrix

* fixed PR comments

* updated license headers to 2020 (#902)

* Preserve order of op's children array when introducing index scans (#912)

* Fix memory leaks (#917)

* Fix memory leaks

* Use original logic for MarkWriter

* added filter tree clone (#915)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Optimize cp (#906)

* added optimize cp

* fixed a bug. added test

* add optimization_util files

* fixed PR comments

* fixed PR comments

* added multiple branch cp optimization

* fixed PR comment

* added in place replacement at ExecutionPlan_RemoveOp

* added test for cp optimization and semi apply

* fixed PR comments

* Added non existing entity runtime error (#919)

* Added non existing entity runtime error

* fixed PR comments

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* RediSearch query error reporting (#925)

* added filter tree compaction (#922)

* added filter tree compaction

* fixed PR comments

* fixed PR comments

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Handle both possible execution orders in concurrent rename test (#926)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Refferencing a RedisGraph javascript client library (#928)

* Improve client libraries sections on both readme and clients.md
Clients libraries are subject to repetitive edition, it seems better to have a concise formatting

* refferencing my own contribution to clients libraries

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Rust client (#931)

* Update README.md

* Update clients.md

* intoduced fpClone in OpBase (#930)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Apply deplete match stream (#913)

* Deplete Apply op's match stream for every left-hand Record

* Disallow ExpandInto ops on variables beneath Apply ops

* Add tests

* PR fixes

* Argument op holds one Record

* PR fixes

* Improve logic for building OpArgument modifies arrays

* standardize logic for building tmp ExecutionPlans

* Remove sub-ExecutionPlan logic

* Fix CondTraverseReset routine

* Fix ExpandIntoReset

* Fix variable-length QueryGraph pointer

* Always reset match branch

* Add explanatory comment

* PR fixes

* PR fixes

* raxValues returns void pointer array

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* increased allowed parameters count to UNIT_MAX. fixed a bug in query_ctx (#933)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* introduce mapping between record entries and projected columns (#936)

* added arr clone with cb (#937)

* added arr clone with cb

* fixed PR comments

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Leak fixes (#935)

* Close Redis key handles

* Fix memory leak in checking whether procedures are read-only

* Improve explanatory comment

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* label matrix should be fetch right before eval (#938)

* More sensible function signature for SIValue_Free (#941)

* Resolve memory leaks on Path SIValues (#940)

* Resolve memory leaks on Path SIValues

* Update op_delete.c

* Update op_cond_var_len_traverse.c

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* GraphBLAS 3.2.0 (#942)

* GraphBLAS 3.2.0

* updated makefiles

* Remove unnecessary GraphBLAS build flag

Co-authored-by: Jeffrey Lovitz <jeffrey.lovitz@gmail.com>

* Simplify OpAggregate logic, remove unnecessary struct members (#947)

* Simplify OpAggregate logic, remove unnecessary struct members

* Update op_aggregate.c

* Update op_aggregate.c

* Update op_aggregate.c

* Update op_aggregate.c

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Add compile-time error for unsupported AST node types (#944)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* removed unused operations and types (#950)

* remove GraphBLAS cacheing from CI (#953)

* version bump

Co-authored-by: Jeffrey Lovitz <jeffrey.lovitz@gmail.com>
Co-authored-by: Guy Korland <gkorland@gmail.com>
Co-authored-by: DvirDukhan <dvir@redislabs.com>
Co-authored-by: Sceat <11330271+Sceat@users.noreply.github.com>

* V2.0.5 (#957)

* Support float inputs for modulo computations (#895)

* Add references

* Update mkdocs.yml

* Update References.md

* fixed ast mapping for path filter (#896)

* fixed ast mapping for path filter

* renamed tests. open a new redis graph client per tests

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* added mutex per matrix (#898)

* added mutex per matrix

* fixed PR comments

* updated license headers to 2020 (#902)

* Preserve order of op's children array when introducing index scans (#912)

* Fix memory leaks (#917)

* Fix memory leaks

* Use original logic for MarkWriter

* added filter tree clone (#915)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Optimize cp (#906)

* added optimize cp

* fixed a bug. added test

* add optimization_util files

* fixed PR comments

* fixed PR comments

* added multiple branch cp optimization

* fixed PR comment

* added in place replacement at ExecutionPlan_RemoveOp

* added test for cp optimization and semi apply

* fixed PR comments

* Added non existing entity runtime error (#919)

* Added non existing entity runtime error

* fixed PR comments

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* RediSearch query error reporting (#925)

* added filter tree compaction (#922)

* added filter tree compaction

* fixed PR comments

* fixed PR comments

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Handle both possible execution orders in concurrent rename test (#926)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Refferencing a RedisGraph javascript client library (#928)

* Improve client libraries sections on both readme and clients.md
Clients libraries are subject to repetitive edition, it seems better to have a concise formatting

* refferencing my own contribution to clients libraries

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Rust client (#931)

* Update README.md

* Update clients.md

* intoduced fpClone in OpBase (#930)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Apply deplete match stream (#913)

* Deplete Apply op's match stream for every left-hand Record

* Disallow ExpandInto ops on variables beneath Apply ops

* Add tests

* PR fixes

* Argument op holds one Record

* PR fixes

* Improve logic for building OpArgument modifies arrays

* standardize logic for building tmp ExecutionPlans

* Remove sub-ExecutionPlan logic

* Fix CondTraverseReset routine

* Fix ExpandIntoReset

* Fix variable-length QueryGraph pointer

* Always reset match branch

* Add explanatory comment

* PR fixes

* PR fixes

* raxValues returns void pointer array

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* increased allowed parameters count to UNIT_MAX. fixed a bug in query_ctx (#933)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* introduce mapping between record entries and projected columns (#936)

* added arr clone with cb (#937)

* added arr clone with cb

* fixed PR comments

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Leak fixes (#935)

* Close Redis key handles

* Fix memory leak in checking whether procedures are read-only

* Improve explanatory comment

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* label matrix should be fetch right before eval (#938)

* More sensible function signature for SIValue_Free (#941)

* Resolve memory leaks on Path SIValues (#940)

* Resolve memory leaks on Path SIValues

* Update op_delete.c

* Update op_cond_var_len_traverse.c

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* GraphBLAS 3.2.0 (#942)

* GraphBLAS 3.2.0

* updated makefiles

* Remove unnecessary GraphBLAS build flag

Co-authored-by: Jeffrey Lovitz <jeffrey.lovitz@gmail.com>

* Simplify OpAggregate logic, remove unnecessary struct members (#947)

* Simplify OpAggregate logic, remove unnecessary struct members

* Update op_aggregate.c

* Update op_aggregate.c

* Update op_aggregate.c

* Update op_aggregate.c

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* Add compile-time error for unsupported AST node types (#944)

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>

* removed unused operations and types (#950)

* remove GraphBLAS cacheing from CI (#953)

* switch from OR AND semiring to ANY PAIR (#955)

* version bump

Co-authored-by: Jeffrey Lovitz <jeffrey.lovitz@gmail.com>
Co-authored-by: Guy Korland <gkorland@gmail.com>
Co-authored-by: DvirDukhan <dvir@redislabs.com>
Co-authored-by: Sceat <11330271+Sceat@users.noreply.github.com>

* rebase master to v2.0.5. Changed version to 2.0.6

* V2.0.7

* V2.0.8

* V2.0.9

* version bump

Co-authored-by: Guy Korland <gkorland@gmail.com>
Co-authored-by: DvirDukhan <dvir@redislabs.com>
Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>
Co-authored-by: Sceat <11330271+Sceat@users.noreply.github.com>
Co-authored-by: Pieter Cailliau <pieter@redislabs.com>
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.

None yet

5 participants