Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Bug]: Redshift Bulk Loader adds padding to the generated csv file in some scenarios #3823

Closed
bamaer opened this issue Apr 18, 2024 · 1 comment
Closed

[Bug]: Redshift Bulk Loader adds padding to the generated csv file in some scenarios #3823

bamaer opened this issue Apr 18, 2024 · 1 comment
Assignees
@bamaer
Labels
bug Database P3 Nice to have Transforms
Milestone
2.9

Comments

@bamaer
Copy link
Contributor

bamaer commented Apr 18, 2024

Apache Hop version?

SNAPSHOT-20240418

Java version?

openjdk version "11.0.22" 2024-01-16 LTS

Operating system

macOS

What happened?

The Redshift Bulk Loader transform contains code that either truncates or pads string fields up to the desired length.
Truncating is useful, but padding creates more problems than it solves.
Suggested solution is to remove the padding functionality for now and add it back later as an option if required.

Issue Priority

Priority: 3

Issue Component

Component: Database, Component: Transforms
@bamaer
Copy link
Contributor Author

bamaer commented Apr 18, 2024

.take-issue

@hansva hansva added this to the 2.9 milestone Apr 19, 2024
@hansva hansva closed this as completed in b8a511a Apr 19, 2024
hansva added a commit that referenced this issue Apr 19, 2024
@hansva
Merge pull request #3824 from bamaer/3823
931e652 
removed padding from Redshift bulk loader. fixes #3823
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bamaer bamaer

Labels
bug Database P3 Nice to have Transforms
Projects
None yet
Milestone
2.9
Development

No branches or pull requests
2 participants
@bamaer @hansva