Skip to content

Commit

Permalink
fix wording
Browse files Browse the repository at this point in the history
  • Loading branch information
stefano-ottolenghi committed Jul 3, 2024
1 parent 77656b3 commit 8da5c23
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion go-manual/modules/ROOT/pages/performance.adoc
Original file line number Diff line number Diff line change
Expand Up @@ -237,7 +237,7 @@ func timer(name string) func() {
-- eagerLoading took 2m29.113482541s -- // <3>
----
<1> With lazy loading, the first record is available almost instantly (i.e. as soon as the server has retrieved it).
<1> With lazy loading, the first record is quickly available.
<2> With eager loading, the first record is available ~25 seconds after the query has been submitted (i.e. after the server has retrieved all 250 records).
<3> The total running time is lower with lazy loading, because while the client processes records the server can fetch the next ones.
With lazy loading, the client could also stop requesting records after some condition is met (by calling `.Consume(ctx)` on the `Result`), saving time and resources.
Expand Down
2 changes: 1 addition & 1 deletion python-manual/modules/ROOT/pages/performance.adoc
Original file line number Diff line number Diff line change
Expand Up @@ -193,7 +193,7 @@ if __name__ == '__main__':
[1719984861.09] --- 149.70468592643738 seconds --- // <4>
----
<1> With lazy loading, the first record is available almost instantly (i.e. as soon as the server has retrieved it).
<1> With lazy loading, the first record is quickly available.
<2> With eager loading, the first record is available ~25 seconds after the query has been submitted (i.e. after the server has retrieved all 250 records).
<3> Memory usage is larger with eager loading than with lazy loading, because the application materializes a list of 250 records.
<4> The total running time is lower with lazy loading, because while the client processes records the server can fetch the next ones.
Expand Down

0 comments on commit 8da5c23

Please sign in to comment.