Skip to content

TOOLS-4148 Pass the errgroup created in streamDocuments to doSequentialStreaming#986

Merged
autarch merged 1 commit intomasterfrom
04-03-tools-4148_pass_the_errgroup_created_in_streamdocuments_to_dosequentialstreaming_
Apr 8, 2026
Merged

TOOLS-4148 Pass the errgroup created in streamDocuments to doSequentialStreaming#986
autarch merged 1 commit intomasterfrom
04-03-tools-4148_pass_the_errgroup_created_in_streamdocuments_to_dosequentialstreaming_

Conversation

@autarch
Copy link
Copy Markdown
Collaborator

@autarch autarch commented Apr 3, 2026

This removes the last bare go func in mongoimport.

@autarch autarch changed the base branch from 04-03-tools-4148_replace_waitgroup_tomb_with_an_errgroup_in_ingestdocuments_ to graphite-base/986 April 3, 2026 21:27
@autarch autarch force-pushed the 04-03-tools-4148_pass_the_errgroup_created_in_streamdocuments_to_dosequentialstreaming_ branch from 6a642dc to a7c9895 Compare April 7, 2026 16:50
@autarch autarch force-pushed the graphite-base/986 branch from 4b83a4c to 231ee85 Compare April 7, 2026 16:50
@autarch autarch changed the base branch from graphite-base/986 to 04-03-tools-4148_replace_waitgroup_tomb_with_an_errgroup_in_ingestdocuments_ April 7, 2026 16:51
@autarch autarch force-pushed the 04-03-tools-4148_pass_the_errgroup_created_in_streamdocuments_to_dosequentialstreaming_ branch from a7c9895 to 6a16202 Compare April 7, 2026 16:55
@autarch autarch force-pushed the 04-03-tools-4148_replace_waitgroup_tomb_with_an_errgroup_in_ingestdocuments_ branch 2 times, most recently from d8b63da to 20dae74 Compare April 7, 2026 21:37
@autarch autarch force-pushed the 04-03-tools-4148_pass_the_errgroup_created_in_streamdocuments_to_dosequentialstreaming_ branch 3 times, most recently from 21b229e to bffbbf6 Compare April 7, 2026 22:08
@autarch autarch force-pushed the 04-03-tools-4148_replace_waitgroup_tomb_with_an_errgroup_in_ingestdocuments_ branch from 20dae74 to ea0773c Compare April 7, 2026 22:08
@autarch autarch requested a review from mmcclimon April 8, 2026 15:28
@autarch autarch marked this pull request as ready for review April 8, 2026 15:28
@autarch autarch requested a review from a team as a code owner April 8, 2026 15:28
@autarch autarch force-pushed the 04-03-tools-4148_replace_waitgroup_tomb_with_an_errgroup_in_ingestdocuments_ branch from ea0773c to b51b373 Compare April 8, 2026 15:32
@autarch autarch force-pushed the 04-03-tools-4148_pass_the_errgroup_created_in_streamdocuments_to_dosequentialstreaming_ branch from bffbbf6 to 4f5e086 Compare April 8, 2026 15:33
Copy link
Copy Markdown
Contributor

@mmcclimon mmcclimon left a comment

Choose a reason for hiding this comment

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

LGTM!

//nolint:errcheck
go iw.processDocuments(t.Context(), true)
eg.Go(
func() error { return iw.processDocuments(t.Context(), true) },
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
func() error { return iw.processDocuments(t.Context(), true) },
func() error { return iw.processDocuments(ctx, true) },

This surely won't matter in practice but is less weird.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Done

@autarch autarch force-pushed the 04-03-tools-4148_pass_the_errgroup_created_in_streamdocuments_to_dosequentialstreaming_ branch from 4f5e086 to 4c3fdbb Compare April 8, 2026 19:53
Copy link
Copy Markdown
Collaborator Author

autarch commented Apr 8, 2026

Merge activity

  • Apr 8, 9:04 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Apr 8, 9:09 PM UTC: Graphite rebased this pull request as part of a merge.
  • Apr 8, 9:10 PM UTC: @autarch merged this pull request with Graphite.

@autarch autarch changed the base branch from 04-03-tools-4148_replace_waitgroup_tomb_with_an_errgroup_in_ingestdocuments_ to graphite-base/986 April 8, 2026 21:06
@autarch autarch changed the base branch from graphite-base/986 to master April 8, 2026 21:07
…ntialStreaming`

This removes the last bare `go func` in mongoimport.
@autarch autarch force-pushed the 04-03-tools-4148_pass_the_errgroup_created_in_streamdocuments_to_dosequentialstreaming_ branch from 4c3fdbb to 4dc12a4 Compare April 8, 2026 21:08
@autarch autarch merged commit 5d5ee94 into master Apr 8, 2026
2 of 3 checks passed
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

Successfully merging this pull request may close these issues.

2 participants