Support the tck format for the output of EXPLAIN #5414
Merged
Conversation
This file contains 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
|
|
This was referenced Mar 19, 2023
xtcyclist
reviewed
Mar 20, 2023
There was a problem hiding this comment.
Hi. Thanks for the contribution. I think an example of using the new format could be added to show how the results look like.
There was a problem hiding this comment.
(root@nebula) [my_space_1]> explain format="tck" FETCH PROP ON player "player_1","player_2","player_3" yield properties(vertex).name as name, properties(vertex).age as age;
Execution succeeded (time spent 261µs/613.718µs)
Execution Plan (optimize time 28 us)
| id | name | dependencies | profiling data | operator info |
| 2 | Project | 1 | | |
| 1 | GetVertices | 0 | | |
| 0 | Start | | | |
Wed, 22 Mar 2023 23:15:52 CST
(root@nebula) [my_space_1]> explain format="tck" show hosts;
Execution succeeded (time spent 158µs/496.504µs)
Execution Plan (optimize time 23 us)
| id | name | dependencies | profiling data | operator info |
| 1 | ShowHosts | 0 | | |
| 0 | Start | | | |
Wed, 22 Mar 2023 23:16:02 CST
(root@nebula) [my_space_1]> profile format="tck" FETCH PROP ON player "player_1","player_2","player_3" yield properties(vertex).name as name, properties(vertex).age as age;
| name | age |
| "Piter Park" | 24 |
| "aaa" | 24 |
| "ccc" | 24 |
Got 3 rows (time spent 1.474ms/2.19677ms)
Execution Plan (optimize time 41 us)
| id | name | dependencies | profiling data | operator info |
| 2 | Project | 1 | {"rows":3,"version":0} | |
| 1 | GetVertices | 0 | {"resp[0]":{"exec":"232(us)","host":"127.0.0.1:9779","total":"758(us)"},"rows":3,"total_rpc":"875(us)","version":0} | |
| 0 | Start | | {"rows":0,"version":0} | |
Wed, 22 Mar 2023 23:16:13 CST
(root@nebula) [my_space_1]> profile format="tck" show hosts;
| Host | Port | Status | Leader count | Leader distribution | Partition distribution | Version |
| "127.0.0.1" | 9779 | "ONLINE" | 100 | "my_space_1:100" | "my_space_1:100" | "" |
Got 1 rows (time spent 1.156ms/1.627659ms)
Execution Plan (optimize time 55 us)
| id | name | dependencies | profiling data | operator info |
| 1 | ShowHosts | 0 | {"rows":1,"version":0} | |
| 0 | Start | | {"rows":0,"version":0} | |
Wed, 22 Mar 2023 23:16:28 CST
(root@nebula) [my_space_1]>
(root@nebula) [my_space_1]> profile format="row" FETCH PROP ON player "player_1","player_2","player_3" yield properties(vertex).name as name, properties(vertex).age as age;
+--------------+-----+
| name | age |
+--------------+-----+
| "Piter Park" | 24 |
| "aaa" | 24 |
| "ccc" | 24 |
+--------------+-----+
Got 3 rows (time spent 1.716ms/2.344165ms)
Execution Plan (optimize time 29 us)
-----+-------------+--------------+-------------------------------+---------------------------------------
| id | name | dependencies | profiling data | operator info |
-----+-------------+--------------+-------------------------------+---------------------------------------
| 2 | Project | 1 | { | outputVar: { |
| | | | "execTime": "27(us)", | "colNames": [ |
| | | | "rows": 3, | "name", |
| | | | "totalTime": "30(us)", | "age" |
| | | | "version": 0 | ], |
| | | | } | "type": "DATASET", |
| | | | | "name": "__Project_2" |
| | | | | } |
| | | | | inputVar: __GetVertices_1 |
| | | | | columns: [ |
| | | | | "properties(VERTEX).name AS name", |
| | | | | "properties(VERTEX).age AS age" |
| | | | | ] |
-----+-------------+--------------+-------------------------------+---------------------------------------
| 1 | GetVertices | 0 | { | outputVar: { |
| | | | "execTime": "113(us)", | "colNames": [], |
| | | | "resp[0]": { | "type": "DATASET", |
| | | | "exec": "317(us)", | "name": "__GetVertices_1" |
| | | | "host": "127.0.0.1:9779", | } |
| | | | "total": "1045(us)" | inputVar: __VAR_0 |
| | | | }, | space: 2 |
| | | | "rows": 3, | dedup: false |
| | | | "totalTime": "1311(us)", | limit: 9223372036854775807 |
| | | | "total_rpc": "1214(us)", | filter: |
| | | | "version": 0 | orderBy: [] |
| | | | } | src: COLUMN[0] |
| | | | | props: [ |
| | | | | { |
| | | | | "props": [ |
| | | | | "_tag", |
| | | | | "age", |
| | | | | "name" |
| | | | | ], |
| | | | | "tagId": 3 |
| | | | | } |
| | | | | ] |
| | | | | exprs: |
-----+-------------+--------------+-------------------------------+---------------------------------------
| 0 | Start | | { | outputVar: { |
| | | | "execTime": "0(us)", | "colNames": [], |
| | | | "rows": 0, | "type": "DATASET", |
| | | | "totalTime": "29(us)", | "name": "__Start_0" |
| | | | "version": 0 | } |
| | | | } | |
-----+-------------+--------------+-------------------------------+---------------------------------------
Wed, 22 Mar 2023 23:25:14 CST
jievince
reviewed
Mar 20, 2023
xtcyclist
previously approved these changes
Mar 29, 2023
xtcyclist
approved these changes
Mar 30, 2023
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.
What type of PR is this?
What problem(s) does this PR solve?
Issue(s) number:
#5274
Description:
Support the tck format for the output of EXPLAIN
How do you solve it?
Explain/Profile has added support for the TCK output format, with the "TCK" string being appended to the kAllowedFmtType list.
Special notes for your reviewer, ex. impact of this fix, design document, etc:
related link:
Checklist:
Tests:
Affects:
Release notes:
Please confirm whether to be reflected in release notes and how to describe: