FILTER keyword for latest anchor queries (planner/memory)#150

Merged

thiagovas merged 26 commits intogoogle:masterfrom rogerlucena:filter-latest-planner-memory

Oct 16, 2020

+1,368-234

Commits on Oct 13, 2020

Add three new triples to the ?test graph of "planner_test.go" (to better test FILTER)
rogerlucena
committed
Add tests for FILTER with latest anchor (in "planner_test.go")
rogerlucena
committed
Add "filters" to the "queryPlan" type
rogerlucena
committed
Update String method of queryPlan to show filters as well
rogerlucena
committed
Add FilterOptions to LookupOptions in "storage.go"
rogerlucena
committed
Add support for FILTER clauses in the planner
rogerlucena
committed
Make the volatile driver consume lo.FilterOptions (focusing on latest anchor only for now)
rogerlucena
committed
Update String method of LookupOptions to show FilterOptions as well
rogerlucena
committed
Minor renaming inside "memory_test.go" to avoid confusion
rogerlucena
committed
Fix problem with partial predicate UUIDs in the volatile driver and make FILTER for latest return multiple triples if they share the same predicate and same latest anchor
rogerlucena
committed
Add tests for the FILTER execution in the volatile driver (in "memory_test.go")
rogerlucena
committed
Allow user to specify for which bindings in the clause each filter can be applied to, improving error checking, and also move verification for supported filter functions to the planner level
rogerlucena
committed
Use a closure to make planner more performant (especially with the addition of multiple filter functions in the future)
rogerlucena
committed
Use an "enum" instead of a "string" for filter operations (less error-prone when implementing the driver)
rogerlucena
committed
Update "memory.go" and "memory_test.go" since filter operations are "enum" now
rogerlucena
committed
Update "hooks_test.go" and "semantic_test.go" since filter operations are "enum" now
rogerlucena
committed
Move "storage.FilteringOptions" to "filter.go" as "filter.StorageOptions"
rogerlucena
committed
Add a String method for filter.StorageOptions
rogerlucena
committed
Use an "enum" instead of a "string" for filter fields (less error-prone when implementing the driver)
rogerlucena
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

FILTER keyword for latest anchor queries (planner/memory)#150

Commits on Oct 13, 2020

Add three new triples to the ?test graph of "planner_test.go" (to better test FILTER)

Add tests for FILTER with latest anchor (in "planner_test.go")

Add "filters" to the "queryPlan" type

Update String method of queryPlan to show filters as well

Add FilterOptions to LookupOptions in "storage.go"

Add support for FILTER clauses in the planner

Make the volatile driver consume lo.FilterOptions (focusing on latest anchor only for now)

Update String method of LookupOptions to show FilterOptions as well

Minor renaming inside "memory_test.go" to avoid confusion

Fix problem with partial predicate UUIDs in the volatile driver and make FILTER for latest return multiple triples if they share the same predicate and same latest anchor

Add tests for the FILTER execution in the volatile driver (in "memory_test.go")

Allow user to specify for which bindings in the clause each filter can be applied to, improving error checking, and also move verification for supported filter functions to the planner level

Use a closure to make planner more performant (especially with the addition of multiple filter functions in the future)

Use an "enum" instead of a "string" for filter operations (less error-prone when implementing the driver)

Update "memory.go" and "memory_test.go" since filter operations are "enum" now

Update "hooks_test.go" and "semantic_test.go" since filter operations are "enum" now

Move "storage.FilteringOptions" to "filter.go" as "filter.StorageOptions"

Add a String method for filter.StorageOptions

Use an "enum" instead of a "string" for filter fields (less error-prone when implementing the driver)

Commits on Oct 15, 2020

Update "memory.go" and "memory_test.go" since filter fields are "enum" now

Add an "isEmpty" method for "filter.Operation"

Update comments for "semantic.FilterClause" and "filter.StorageOptions"

Refactor "WhereFilterClauseHook" to make it clearer (separating code into auxiliar functions)

Use "t.Run" for FILTER tests in "memory_test.go", and make it cleaner

Do not use naked returns inside "compatibleBindingsInClauseForFilterOperation" anymore

Return "compatibleBindingsInClause" directly for each switch case inside "compatibleBindingsInClauseForFilterOperation"