Skip to content

Conversation

@champ-isaac
Copy link
Contributor

@champ-isaac champ-isaac commented Feb 24, 2025

feat(filters): process the same headers but different order of columns for CSV files

Even supposed sources should be consistent. there is a scenario with same formats of CSV files but different order of header columns.
So I implement the process for the same headers but different order of columns for CSV files.

@champ-isaac champ-isaac changed the title process the same headers but different order of columns for CSV files feat(filters): process the same headers but different order of columns for CSV files Feb 25, 2025
@champ-isaac champ-isaac marked this pull request as draft February 26, 2025 10:21
@champ-isaac champ-isaac marked this pull request as ready for review February 26, 2025 10:23
@champ-isaac
Copy link
Contributor Author

Hi @fhussonnois! Could you please take a look at this PR when you have time? thanks

@champ-isaac
Copy link
Contributor Author

hi @fhussonnois I saw the PR build error and it shows that vectorized/redpanda:latest docker image doesn't exist anymore.
could you update to the new url of redpanda docker image?

Caused by: com.github.dockerjava.api.exception.NotFoundException: Status 404: {"message":"pull access denied for vectorized/redpanda, repository does not exist or may require 'docker login': denied: requested access to the resource is denied"}

@fhussonnois
Copy link
Member

Hey @champ-isaac, thank you very much for your PR. You can rebase it with the master branch to re-run the CI. Thanks

@fhussonnois fhussonnois merged commit df731e4 into streamthoughts:master Mar 18, 2025
2 of 3 checks passed
@champ-isaac champ-isaac deleted the process-different-order-of-headers branch April 1, 2025 07:36
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