Skip to content

ETL: Fix split_all_docs to return the splits#12

Closed
awilfox wants to merge 0 commit intomainfrom
awilfox/fix-split-oops
Closed

ETL: Fix split_all_docs to return the splits#12
awilfox wants to merge 0 commit intomainfrom
awilfox/fix-split-oops

Conversation

@awilfox
Copy link
Copy Markdown
Member

@awilfox awilfox commented Aug 6, 2025

My suggestion in the AP-369 PR wasn't exactly correct. We need to return the actual split documents, not a list of lists containing them.

Fixes: eef344b ("AP-369 Doc processing (#10)")

@awilfox awilfox requested a review from jason-raitz August 6, 2025 21:46
@awilfox awilfox self-assigned this Aug 6, 2025
Copy link
Copy Markdown
Contributor

@jason-raitz jason-raitz left a comment

Choose a reason for hiding this comment

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

Lgtm

@awilfox awilfox closed this Aug 7, 2025
@awilfox awilfox force-pushed the awilfox/fix-split-oops branch from f8e1337 to e7752d8 Compare August 7, 2025 16:33
@awilfox awilfox deleted the awilfox/fix-split-oops branch August 7, 2025 16:34
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