Skip to content

Conversation

@nickschuch
Copy link
Collaborator

@nickschuch nickschuch commented Sep 1, 2025

Overview

The RDS provider was using fmt.Println to output the S3 OUTFILE statement.

This should have used the writer to ensure all dump file output can be written anywhere and tested.

Solution

  • Updated each provider to implement WriteTableData instead
  • Move stdout data handling into the stdout provider
  • Update RDS provider to utilise the writer

@nickschuch nickschuch merged commit 4d8e577 into main Sep 8, 2025
3 checks passed
@nickschuch nickschuch deleted the rds-writer branch September 8, 2025 09:26
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