-
Notifications
You must be signed in to change notification settings - Fork 183
[DOC-12349] Update Query Execution Diagram #3962
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
base: release/8.0
Are you sure you want to change the base?
Conversation
| @@ -0,0 +1,3 @@ | |||
| sources: | |||
| docs-devex: | |||
| branches: [DOC-12349-query-execution-diagram] | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Delete this before merging!
simon-dew
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Otherwise LGTM
| Next, it performs Fetch operations to get the actual data from the Data Service, and then uses this data for Join operations. | ||
|
|
||
| The Query Service processes the data further by applying Filter, Aggregate, Project, and Sort operations. | ||
| These operations often run in parallel, as represented by the vertically aligned groups of right-pointing arrows. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The parallelism is represented by the little boxes within each operation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can't merge this until someone from Query has reviewed the diagram at
This PR updates the Query Execution diagram and its description to match the latest changes.
Jira: DOC-12349
Docs Preview: Query Execution
Must be merged at the same time as PR 469, as it includes the updated PlantUML file.
The changes also need to be backported to 7.6 and 7.2.