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

Speed up results display #66

Merged
merged 3 commits into from
Dec 29, 2022
Merged

Speed up results display #66

merged 3 commits into from
Dec 29, 2022

Conversation

grzegorz8
Copy link
Member

Previously, there was a short sleep after reading each record. In consequence, it took at least a few secods to show tens or hundreds of rows.

Speed up results display
3120046 
Previously, there was a short sleep after reading each record.
In consequence, it took at least a few secods to show tens
or hundreds of rows.
swtwsk
swtwsk approved these changes Dec 28, 2022
View reviewed changes
Copy link
Contributor

@swtwsk swtwsk left a comment

Choose a reason for hiding this comment

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

It's beautiful

@andrzejdackiewicz andrzejdackiewicz self-requested a review December 28, 2022 12:14
@grzegorz8 grzegorz8 merged commit 29b777d into develop Dec 29, 2022
@grzegorz8 grzegorz8 deleted the speed-up-results-display branch March 13, 2023 13:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andrzejdackiewicz andrzejdackiewicz andrzejdackiewicz approved these changes

@swtwsk swtwsk swtwsk approved these changes

@niladrem niladrem niladrem approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@grzegorz8 @niladrem @swtwsk @andrzejdackiewicz