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

Read data in bulks for non-foreground reporters #91

Open
jgebal opened this issue Jul 17, 2019 · 2 comments
Open

Read data in bulks for non-foreground reporters #91

jgebal opened this issue Jul 17, 2019 · 2 comments

Comments

@jgebal
Copy link
Member

jgebal commented Jul 17, 2019

Currently, when running tests with coverage, where the data produced by coverage report is large, each reporter is most probably running a row-by-row query from client to DB.

We should prevent this as it significantly impacts the overall execution time of run when done va utPLSQL-cli.

Ideally, data from reporters that do not report to screen but do reporting in background , should be read in bulks of 100, 1000 or even 10000 rows.

If you have a look at our test runs on Travis,
~50 seconds is time for utPSLQL unit test run, while overall end-to-end run takes ~80 seconds.
I can imagine that on really large schema, most of the time might be spent on reading coverage report by cli.

@jgebal
Copy link
Member Author

jgebal commented Jul 17, 2019

Alternatively, we could add additional flag to reporters, so they can indicate if they are real-time or post-run reporting.

That way you could additionally decide what should be read in real-time based on reporter capability .

@pesse
Copy link
Member

pesse commented Jul 17, 2019

Sounds like a great idea, though I think this should be done in cli and has utPLSQL/utPLSQL-cli#120 as a precondition.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants