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.

Sign up for GitHub

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

Jump to bottom

Execute SQL window: Move query executiontime to top of success/error pane #381

Closed
pamtbaau opened this issue Jul 9, 2015 · 4 comments
Closed

Execute SQL window: Move query executiontime to top of success/error pane #381

pamtbaau opened this issue Jul 9, 2015 · 4 comments
Assignees
@MKleusberg
Labels
enhancement Feature requests.
Milestone
3.8.0 - Secret Sa...

Comments

@pamtbaau
Copy link

pamtbaau commented Jul 9, 2015

Together with amount of rows returned, the amount of time the query took is the most important info if the query is successful. This info is at the bottom of the success/error pane. Could that info be on the first line together with the amount of rows? E.g. "5 Rows returned in 115ms from:\n"
MKleusberg added a commit that referenced this issue Jul 9, 2015
@MKleusberg
Optimise success message in Execute SQL tab
86fb175 
Change the success message for SELECT statement so that the amount of
time taken comes before the SQL statement itself because the time is a
new information whereas the statement is just a repitition for the user.

See issue #381.
@MKleusberg
Copy link
Member

Sorry, I'm a bit in a hurry right now but I've just pushed a commit for each of your issues. I'm quite confident this one will be fixed in the next nightly build 😃 The code for #380 has been changed as well, but we'll have to see how it works for you.

@justinclift justinclift added the enhancement Feature requests. label Jul 9, 2015
@justinclift
Copy link
Member

@pamtbaau Is this one working well for you now too? 😄

@pamtbaau
Copy link
Author

@justinclift Yes, this issue may be closed as well

@justinclift
Copy link
Member

Thanks. 😄

@justinclift justinclift added this to the 3.8.0 - Awesome Sauce! milestone Jul 14, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MKleusberg MKleusberg

Labels
enhancement Feature requests.
Projects
None yet
Milestone
3.8.0 - Secret Santa :)
Development

No branches or pull requests
3 participants
@justinclift @MKleusberg @pamtbaau