feat: Add ability to Explain the response plan.#385

Merged

shahzadlone merged 24 commits intodevelopfrom lone/i-325/simple-explain-in-json

Jun 1, 2022

+3,024-392

Commits on May 31, 2022

wip: Add parsing of the @explain directive successfully.
shahzadlone
committed
wip: Add explaining implementation using Kind() method (no attributes, yet).
shahzadlone
committed
wip: Add ability to return the explain result in JSON.
shahzadlone
committed
wip: Add integration tests for explaining topSelectNode, selectNode, and the scanNode with their attributes.
shahzadlone
committed
wip: Adhere to the review suggestions #1 .
shahzadlone
committed
wip: Fix valuesNode to satisfy the planNode interface.
shahzadlone
committed
wip: Fix unit test to adhere to the plain key-able JSON style.
shahzadlone
committed
wip: Remove anon-structs and use nano-sub package for types.
shahzadlone
committed
wip: Adhere to the review suggestions #2 .
shahzadlone
committed
wip: Add ability to explain deleteNode and handle errors for
shahzadlone
committed
wip: Add deleteNode explain support and test deleteNode and
shahzadlone
committed
wip: Convert if statements to a type-switch.
shahzadlone
committed
wip: Add ability to explain typeIndexJoin nodes.
shahzadlone
committed
wip: Handle wrapping plan.Close() error instead of logging it.
shahzadlone
committed
wip: Implement all the explainable nodes and move them all to their
shahzadlone
committed
wip: Handle wrapping errors of executeRequest function with a helper
shahzadlone
committed
wip: Link issue number to todos or the ones that aren't needed.
shahzadlone
committed
wip: Adhere to code-review #3
shahzadlone
committed
wip: Keep the plan nil checks inside the inner functions.
shahzadlone
committed
wip: type alias dataMap to map[string]interface{} in tests.
shahzadlone
committed
wip: Leave the types inside planner package rather than types/types.go.
shahzadlone
committed
wip: Fix #486 , wrapped scanNode bug by making multiscanNode adhere to
shahzadlone
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: Add ability to Explain the response plan.#385

Commits on May 31, 2022

wip: Add parsing of the `@explain` directive successfully.

wip: Add explaining implementation using Kind() method (no attributes, yet).

wip: Add ability to return the explain result in JSON.

wip: Add integration tests for explaining `topSelectNode`, `selectNode`, and the `scanNode` with their attributes.

wip: Adhere to the review suggestions #1 .

wip: Fix `valuesNode` to satisfy the `planNode` interface.

wip: Fix unit test to adhere to the plain key-able JSON style.

wip: Remove anon-structs and use nano-sub package for types.

wip: Adhere to the review suggestions #2 .

wip: Add ability to explain `deleteNode` and handle errors for

wip: Add deleteNode explain support and test deleteNode and

wip: Convert if statements to a type-switch.

wip: Add ability to explain `typeIndexJoin` nodes.

wip: Handle wrapping plan.Close() error instead of logging it.

wip: Implement all the explainable nodes and move them all to their

wip: Handle wrapping errors of `executeRequest` function with a helper

wip: Link issue number to todos or the ones that aren't needed.

wip: Adhere to code-review #3

wip: Keep the plan nil checks inside the inner functions.

wip: type alias `dataMap` to `map[string]interface{}` in tests.

wip: Leave the types inside planner package rather than `types/types.go`.

wip: Fix #486 , wrapped scanNode bug by making multiscanNode adhere to

Commits on Jun 1, 2022

wip: Make a `parseExplainDirective` function that iterates through all

wip: Support return of a `nil` from Explain to signal empty attributes.