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

Remove extra new lines for result of EXPLAIN statement #68

Merged
merged 1 commit into from
Jun 16, 2020

Conversation

yfuruyama
Copy link
Collaborator

I found EXPLAIN statement shows extra new lines like this:

spanner> explain select 1;             
+-------------------------------------+
| Query_Execution_Plan (EXPERIMENTAL) |
+-------------------------------------+                                                                                                    
|                                     |                 
| .                                   |
| +- Serialize Result                 |
|     +- Unit Relation                |
|                                     |
+-------------------------------------+
1 rows in set (0.34 sec)

I don't remember why I appended new line for the result, but anyway now I removed those new lines so it can align with the result of EXPLAIN ANALYZE statement.

spanner> explain select 1;
+-------------------------------------+
| Query_Execution_Plan (EXPERIMENTAL) |
+-------------------------------------+
| .                                   |
| +- Serialize Result                 |
|     +- Unit Relation                |
+-------------------------------------+
1 rows in set (2.01 sec)

spanner> explain analyze select 1;
+----------------------+---------------+------------+---------------+
| Query_Execution_Plan | Rows_Returned | Executions | Total_Latency |
+----------------------+---------------+------------+---------------+
| .                    |               |            |               |
| +- Serialize Result  | 1             | 1          | 0.01 msecs    |
|     +- Unit Relation | 1             | 1          | 0 msecs       |
+----------------------+---------------+------------+---------------+
1 rows in set (2.8 msecs)
timestamp: 2020-06-16T20:10:20.939172+09:00
cpu:       1.06 msecs
scanned:   0 rows
optimizer: 2

@googlebot googlebot added the cla: yes CLA signed label Jun 16, 2020
@yfuruyama yfuruyama requested a review from apstndb June 16, 2020 11:11
Copy link
Collaborator

@apstndb apstndb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@yfuruyama yfuruyama merged commit 433183c into master Jun 16, 2020
@yfuruyama yfuruyama deleted the fix/explain_space branch June 16, 2020 11:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes CLA signed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants