Skip to content

HOP-7045 [Bug]: Default values overwrite empty fields in TextFileOutput#7079

Closed
fskorgen wants to merge 1 commit into
apache:mainfrom
fskorgen:fix/textfileoutput_overwrite_empty_fields
Closed

HOP-7045 [Bug]: Default values overwrite empty fields in TextFileOutput#7079
fskorgen wants to merge 1 commit into
apache:mainfrom
fskorgen:fix/textfileoutput_overwrite_empty_fields

Conversation

@fskorgen
Copy link
Copy Markdown
Contributor

@fskorgen fskorgen commented May 2, 2026

Moved setting of default values for separator and enclosure from constructor to setDefault
Empty values are now accepted instead of being overridden by defaults


Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

  • Run mvn clean install apache-rat:check to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
  • If you have a group of commits related to the same change, please squash your commits into one and force push your branch using git rebase -i.
  • Mention the appropriate issue in your description (for example: addresses #123), if applicable.

To make clear that you license your contribution under the Apache License Version 2.0, January 2004
you have to acknowledge this by using the following check-box.

@hansva
Copy link
Copy Markdown
Contributor

hansva commented May 4, 2026

This will be a broader issue caused by the deserialization changes introduced, while this fix solves the issue for textfileOutput the same issue might be present in other spots. I am looking at a global fix.

@hansva
Copy link
Copy Markdown
Contributor

hansva commented May 4, 2026

should be fixed with #7087

@hansva hansva closed this May 4, 2026
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